From: Stefan Metzmacher <metze@samba.org>
Date: Thu, 14 Jun 2018 06:29:45 +0000 (+0200)
Subject: smbd: remove set_current_service() from defer_rename_done()
X-Git-Tag: tevent-0.9.37~305
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=03f69e82fc81f61d21def4465afa7f4a1b493c8b;p=thirdparty%2Fsamba.git

smbd: remove set_current_service() from defer_rename_done()

The change_to_user() above already called chdir_current_service().
And for smb2 we don't have per packet conn->case_sensitive anyway.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
---

diff --git a/source3/smbd/smb2_setinfo.c b/source3/smbd/smb2_setinfo.c
index 9cd4fdf9202..4958bb8f16e 100644
--- a/source3/smbd/smb2_setinfo.c
+++ b/source3/smbd/smb2_setinfo.c
@@ -302,12 +302,6 @@ static void defer_rename_done(struct tevent_req *subreq)
 		return;
 	}
 
-	ok = set_current_service(state->smb2req->tcon->compat, 0, true);
-	if (!ok) {
-		tevent_req_nterror(state->req, NT_STATUS_ACCESS_DENIED);
-		return;
-	}
-
 	/* Do we still need to wait ? */
 	lck = get_existing_share_mode_lock(state->req, state->fsp->file_id);
 	if (lck == NULL) {