[SCM] Samba Shared Repository - branch master updated

David Disseldorp ddiss at samba.org
Mon Apr 9 22:46:02 UTC 2018


The branch, master has been updated
       via  3227b11 s3: smbd: Unix extensions attempts to change wrong field in fchown call.
      from  461a117 s3: smbd: Fix memory leak in vfswrap_getwd()

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 3227b110d065500ed84fc70063da70ab35823a2e
Author: Jeremy Allison <jra at samba.org>
Date:   Mon Apr 9 09:32:23 2018 -0700

    s3: smbd: Unix extensions attempts to change wrong field in fchown call.
    
    Cut and paste error.
    
    BUG: https://bugzilla.samba.org/show_bug.cgi?id=13375
    
    Reported-by: Rungta, Vandana <vrungta at amazon.com>
    Signed-off-by: Jeremy Allison <jra at samba.org>
    Reviewed-by: David Disseldorp <ddiss at samba.org>
    
    Autobuild-User(master): David Disseldorp <ddiss at samba.org>
    Autobuild-Date(master): Tue Apr 10 00:45:56 CEST 2018 on sn-devel-144

-----------------------------------------------------------------------

Summary of changes:
 source3/smbd/trans2.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c
index b49c3ca..af7a0d7 100644
--- a/source3/smbd/trans2.c
+++ b/source3/smbd/trans2.c
@@ -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


-- 
Samba Shared Repository



More information about the samba-cvs mailing list