smbcacls support for automatic inheritance propagation
David Disseldorp
ddiss at suse.de
Fri Jan 31 08:26:50 MST 2014
On Thu, 30 Jan 2014 16:25:40 +0000, Noel Power wrote:
> On 30/01/14 15:41, Noel Power wrote:
> > thanks (updated patches to follow) Noel
> I failed miserably with git send-mail, no doubt I have now a couple of
> patches in the moderation queue, please see
> http://cgit.freedesktop.org/~noelp/noelp-samba/log/?h=smbcacls_review%232 for
> the patches instead
This round looks good, there are a few more things I ran into during
testing/review:
prepare_inheritance_propagation() validates the command line ACL after
connecting to the share. I'd prefer it if this was done beforehand.
cacl_set() should use the same code path as inheritance_cacl_set(),
as it's basically the same as doing a inheritance_cacl_set() with
acl_no_propagate.
There are still some mixed declarations and code (lines 1620 and 1683).
Cheers, David
More information about the samba-technical
mailing list