[PATCH] vfs_glusterfs: Make sure posix acls are formatted "before setting"/"after getting"
Jeremy Allison
jra at samba.org
Mon Aug 12 09:58:40 MDT 2013
On Sun, Aug 11, 2013 at 03:12:34PM -0400, Raghavendra Talur wrote:
> Simo, Andrew, Jeremy and Chris,
>
> Thank you all for the feedback. I apologize for not replying
> earlier, I was on a vacation.
>
> 1. I agree with the formatting changes.
> 2. When I first read the plugin code I had the same questions as
> Jeremy has, but, when I looked at the libacl code:
>
> a. http://acl.sourcearchive.com/documentation/2.2.49-1/____acl__from__xattr_8c-source.html
> b. http://acl.sourcearchive.com/documentation/2.2.49-1/acl__ea_8h-source.html
>
> I was convinced that this was a acceptable way of handling acls. As the plugin interacts
> only with Samba on one side and libgfapi of Gluster on the other, both residing on the
> same machine, I could not think of any case where plugin would need to handle anything
> extra than what libacl does.
>
> Not to undermine any of the reviews, I am asking this just to make it clear to any new
> person who would ask the same question of "why din't we do what libacl does?"
This is libacl code, not Samba code. libacl is a Linux
specific library, and can do whatever it likes in terms
of portability.
I'd prefer higher standard for Samba code please.
Jeremy.
More information about the samba-technical
mailing list