* passwd.cc (read_etc_passwd): Don't bother with locking when

in cygwin initialization since there is only one thread.
* grp.cc (read_etc_group): Ditto.
This commit is contained in:
Christopher Faylor 2001-09-28 07:01:22 +00:00
parent 9c61aed684
commit 8b51edbfa6
3 changed files with 33 additions and 14 deletions

View File

@ -1,3 +1,9 @@
Fri Sep 28 02:57:03 2001 Christopher Faylor <cgf@cygnus.com>
* passwd.cc (read_etc_passwd): Don't bother with locking when
in cygwin initialization since there is only one thread.
* grp.cc (read_etc_group): Ditto.
Fri Sep 28 01:50:09 2001 Christopher Faylor <cgf@cygnus.com> Fri Sep 28 01:50:09 2001 Christopher Faylor <cgf@cygnus.com>
* pipe.cc (fhandler_pipe::hit_eof): Return correct value when there is * pipe.cc (fhandler_pipe::hit_eof): Return correct value when there is

View File

@ -117,6 +117,14 @@ add_grp_line (const char *line)
curr_lines++; curr_lines++;
} }
class group_lock
{
pthread_mutex_t mutex;
public:
group_lock (): mutex ((pthread_mutex_t) PTHREAD_MUTEX_INITIALIZER) {}
void arm () {pthread_mutex_lock (&mutex); }
};
/* Cygwin internal */ /* Cygwin internal */
/* Read in /etc/group and save contents in the group cache */ /* Read in /etc/group and save contents in the group cache */
/* This sets group_in_memory_p to 1 so functions in this file can /* This sets group_in_memory_p to 1 so functions in this file can
@ -132,15 +140,13 @@ read_etc_group ()
strncpy (group_name, "Administrators", sizeof (group_name)); strncpy (group_name, "Administrators", sizeof (group_name));
static NO_COPY pthread_mutex_t etc_group_mutex = (pthread_mutex_t) PTHREAD_MUTEX_INITIALIZER; static NO_COPY group_lock here;
pthread_mutex_lock (&etc_group_mutex); if (cygwin_finished_initializing)
here.arm ();
/* if we got blocked by the mutex, then etc_group may have been processed */ /* if we got blocked by the mutex, then etc_group may have been processed */
if (group_state != uninitialized) if (group_state != uninitialized)
{ return;
pthread_mutex_unlock(&etc_group_mutex);
return;
}
if (group_state != initializing) if (group_state != initializing)
{ {
@ -193,7 +199,7 @@ read_etc_group ()
} }
} }
pthread_mutex_unlock(&etc_group_mutex); return;
} }
extern "C" extern "C"

View File

@ -109,6 +109,14 @@ add_pwd_line (char *line)
parse_pwd (passwd_buf[curr_lines++], line); parse_pwd (passwd_buf[curr_lines++], line);
} }
class passwd_lock
{
pthread_mutex_t mutex;
public:
passwd_lock (): mutex ((pthread_mutex_t) PTHREAD_MUTEX_INITIALIZER) {}
void arm () {pthread_mutex_lock (&mutex); }
};
/* Read in /etc/passwd and save contents in the password cache. /* Read in /etc/passwd and save contents in the password cache.
This sets passwd_state to loaded or emulated so functions in this file can This sets passwd_state to loaded or emulated so functions in this file can
tell that /etc/passwd has been read in or will be emulated. */ tell that /etc/passwd has been read in or will be emulated. */
@ -120,15 +128,14 @@ read_etc_passwd ()
* for non-shared mutexs in the future. Also, this function will at most be called * for non-shared mutexs in the future. Also, this function will at most be called
* once from each thread, after that the passwd_state test will succeed * once from each thread, after that the passwd_state test will succeed
*/ */
static NO_COPY pthread_mutex_t etc_passwd_mutex = (pthread_mutex_t) PTHREAD_MUTEX_INITIALIZER; static NO_COPY passwd_lock here;
pthread_mutex_lock (&etc_passwd_mutex);
if (cygwin_finished_initializing)
here.arm ();
/* if we got blocked by the mutex, then etc_passwd may have been processed */ /* if we got blocked by the mutex, then etc_passwd may have been processed */
if (passwd_state != uninitialized) if (passwd_state != uninitialized)
{ return;
pthread_mutex_unlock(&etc_passwd_mutex);
return;
}
if (passwd_state != initializing) if (passwd_state != initializing)
{ {
@ -165,7 +172,7 @@ read_etc_passwd ()
} }
pthread_mutex_unlock (&etc_passwd_mutex); return;
} }
/* Cygwin internal */ /* Cygwin internal */