diff options
author | Dan Robertson <drobertson@tripwire.com> | 2018-02-22 20:47:11 +0000 |
---|---|---|
committer | Stefan Metzmacher <metze@samba.org> | 2018-03-07 15:15:22 +0100 |
commit | d6753a1c87ab3c0320694e3ec6bf59e123162933 (patch) | |
tree | 26b16cef57285246549cbbc142eb7cbd2cb3a6ef /source3 | |
parent | e176cabddd32168b0ee5f3493ff78eaadf6940e8 (diff) | |
download | samba-d6753a1c87ab3c0320694e3ec6bf59e123162933.tar.gz |
libsmb: Use smb2 tcon if conn_protocol >= SMB2_02
When the connection protocol is SMB2 the tid from the smb1 member is
used instead of smb2 in cli_state_set_tid which often results in a null
deref.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13310
Signed-off-by: Dan Robertson <drobertson@tripwire.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
(cherry picked from commit b67ffaf518c971817b167b41bf6226cddfdcfd2f)
Diffstat (limited to 'source3')
-rw-r--r-- | source3/libsmb/clientgen.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/libsmb/clientgen.c b/source3/libsmb/clientgen.c index 44afee1d4a0..2e4dd15ab62 100644 --- a/source3/libsmb/clientgen.c +++ b/source3/libsmb/clientgen.c @@ -371,7 +371,7 @@ uint32_t cli_state_set_tid(struct cli_state *cli, uint32_t tid) uint32_t ret; if (smbXcli_conn_protocol(cli->conn) >= PROTOCOL_SMB2_02) { ret = smb2cli_tcon_current_id(cli->smb2.tcon); - smb2cli_tcon_set_id(cli->smb1.tcon, tid); + smb2cli_tcon_set_id(cli->smb2.tcon, tid); } else { ret = smb1cli_tcon_current_id(cli->smb1.tcon); smb1cli_tcon_set_id(cli->smb1.tcon, tid); |