s3: smbd: Unix extensions attempts to change wrong field in fchown call.
authorJeremy Allison <jra@samba.org>
Mon, 9 Apr 2018 16:32:23 +0000 (09:32 -0700)
committerDavid Disseldorp <ddiss@samba.org>
Mon, 9 Apr 2018 22:45:56 +0000 (00:45 +0200)
Cut and paste error.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=13375

Reported-by: Rungta, Vandana <vrungta@amazon.com>
Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Autobuild-User(master): David Disseldorp <ddiss@samba.org>
Autobuild-Date(master): Tue Apr 10 00:45:56 CEST 2018 on sn-devel-144

source3/smbd/trans2.c

index b49c3ca7836fa4821bd99cb782842d2aca8eb260..af7a0d7248420929cd1272816ebc65f42ae7e275 100644 (file)
@@ -7778,10 +7778,10 @@ static NTSTATUS smb_set_file_unix_basic(connection_struct *conn,
 
                DEBUG(10,("smb_set_file_unix_basic: SMB_SET_FILE_UNIX_BASIC "
                          "changing group %u for file %s\n",
-                         (unsigned int)set_owner,
+                         (unsigned int)set_grp,
                          smb_fname_str_dbg(smb_fname)));
                if (fsp && fsp->fh->fd != -1) {
-                       ret = SMB_VFS_FCHOWN(fsp, set_owner, (gid_t)-1);
+                       ret = SMB_VFS_FCHOWN(fsp, (uid_t)-1, set_grp);
                } else {
                        /*
                         * UNIX extensions calls must always operate