s3/libsmd: clang: Fix some uninitialized value errors
authorNoel Power <noel.power@suse.com>
Tue, 13 Aug 2019 14:22:37 +0000 (15:22 +0100)
committerRalph Boehme <slow@samba.org>
Tue, 22 Oct 2019 17:28:27 +0000 (17:28 +0000)
Fixes:

source3/libsmb/clilist.c:534:14: warning: 1st function call argument is an uninitialized value <--[clang]
        num_finfo = talloc_array_length(finfo);
                    ^
source3/libsmb/clilist.c:1007:13: warning: The right operand of '<' is a garbage value <--[clang]
        for (i=0; i<num_finfo; i++) {
                   ^~~~~~~~~~
2 warnings generated.

Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: Ralph Boehme <slow@samba.org>
source3/libsmb/clilist.c

index 5cb1fce4338b17c3ba0ca4ef016a629755038d31..58dac65f4c73cf08628cb5b3e40b82635f3de890 100644 (file)
@@ -506,7 +506,7 @@ NTSTATUS cli_list_old(struct cli_state *cli, const char *mask,
        struct tevent_context *ev;
        struct tevent_req *req;
        NTSTATUS status = NT_STATUS_NO_MEMORY;
-       struct file_info *finfo;
+       struct file_info *finfo = NULL;
        size_t i, num_finfo;
 
        if (smbXcli_conn_has_async_calls(cli->conn)) {
@@ -967,7 +967,7 @@ NTSTATUS cli_list(struct cli_state *cli, const char *mask, uint16_t attribute,
        struct tevent_req *req;
        NTSTATUS status = NT_STATUS_NO_MEMORY;
        struct file_info *finfo;
-       size_t i, num_finfo;
+       size_t i, num_finfo = 0;
        uint16_t info_level;
 
        if (smbXcli_conn_protocol(cli->conn) >= PROTOCOL_SMB2_02) {