From: Corinna Vinschen <[email protected]>
Fix broken code overriding primary group at process tree startup and
overriding groups from local SAM comment on domain member machines.
The first bug is fallout from adding newgrp(1) to the Cygwin
utils. It turned out that the Cygwin startup code didn't take
an already changed primary group in the user token into account,
which made newgrp(1) a no-op.
Unfortunately the fix from commit dc7b67316d01 ("Cygwin: uinfo: prefer
token primary group") made newgrp(1) work as desired, but broke the
scenario where a user's primary group was changed in the passwd entry
or in a SAM comment.
The second bug is actually pretty long-standing behaviour, but
apparently local SAM accounts are not in muchg use on domain member
machines...
Corinna Vinschen (4):
Cygwin: uinfo: correctly check and override primary group
Cygwin: uinfo: allow to override user account as primary group
Cygwin: uinfo: fix overriding group from SAM comment on AD member
machines
Cygwin: add release note for primary group override fix
winsup/cygwin/release/3.6.6 | 4 ++++
winsup/cygwin/uinfo.cc | 25 +++++++++++++++++++------
2 files changed, 23 insertions(+), 6 deletions(-)
--
2.52.0
