s3: modules: vfs_ceph: Remove FCHMOD_ACL in cephwrap_fchmod().
authorJeremy Allison <jra@samba.org>
Thu, 17 May 2018 18:46:21 +0000 (11:46 -0700)
committerJeremy Allison <jra@samba.org>
Fri, 25 May 2018 16:39:25 +0000 (18:39 +0200)
Now I understand the use of the mask in POSIX ACLs
this extra step is no longer needed.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
source3/modules/vfs_ceph.c

index 818d0cb54908afd817193da502e25b72c6e0486c..b6ca27dd47ef7435c54bf97035022e098da463f4 100644 (file)
@@ -957,21 +957,6 @@ static int cephwrap_fchmod(struct vfs_handle_struct *handle, files_struct *fsp,
 
        DBG_DEBUG("[CEPH] fchmod(%p, %p, %d)\n", handle, fsp, mode);
 
-       /*
-        * We need to do this due to the fact that the default POSIX ACL
-        * chmod modifies the ACL *mask* for the group owner, not the
-        * group owner bits directly. JRA.
-        */
-
-       {
-               int saved_errno = errno; /* We might get ENOSYS */
-               if ((result = SMB_VFS_FCHMOD_ACL(fsp, mode)) == 0) {
-                       return result;
-               }
-               /* Error - return the old errno. */
-               errno = saved_errno;
-       }
-
 #if defined(HAVE_FCHMOD)
        result = ceph_fchmod(handle->data, fsp->fh->fd, mode);
        DBG_DEBUG("[CEPH] fchmod(...) = %d\n", result);