The branch, master has been updated via 2575e4066bf s3/libsmb: clang: Fix 'warning: Assigned value is garbage or undefined ' via ad8fd31441f s3/libsmb: clang: Fix 'Assigned value is garbage or undefined' via fcb0355a74e s3/libsmb: clang: Fix '2nd func call argument is uninitalized value' via e1ef9922368 s3/libsmb: clang: Fix 'The left operand of '>' or '<' is a garbage value via 78ca70925ca libcli/auth: clang: Fix 'Value stored to 'status' is never read' via e874cc909a7 s3/libsmb: clang: Fix 'Value stored to 'p' is never read' warning via bf567380a7c s3/libsmb: clang: Fix 'Value stored to 'ea_size' is never read' via ce9fef5fd5e s3/libsmb: clang: Fix 'Assigned value is garbage or undefined' via 4bb9255473d s3/libsmb: clang: Fix 'The left operand of '&' is a garbage value' via e06d34e98e1 s3/libsmb: clang: Fix 'The left operand of '!=' is a garbage value' from 8190993d992 ctdb-recoverd: Fix typo in previous fix
https://git.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit 2575e4066bf84d7a56f0b7126841f4ea5cc027fd Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 14:53:01 2019 +0100 s3/libsmb: clang: Fix 'warning: Assigned value is garbage or undefined ' Fixes: source3/libsmb/clireadwrite.c:1259:17: warning: Assigned value is garbage or undefined <--[clang] state->written += written; ^ ~~~~~~~ source3/libsmb/clireadwrite.c:1722:14: warning: The left operand of '>' is a garbage value <--[clang] if (written > expected) { ~~~~~~~ ^ Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> Autobuild-User(master): Gary Lockyer <g...@samba.org> Autobuild-Date(master): Wed Aug 28 00:22:49 UTC 2019 on sn-devel-184 commit ad8fd31441f8012ae01e972d7faeba128507c5d1 Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 14:51:23 2019 +0100 s3/libsmb: clang: Fix 'Assigned value is garbage or undefined' Fixes: source3/libsmb/clireadwrite.c:851:10: warning: Assigned value is garbage or undefined <--[clang] *nread = ret; ^ Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit fcb0355a74ea1c5e446c8465dd2fb017970d408e Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 14:49:51 2019 +0100 s3/libsmb: clang: Fix '2nd func call argument is uninitalized value' Fixes: source3/libsmb/clireadwrite.c:805:2: warning: 2nd function call argument is an uninitialized value <--[clang] memcpy(state->buf, buf, received); ^ ~~~ Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit e1ef99223684e830e4bd5c8e7b33323e25ffafc1 Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 14:42:06 2019 +0100 s3/libsmb: clang: Fix 'The left operand of '>' or '<' is a garbage value Fixes: source3/libsmb/clireadwrite.c:601:15: warning: The left operand of '>' is a garbage value <--[clang] if (received > expected) { source3/libsmb/clireadwrite.c:799:16: warning: The left operand of '<' is a garbage value <--[clang] if ((received < 0) || (received > state->buflen)) { ~~~~~~~~ ^ Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit 78ca70925ca00ae96065db4fd7a963cb999f2f3d Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 14:31:07 2019 +0100 libcli/auth: clang: Fix 'Value stored to 'status' is never read' Fixes: libcli/auth/netlogon_creds_cli.c:2622:2: warning: Value stored to 'status' is never read <--[clang] status = netlogon_creds_decrypt_samlogon_validation(&state->tmp_creds, ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit e874cc909a7f12286ae2514ceb063193469a8946 Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 14:28:16 2019 +0100 s3/libsmb: clang: Fix 'Value stored to 'p' is never read' warning Fixes: source3/libsmb/clirap2.c:1197:4: warning: Value stored to 'p' is never read <--[clang] p += rap_getstringp(frame, ^ ~~~~~~~~~~~~~~~~~~~~~ source3/libsmb/clirap2.c:1525:5: warning: Value stored to 'p' is never read <--[clang] p += rap_getstring(frame, ^ ~~~~~~~~~~~~~~~~~~~~ source3/libsmb/clirap2.c:2358:3: warning: Value stored to 'p' is never read <--[clang] p += WORDSIZE; /* skip rsize */ ^ ~~~~~~~~ source3/libsmb/clirap2.c:2380:3: warning: Value stored to 'p' is never read <--[clang] p += rap_getstringp(frame, ^ ~~~~~~~~~~~~~~~~~~~~~ 4 warnings generated. Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit bf567380a7cc69bf4dc4cf7bad6bc774fcef6085 Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 12:21:32 2019 +0100 s3/libsmb: clang: Fix 'Value stored to 'ea_size' is never read' Fixes: source3/libsmb/clifile.c:5290:2: warning: Value stored to 'ea_size' is never read <--[clang] ea_size = (size_t)IVAL(rdata,0); ^ ~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit ce9fef5fd5ea5475748e99ba1e72abcb557afd83 Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 11:59:05 2019 +0100 s3/libsmb: clang: Fix 'Assigned value is garbage or undefined' Fixes: source3/libsmb/clifile.c:4898:10: warning: Assigned value is garbage or undefined <--[clang] *bsize = (uint64_t)old_bsize; ^ ~~~~~~~~~~~~~~~~~~~ source3/libsmb/clifile.c:4901:10: warning: Assigned value is garbage or undefined <--[clang] *total = (uint64_t)old_total; ^ ~~~~~~~~~~~~~~~~~~~ source3/libsmb/clifile.c:4904:10: warning: Assigned value is garbage or undefined <--[clang] *avail = (uint64_t)old_avail; Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit 4bb9255473dae1e55b7711a9a2851533503ddb22 Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 11:40:46 2019 +0100 s3/libsmb: clang: Fix 'The left operand of '&' is a garbage value' Fixes: source3/libsmb/clifile.c:3108:26: warning: The left operand of '&' is a garbage value <--[clang] (cr.file_attributes & FILE_ATTRIBUTE_DIRECTORY)) ~~~~~~~~~~~~~~~~~~ ^ Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> commit e06d34e98e1d01d04997c342255595ddf74fb388 Author: Noel Power <noel.po...@suse.com> Date: Wed Aug 14 11:25:19 2019 +0100 s3/libsmb: clang: Fix 'The left operand of '!=' is a garbage value' Fixes: source3/libsmb/clifile.c:789:15: warning: The left operand of '!=' is a garbage value <--[clang] if (num_data != 100) { ~~~~~~~~ ^ Signed-off-by: Noel Power <noel.po...@suse.com> Reviewed-by: Gary Lockyer <g...@catalyst.net.nz> ----------------------------------------------------------------------- Summary of changes: libcli/auth/netlogon_creds_cli.c | 2 +- source3/libsmb/clifile.c | 7 +++---- source3/libsmb/clirap2.c | 7 +++---- source3/libsmb/clireadwrite.c | 10 +++++----- 4 files changed, 12 insertions(+), 14 deletions(-) Changeset truncated at 500 lines: diff --git a/libcli/auth/netlogon_creds_cli.c b/libcli/auth/netlogon_creds_cli.c index 50a5f50a57d..c5a100c3c0e 100644 --- a/libcli/auth/netlogon_creds_cli.c +++ b/libcli/auth/netlogon_creds_cli.c @@ -2622,7 +2622,7 @@ static void netlogon_creds_cli_LogonSamLogon_done(struct tevent_req *subreq) status = netlogon_creds_decrypt_samlogon_validation(&state->tmp_creds, state->validation_level, state->validation); - if (tevent_req_nterror(req, result)) { + if (tevent_req_nterror(req, status)) { netlogon_creds_cli_LogonSamLogon_cleanup(req, result); return; } diff --git a/source3/libsmb/clifile.c b/source3/libsmb/clifile.c index e696b536093..987b04d0a66 100644 --- a/source3/libsmb/clifile.c +++ b/source3/libsmb/clifile.c @@ -777,7 +777,7 @@ static void cli_posix_stat_done(struct tevent_req *subreq) struct stat_state *state = tevent_req_data(req, struct stat_state); SMB_STRUCT_STAT *sbuf = state->sbuf; uint8_t *data; - uint32_t num_data; + uint32_t num_data = 0; NTSTATUS status; status = cli_qpathinfo_recv(subreq, state, &data, &num_data); @@ -3004,7 +3004,7 @@ NTSTATUS cli_open(struct cli_state *cli, const char *fname, int flags, unsigned int openfn = 0; unsigned int dos_deny = 0; uint32_t access_mask, share_mode, create_disposition, create_options; - struct smb_create_returns cr; + struct smb_create_returns cr = {0}; /* Do the initial mapping into OpenX parameters. */ if (flags & O_CREAT) { @@ -4839,7 +4839,7 @@ NTSTATUS cli_disk_size(struct cli_state *cli, const char *path, uint64_t *bsize, { uint64_t sectors_per_block; uint64_t bytes_per_sector; - int old_bsize, old_total, old_avail; + int old_bsize = 0, old_total = 0, old_avail = 0; NTSTATUS status; if (smbXcli_conn_protocol(cli->conn) >= PROTOCOL_SMB2_02) { @@ -5287,7 +5287,6 @@ static bool parse_ea_blob(TALLOC_CTX *ctx, const uint8_t *rdata, return false; } - ea_size = (size_t)IVAL(rdata,0); p = rdata + 4; for (num_eas = 0; num_eas < *pnum_eas; num_eas++ ) { diff --git a/source3/libsmb/clirap2.c b/source3/libsmb/clirap2.c index 3dd299c1cb3..fdff34fcf64 100644 --- a/source3/libsmb/clirap2.c +++ b/source3/libsmb/clirap2.c @@ -1194,7 +1194,7 @@ int cli_NetFileGetInfo(struct cli_state *cli, uint32_t file_id, void (*fn)(const rdata, converter, endp); - p += rap_getstringp(frame, + rap_getstringp(frame, p, &fuser, rdata, @@ -1522,7 +1522,7 @@ bool cli_get_pdc_name(struct cli_state *cli, const char *workgroup, char **pdc_n if (count > 0) { TALLOC_CTX *frame = talloc_stackframe(); char *dcname; - p += rap_getstring(frame, + rap_getstring(frame, p, &dcname, endp); @@ -2355,7 +2355,6 @@ int cli_NetSessionGetInfo(struct cli_state *cli, const char *workstation, p = rparam + WORDSIZE; GETWORD(p, converter,endp); - p += WORDSIZE; /* skip rsize */ p = rdata; endp = rdata + rdrcnt; @@ -2377,7 +2376,7 @@ int cli_NetSessionGetInfo(struct cli_state *cli, const char *workstation, GETDWORD(p, sess_time, endp); GETDWORD(p, idle_time, endp); GETDWORD(p, user_flags, endp); - p += rap_getstringp(frame, + rap_getstringp(frame, p, &clitype_name, rdata, diff --git a/source3/libsmb/clireadwrite.c b/source3/libsmb/clireadwrite.c index da188db7c31..9349b9485e8 100644 --- a/source3/libsmb/clireadwrite.c +++ b/source3/libsmb/clireadwrite.c @@ -580,7 +580,7 @@ static void cli_pull_chunk_done(struct tevent_req *subreq) struct cli_pull_state); NTSTATUS status; size_t expected = chunk->total_size - chunk->tmp_size; - ssize_t received; + ssize_t received = 0; uint8_t *buf = NULL; chunk->subreq = NULL; @@ -781,7 +781,7 @@ static void cli_read_done(struct tevent_req *subreq) req, struct cli_read_state); NTSTATUS status; ssize_t received; - uint8_t *buf; + uint8_t *buf = NULL; if (smbXcli_conn_protocol(state->cli->conn) >= PROTOCOL_SMB2_02) { status = cli_smb2_read_recv(subreq, &received, &buf); @@ -796,7 +796,7 @@ static void cli_read_done(struct tevent_req *subreq) if (tevent_req_nterror(req, status)) { return; } - if ((received < 0) || (received > state->buflen)) { + if ((buf == NULL) || (received < 0) || (received > state->buflen)) { state->received = 0; tevent_req_nterror(req, NT_STATUS_UNEXPECTED_IO_ERROR); return; @@ -839,7 +839,7 @@ NTSTATUS cli_read(struct cli_state *cli, uint16_t fnum, size_t *nread) { NTSTATUS status; - off_t ret; + off_t ret = 0; status = cli_pull(cli, fnum, offset, size, size, cli_read_sink, &buf, &ret); @@ -1248,7 +1248,7 @@ static void cli_smb1_writeall_written(struct tevent_req *subreq) struct cli_smb1_writeall_state *state = tevent_req_data( req, struct cli_smb1_writeall_state); NTSTATUS status; - size_t written, to_write; + size_t written = 0, to_write; status = cli_write_andx_recv(subreq, &written); TALLOC_FREE(subreq); -- Samba Shared Repository