[PATCH] Fix open bug found at Microsoft interop event - version 2

Ira Cooper ira at samba.org
Fri Jun 21 12:28:04 MDT 2013


Reviewed by: <ira at samba.org>

If others wish to review, I wouldn't discourage that, but I think the
change is sound.

-Ira


On Fri, Jun 21, 2013 at 11:20 AM, Jeremy Allison <jra at samba.org> wrote:

> On Fri, Jun 21, 2013 at 10:57:57AM -0700, Jeremy Allison wrote:
> > On Fri, Jun 21, 2013 at 09:46:20AM -0700, Jeremy Allison wrote:
> > >
> > > I'll resubmit the modified version shortly.
> >
> > Here is the modified version. It's the same code logic,
> > it just re-uses the existing "acl map full control" parameter
> > that is already set to "true" and used in the posix ACL
> > map code, rather than adding a new parameter.
> >
> > As this is now on by default, that means it's tested
> > in the default selftests in the raw.acl tests on top
> > of vfs_zfsacl, so the extra test suite changes aren't
> > needed.
> >
> > I still added the module docs for vfs_zfsacl and made
> > a note of the behaviour change in the vfs_gpfs module
> > documentation. This makes it a much shorter patchset.
>
> Version with typos noticed by Ira (and man page
> incorrect file name) fixed.
>
> Think this one should do the trick :-).
>
> Cheers,
>
>         Jeremy.
>


More information about the samba-technical mailing list