setfacl Recursive Functionality

Shawn Webb lattera at gmail.com
Tue Mar 1 04:40:12 UTC 2011


One last patch before I do a send-pr to send the patch upstream. This patch
depends on my ZFS ACL Sets patch for the -k flag. I wanted to zero-out the
ACL entries recursively for a directory, which I can do in Solaris with
"chmod -R A= /path/to/directory", but ZFS doesn't allow zero-number ACL
sets. To work around that, I'm giving owner@ full permissions (the full_set
ACL set).

I'll also include my ZFS ACL Sets patch. That has already been submitted
upstream and I'm awaiting approval.

Thanks,

Shawn

On Tue, Feb 8, 2011 at 10:58 AM, Shawn Webb <lattera at gmail.com> wrote:

> I've just finished a patch to add recursive functionality to setfacl.
> Before I officially submit it, I'd like a few suggestions on how to improve
> the patch.
>
> The part I'm worried about involves the #define directive at top. I'm not
> sure what ramifications using that define might have. I needed it for my
> remove_invalid_inherit() function to work.
>
> Thanks,
>
> Shawn
>
> attached: setfacl.patch (universal diff)
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: setfacl_recursive.patch
Type: application/octet-stream
Size: 7219 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-current/attachments/20110301/bf1142b2/setfacl_recursive.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: zfs_acl_set.patch
Type: application/octet-stream
Size: 1436 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-current/attachments/20110301/bf1142b2/zfs_acl_set.obj


More information about the freebsd-current mailing list