I forgot...
On Dec 18 12:23, Corinna Vinschen wrote:
> 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(-)
v2: Try to improve commit messages, add patch fixing primary group
override for local accounts on domain member machines to the
patchset.
Sorry,
Corinna
