s3: smbd: change_file_owner_to_parent_fsp(). Don't re-stat the pathref.

Optimization now becomes clear. We already have a valid stat of the parent
directory so we don't need to re-do a system call.

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

diff --git a/source3/smbd/open.c b/source3/smbd/open.c
index 22bd6050eb8963940ea9a1b319e9337f4b98f09c..3d7fea752528448eb6426eb6d843cc1eddbf3266 100644 (file)
--- a/source3/smbd/open.c
+++ b/source3/smbd/open.c
@@ -944,15 +944,6 @@ static void change_file_owner_to_parent_fsp(struct files_struct *parent_fsp,
 {
        int ret;
 
-       ret = SMB_VFS_FSTAT(parent_fsp, &parent_fsp->fsp_name->st);
-       if (ret == -1) {
-               DBG_ERR("failed to stat parent "
-                       "directory %s. Error was %s\n",
-                       fsp_str_dbg(parent_fsp),
-                       strerror(errno));
-                return;
-       }
-
        if (parent_fsp->fsp_name->st.st_ex_uid == fsp->fsp_name->st.st_ex_uid) {
                /* Already this uid - no need to change. */
                DBG_DEBUG("file %s is already owned by uid %u\n",