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]

Re: [PATCH] Re: pthread -- Corinna?


On Mon, Apr 16, 2001 at 10:54:23PM -0400, Christopher Faylor wrote:
>It still seems like there is potential for error if two threads call
>get_id_from_sid.  One may correctly read a UID via getpwuid, one will
>short-circuit.  Hmm.  This would be a problem even if we were attempting
>to detect recursion via a static variable.
>
>Maybe we need another passwd_state == 'initializing'.
>
>Hmm, again.  We actually have *three* variables controlling how this
>operates now, passwd_state, passwd_sem, and etc_passwd_mutex.  IMO,
>that's too many.

On reflection, I don't see how we can ever get into the state where
fopen is called recursively from read_etc_passwd.  Your mutex protects
against that.  So, I am still of the opinion that we can remove the
passwd_sem.

And, we should do something similar for group_sem, too.

cgf


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