This is the mail archive of the
cygwin-apps@cygwin.com
mailing list for the Cygwin project.
Re: ntsec patch for setup
- From: "Max Bowsher" <maxb at ukf dot net>
- To: <cygwin-apps at cygwin dot com>,"Pierre A. Humblet" <Pierre dot Humblet at ieee dot org>
- Date: Sun, 19 Jan 2003 18:25:41 -0000
- Subject: Re: ntsec patch for setup
- References: <3.0.5.32.20030110223625.007fe100@mail.attbi.com>
Pierre A. Humblet wrote:
> This is the patch mentioned in the previous e-mail. I seems
> to work fine on NT and an early version was tested on 2000 (there
> are differences).
>
> Essentially the installed files should be in the Users or
> Admins groups instead of None.
> Nothing changes if the user running setup does not have the
> None gid.
Sounds like this should migitate most new user's ntsec problems.
It would probably be nice to add some logging to the success case, so its
easier to see that it has actually taken effect.
I don't understand this code:
+ isadmins = isadmins || EqualSid(groups->Groups[pg].Sid, asid);
+ isusers = isusers || EqualSid(groups->Groups[pg].Sid, usid);
If I'm reading it right, isadmins and isusers were initialized as false
several lines above, and not touched since - so why not a direct assignment?
Max.