Bug 139354

Summary: NFS ACL umask uses client umask AND default umask instead of the default umask
Product: Red Hat Enterprise Linux 3 Reporter: David Barker <d.r.barker>
Component: nfs-utilsAssignee: Jeff Layton <jlayton>
Status: CLOSED WONTFIX QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 3.0CC: staubach, steved
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-07-18 12:42:20 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description David Barker 2004-11-15 15:03:10 UTC
Description of problem:

NFS Mounts with acl's enabled on fs and exports. ACL's are applied
correctly, but the client umask is combined (logical AND) with the
default ACL umask instead of using the default umask as would happen
on fs without nfs.

e.g. 

[root@server webdata1]# mkdir a
[root@server webdata1]# setfacl -d -m u:user:rwx a
[root@server webdata1]# cd a
[root@server a]# mkdir b
[root@server a]# getfacl b
# file: b
# owner: root
# group: root
user::rwx
user:user:rwx               #effective:r-x
group::r-x
mask::r-x
other::r-x
default:user::rwx
default:user:user:rwx
default:group::r-x
default:mask::rwx
default:other::r-x 


Version-Release number of selected component (if applicable):


How reproducible:

always


Steps to Reproduce:

see above
  
Actual results:

logical AND of client umask and default umask applied

Expected results:

default umask applied

Additional info:

Comment 1 Jeff Layton 2007-07-18 12:42:20 UTC
Given that RHEL3 is now in maintenance mode and that this is not a catastrophic
bug, I don't believe we'll be able to patch this there.

A quick check of RHEL4 seems to show that NFS ACL's work identically to local
filesystems. I expect that the same will be the case on RHEL5 and beyond.
Closing this as WONTFIX.