From: Jeremy Allison <jra@samba.org>
Date: Wed, 23 Jun 2021 02:05:06 +0000 (-0700)
Subject: s3: VFS: fake_acls: In fake_acls_stat() - remove call to get_full_smb_filename().
X-Git-Tag: tevent-0.11.0~187
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6e6f532c5fc7daad019d717302abe72f6fc97de0;p=thirdparty%2Fsamba.git

s3: VFS: fake_acls: In fake_acls_stat() - remove call to get_full_smb_filename().

It serves no purpose here. We no longer need the frame pointer.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
---

diff --git a/source3/modules/vfs_fake_acls.c b/source3/modules/vfs_fake_acls.c
index 818c2f0f896..4d1211a0649 100644
--- a/source3/modules/vfs_fake_acls.c
+++ b/source3/modules/vfs_fake_acls.c
@@ -115,37 +115,20 @@ static int fake_acls_stat(vfs_handle_struct *handle,
 
 	ret = SMB_VFS_NEXT_STAT(handle, smb_fname);
 	if (ret == 0) {
-		TALLOC_CTX *frame = talloc_stackframe();
-		char *path;
 		struct smb_filename smb_fname_base = {
 			.base_name = smb_fname->base_name
 		};
-		NTSTATUS status;
-		/*
-		 * As we're calling getxattr directly here
-		 * we need to use only the base_name, not
-		 * the full name containing any stream name.
-		 */
-		status = get_full_smb_filename(frame, &smb_fname_base, &path);
-		if (!NT_STATUS_IS_OK(status)) {
-			errno = map_errno_from_nt_status(status);
-			TALLOC_FREE(frame);
-			return -1;
-		}
-		
+
 		ret = fake_acls_uid(handle, &smb_fname_base,
 					&smb_fname->st.st_ex_uid);
 		if (ret != 0) {
-			TALLOC_FREE(frame);
 			return ret;
 		}
 		ret = fake_acls_gid(handle, &smb_fname_base,
 					&smb_fname->st.st_ex_gid);
 		if (ret != 0) {
-			TALLOC_FREE(frame);
 			return ret;
 		}
-		TALLOC_FREE(frame);
 	}
 
 	return ret;