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

s3: VFS: fake_acls: In fake_acls_lstat() - 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 28d6acef163..5f2fe85f69c 100644
--- a/source3/modules/vfs_fake_acls.c
+++ b/source3/modules/vfs_fake_acls.c
@@ -158,24 +158,9 @@ static int fake_acls_lstat(vfs_handle_struct *handle,
 
 	ret = SMB_VFS_NEXT_LSTAT(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;
-		}
-
 		/* This isn't quite right (calling getxattr not
 		 * lgetxattr), but for the test purposes of this
 		 * module (fake NT ACLs from windows clients), it is
@@ -187,7 +172,6 @@ static int fake_acls_lstat(vfs_handle_struct *handle,
 			&smb_fname->st.st_ex_uid);
 		fake_acls_gid(handle, &smb_fname_base,
 			&smb_fname->st.st_ex_gid);
-		TALLOC_FREE(frame);
 	}
 
 	return ret;