diff options
author | Ralph Boehme <slow@samba.org> | 2018-08-30 15:57:33 +0200 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2018-09-01 01:26:35 +0200 |
commit | 5d95f79f604d90c2646225a0f2470f05dd71e19e (patch) | |
tree | ba7244844e0d8f0d20008fd001f3ec26201fcfba /source3 | |
parent | 8f6edcc1645e0ed35eaec914bd0b672500ce986c (diff) | |
download | samba-5d95f79f604d90c2646225a0f2470f05dd71e19e.tar.gz |
s3:smbd: add a comment stating that file_close_user() is redundant for SMB2
Bug: https://bugzilla.samba.org/show_bug.cgi?id=13549
Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Sat Sep 1 01:26:35 CEST 2018 on sn-devel-144
Diffstat (limited to 'source3')
-rw-r--r-- | source3/smbd/smbXsrv_session.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/source3/smbd/smbXsrv_session.c b/source3/smbd/smbXsrv_session.c index 1193b4a3bea..19f4bc05600 100644 --- a/source3/smbd/smbXsrv_session.c +++ b/source3/smbd/smbXsrv_session.c @@ -1664,6 +1664,12 @@ NTSTATUS smbXsrv_session_logoff(struct smbXsrv_session *session) session->status = NT_STATUS_USER_SESSION_DELETED; if (session->compat) { + /* + * For SMB2 this is a bit redundant as files are also close + * below via smb2srv_tcon_disconnect_all() -> ... -> + * smbXsrv_tcon_disconnect() -> close_cnum() -> + * file_close_conn(). + */ file_close_user(sconn, session->compat->vuid); } |