This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: initgroups


Corinna Vinschen wrote:

> > If we decide on 1) shouldn't we remove calls to {ug}id16to(ug}id32 from
> > passwd.cc, grp.cc and syscalls.cc, EXCEPT in the various cases of chown
> > (i.e. simply do as getgrgid (), which doesn't call gid16togid32)?
> > Also, we shouldn't rely on ILLEGAL_UID in dcrt0.
> > If we decide on 2), shouldn't we enforce it everywhere? One possibility is
> > not to read in passwd and group entries with "illegal" {ug}id values.
> 
> After looking into this I think 2) is the way to go.  We can't support
> that uid/gid for apparent reasons so we should take the approach to
> invalidate it everywhere, yes.
> 
> However, that would mean that we have to treat both values as
> illegal, ILLEGAL_[UG]ID and ILLEGAL_[UG]ID16.  This looks a little
> bit weird to me...

Good point. But that's due to the transition from 16 to 32 bits (when do
you plan to complete it?) and it depends on the user's perspective.

If the user thinks that cygwin allows only 16 bit {ug}id's in passwd & group, 
then only 0xFFFF is illegal and nothing is weird.

If the user thinks cygwin allows 32 bit entries in passwd & group, then all 
entries ending with 0xFFFF are illegal, which is weird, but there
are much weirder things in that situation.

For the moment if a group has a gid > 0xFFFF,
then getgrnam() will find it but will return a truncated gid.
However getgrgid(gid) and even getgrgid((getgrnam())->gr_gid) never work 
properly. They either fail or return an aliased group. 
If the truncated gid was stored in the Cygwin internal passwd structures, 
then getgrgid(gid) would always return a group, but possibly an alias...

Until 32 bit {ug}id's are completely supported, I would consider as invalid 
everything ending with 0xFFFF.

Pierre


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]