]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s3: VFS: fake_acls: In fake_acls_lstat() - remove call to get_full_smb_filename().
authorJeremy Allison <jra@samba.org>
Wed, 23 Jun 2021 02:05:57 +0000 (19:05 -0700)
committerRalph Boehme <slow@samba.org>
Fri, 25 Jun 2021 15:53:31 +0000 (15:53 +0000)
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>
source3/modules/vfs_fake_acls.c

index 28d6acef163942eaeea5d95557890029d106c9b5..5f2fe85f69ce0664256131c222e81b96190be1cd 100644 (file)
@@ -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;