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: The Everyone group


On Thu, Aug 01, 2002 at 09:56:54PM -0400, Pierre A. Humblet wrote:
> 3 patches are included: those to syscalls.cc and security.cc
> are minor optimizations related to setgroups. The one to
> grp.cc removes Everyone from the output of getgroups32
> (as if it weren't in /etc/group).

Applied.

> Additionally I would suggest modifying mkpasswd and mkgroup to
> - remove Everyone from the output
> - set the gid of SYSTEM to 544 (instead of 18, which is redundant anyway).
>   Without this, doing setgroups(0, NULL) while running as SYSTEM removes
>   the Administrators group, which is not at all what happens in Unix when
>   running as root.
> If you think it's a good idea I'll get to that in a couple of weeks,
> after some vacations.

It sounds like a good plan.  I'd just like to be careful if nothing
weird happens when removing it from passwd and group.  However, I've
changed mkpasswd and mkgroup already and I've changed my passwd and group
files to test that.

> Note that if Everyone is removed from /etc/group applying the patch to 
> grp.cc makes no difference.

It makes a difference for existing setups so it's better to have it.

Thanks,
Corinna

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


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