On Feb 12 13:33, Pierre A. Humblet wrote: > From: "Corinna Vinschen" > | On Feb 12 18:39, Corinna Vinschen wrote: > | > Cool! Look at the date! So I did it again wrong after you already > | > fixed it in 2002. Oh well. Btw, I tested this today and it seems the > | > patch is working. I'll just change it to use a well_known_builtin_sid > | > rather than creating the SID on the fly. > | > | Nevertheless, would you mind to test it as well? Easy chance that I > | missed something. > > Matthias just tested what I did last night, which was based on what > you had sent, but also defining well_known_users_sid in security.h etc..
Sorry, but I don't understand what you mean. well_known_users_sid *is* already defined in security.h since my patch from 2010-01-08. > Now cron works on his system. Looks like we are duplicating efforts! > > I just refreshed cvs and get_token_group_sidlist still has: > grp_list *= well_known_users_sid; Yes, sure. What's the problem? You wrote in http://cygwin.com/ml/cygwin/2010-02/msg00291.html that always adding well_known_users_sid is a good idea from your POV. I currently don't understand what you're trying to say. > I was also looking at the lsaauth code and noticed you exclude all > well-known groups. > Does lsa put them in the token anyway? Otherwise how does > USERS get in there? Yes. > I have many other observations but that will be for later. Ok. Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Project Co-Leader cygwin AT cygwin DOT com Red Hat -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
