Should 'map readonly' be overridden by 'store dos attributes' or NFSv4 ACLs?
Jeremy Allison
jra at samba.org
Mon May 6 10:51:16 MDT 2013
On Mon, May 06, 2013 at 09:32:43PM +1200, Andrew Bartlett wrote:
> The manpage for 'map readonly' confidently states (even with an
> underline) that:
>
>
> If store dos attributes is set to yes then this parameter is ignored
>
> However, it (as far as I read the code) has never been the case.
>
> This comes up because on NFSv4 systems, as discussed in the thread
> regarding the ACL patches, the owner of a file often has 0 permissions
> (due to there being no @owner acl), and so has the dos readonly flag
> set. This is a pain, as things on windows respect this flag.
>
> To work around this, many platforms with ACLs also set 'store dos
> attributes = true', which makes a lot of sense (as we don't want to be
> messing with permission bits, that would break ACLs badly). A trusting
> user might even dare to trust the we behave as the man-page indicates.
>
> I would like to know if we should change the code to match the manpage,
> or change the manpage to match the code (drop this sentence).
Yep, I noticed this too in my recent review of the dos attr
code bugs.
We should change the code to match the manpage. I'll submit
a fix for this.
Thanks,
Jeremy.
More information about the samba-technical
mailing list