[PATCH] Second part of bugfix for bug #13319 (round-tripping ACE's through vfs_fruit).

Ralph Böhme slow at samba.org
Fri Mar 16 18:55:04 UTC 2018

On Fri, Mar 16, 2018 at 11:47:09AM -0700, Jeremy Allison wrote:
> On Fri, Mar 16, 2018 at 07:34:54PM +0100, Ralph Böhme wrote:
> > Hi Jeremy,
> > 
> > On Thu, Mar 15, 2018 at 03:28:03PM -0700, Jeremy Allison wrote:
> > > On Thu, Mar 15, 2018 at 06:15:04PM -0400, jim via samba-technical wrote:
> > > > Should vfs_fruit.c debug message use DEBUG_WARNING instead of DEBUG(1, per
> > > > replaced code?
> > > 
> > > Yes, that's correct, I missed that. Updated patch attached.
> > 
> > thanks! Lgtm. Sorry for not catching this in the initial review.
> Is that lgtm a RB+ ? Just checking :-).

Not sure. :)

> > Once question: why do you add a new share and test-suite for the test? The
> > option is enabled by default, so we shouldn't need this unless you wanted to
> > make this explicit in case the default changes.
> The default share doesn't have xattr_tdb on the end of
> the vfs modules. It it gets run on a filesystem that
> doesn't support native xattrs the test fails (took
> me a long time to figure out why running it manually
> worked - ext4, vs running make test failed - tmpfs :-).

Hm, I guess that is just a mistake on my side and we should add xattr_tdb to all
fruit shares. I can prepare a patch if you like.

> I didn't want to add that to the "standard" vfs_fruit
> share just in case this was intentional or caused
> other tests to fail.

Would be it possible to just use the existing shares given they use xattr_tdb?
I'd like to avoid share and test-suite bloat...


Ralph Boehme, Samba Team       https://samba.org/
Samba Developer, SerNet GmbH   https://sernet.de/en/samba/
GPG Key Fingerprint:           FAE2 C608 8A24 2520 51C5
                               59E4 AA1E 9B71 2639 9E46

More information about the samba-technical mailing list