the order of ACEs.

Corinna Vinschen cygwin-patches@cygwin.com
Thu Apr 26 02:22:00 GMT 2001


On Thu, Apr 26, 2001 at 09:14:34AM +0900, Kazuhiro Fujieda wrote:
> >>> On Wed, 25 Apr 2001 23:09:39 +0200
> >>> Corinna Vinschen <cygwin-patches@cygwin.com> said:
> 
> > > @@ -661,13 +665,7 @@ alloc_sd (uid_t uid, gid_t gid, const ch
> > >  	      return NULL;
> > >  	    }
> > >  	  acl_len += ace->Header.AceSize;
> > > -	  ++ace_off;
> > 
> > ...why does your patch drop the `++ace_off'? This seems to be
> > a mistake.
> 
> I understood the `++ace_off' was only for appending `everyone'
> ACE afterward. So I inferred it wasn't necessary anymore.

Oops, you're right. I should read my own code again...

Thanks for the patch, it's applied.

Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.



More information about the Cygwin-patches mailing list