[PATCH] Follow-up patch for bug in dealing with "Owner Rights" ACEs when calculating maximum access

[Jeremy Allison]

On Fri, Mar 01, 2019 at 12:38:55AM +0100, David Disseldorp wrote:
> On Thu, 28 Feb 2019 15:12:56 -0800, Jeremy Allison via samba-technical wrote:
> 
> > +				uint32_t already_granted = (granted |
> > +							owner_rights_allowed);
> > +
> > +				owner_rights_denied |= (ace->access_mask &
> > +							~already_granted);
> 
> Wouldn't this now mean that an owner_rights_allowed ACE now takes
> precedence over an owner_rights_denied ACE if the former comes first?
> I'll need to take a closer look at the spec tomorrow for this.

Yes, that's exactly the case. That's what the test shows.

Check out test_owner_rights_deny1(), it tests this case.