s3:rpc_server: Initialize auxiliary variable checked in error path
authorSamuel Cabrero <scabrero@suse.de>
Wed, 27 Feb 2019 15:54:49 +0000 (16:54 +0100)
committerStefan Metzmacher <metze@samba.org>
Mon, 22 Jul 2019 16:49:15 +0000 (16:49 +0000)
The auxiliary variable is checked in the function's exit code so reset
it after each usage.

Signed-off-by: Samuel Cabrero <scabrero@suse.de>
Reviewed-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
source3/rpc_server/lsasd.c
source3/rpc_server/mdssd.c

index 5bc0b740fa29abda7a9c619a491cd32648e65795..be5e1af6c8cac11574bcb578af88ada9adbf9a8c 100644 (file)
@@ -651,6 +651,7 @@ static bool lsasd_create_sockets(struct tevent_context *ev_ctx,
        listen_fd[*listen_fd_size].fd = fd;
        listen_fd[*listen_fd_size].fd_data = NULL;
        (*listen_fd_size)++;
+       fd = -1;
 
        status = dcesrv_create_ncacn_np_socket("lsass", &fd);
        if (!NT_STATUS_IS_OK(status)) {
@@ -666,6 +667,7 @@ static bool lsasd_create_sockets(struct tevent_context *ev_ctx,
        listen_fd[*listen_fd_size].fd = fd;
        listen_fd[*listen_fd_size].fd_data = NULL;
        (*listen_fd_size)++;
+       fd = -1;
 
        status = dcesrv_create_ncalrpc_socket("lsarpc", &fd);
        if (!NT_STATUS_IS_OK(status)) {
@@ -723,6 +725,7 @@ static bool lsasd_create_sockets(struct tevent_context *ev_ctx,
        listen_fd[*listen_fd_size].fd = fd;
        listen_fd[*listen_fd_size].fd_data = NULL;
        (*listen_fd_size)++;
+       fd = -1;
 
        status = dcesrv_create_ncalrpc_socket("samr", &fd);
        if (!NT_STATUS_IS_OK(status)) {
@@ -780,6 +783,7 @@ static bool lsasd_create_sockets(struct tevent_context *ev_ctx,
        listen_fd[*listen_fd_size].fd = fd;
        listen_fd[*listen_fd_size].fd_data = NULL;
        (*listen_fd_size)++;
+       fd = -1;
 
        status = dcesrv_create_ncalrpc_socket("netlogon", &fd);
        if (!NT_STATUS_IS_OK(status)) {
index 37200e996f752e9c8626bb4fbeeb039e453acd18..08c5c260111e7c016d80cce27b5d1ccb7cdf7f3a 100644 (file)
@@ -570,6 +570,7 @@ static bool mdssd_create_sockets(struct tevent_context *ev_ctx,
        listen_fd[*listen_fd_size].fd = fd;
        listen_fd[*listen_fd_size].fd_data = NULL;
        (*listen_fd_size)++;
+       fd = -1;
 
        status = dcesrv_create_ncalrpc_socket("mdssvc", &fd);
        if (!NT_STATUS_IS_OK(status)) {