s3:libsmb: Fix clang warnings that fnum might be used uninitialized
authorAndreas Schneider <asn@samba.org>
Wed, 25 Nov 2020 10:34:09 +0000 (11:34 +0100)
committerAndreas Schneider <asn@cryptomilk.org>
Thu, 26 Nov 2020 09:44:41 +0000 (09:44 +0000)
Found by covscan.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source3/libsmb/cli_smb2_fnum.c

index 2dd76de967a03223394a869187e991e38f6cbc44..f70639e41bd49294ec0bc804cb6de89387785468 100644 (file)
@@ -815,7 +815,7 @@ static void cli_smb2_mkdir_opened(struct tevent_req *subreq)
        struct cli_smb2_mkdir_state *state = tevent_req_data(
                req, struct cli_smb2_mkdir_state);
        NTSTATUS status;
-       uint16_t fnum;
+       uint16_t fnum = 0xffff;
 
        status = cli_smb2_create_fnum_recv(subreq, &fnum, NULL, NULL, NULL);
        TALLOC_FREE(subreq);
@@ -1082,7 +1082,7 @@ static void cli_smb2_unlink_opened1(struct tevent_req *subreq)
                subreq, struct tevent_req);
        struct cli_smb2_unlink_state *state = tevent_req_data(
                req, struct cli_smb2_unlink_state);
-       uint16_t fnum;
+       uint16_t fnum = 0xffff;
        NTSTATUS status;
 
        status = cli_smb2_create_fnum_recv(subreq, &fnum, NULL, NULL, NULL);
@@ -1135,7 +1135,7 @@ static void cli_smb2_unlink_opened2(struct tevent_req *subreq)
                subreq, struct tevent_req);
        struct cli_smb2_unlink_state *state = tevent_req_data(
                req, struct cli_smb2_unlink_state);
-       uint16_t fnum;
+       uint16_t fnum = 0xffff;
        NTSTATUS status;
 
        status = cli_smb2_create_fnum_recv(subreq, &fnum, NULL, NULL, NULL);
@@ -1682,7 +1682,7 @@ static void cli_smb2_chkpath_opened(struct tevent_req *subreq)
        struct cli_smb2_chkpath_state *state = tevent_req_data(
                req, struct cli_smb2_chkpath_state);
        NTSTATUS status;
-       uint16_t fnum;
+       uint16_t fnum = 0xffff;
 
        status = cli_smb2_create_fnum_recv(subreq, &fnum, NULL, NULL, NULL);
        TALLOC_FREE(subreq);