summaryrefslogtreecommitdiff
path: root/source3
diff options
context:
space:
mode:
authorRalph Boehme <slow@samba.org>2018-08-30 15:57:33 +0200
committerJeremy Allison <jra@samba.org>2018-09-01 01:26:35 +0200
commit5d95f79f604d90c2646225a0f2470f05dd71e19e (patch)
treeba7244844e0d8f0d20008fd001f3ec26201fcfba /source3
parent8f6edcc1645e0ed35eaec914bd0b672500ce986c (diff)
downloadsamba-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.c6
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);
}