diff options
| author | Corinna Vinschen <vinschen@redhat.com> | 2007-12-11 16:14:33 +0000 |
|---|---|---|
| committer | Corinna Vinschen <vinschen@redhat.com> | 2007-12-11 16:14:33 +0000 |
| commit | c4472fbcc77eaea58548e0e7bfacd22d73630438 (patch) | |
| tree | 75c8377da416fc767ed691b1e403fcc29c3fbab5 | |
| parent | f8154f6cb6c20bafe85aeea8d46a3268a6049934 (diff) | |
| download | gdb-c4472fbcc77eaea58548e0e7bfacd22d73630438.tar.gz | |
* localtime.cc (tzset): Guard by a muto for thread safety.
| -rw-r--r-- | winsup/cygwin/ChangeLog | 4 | ||||
| -rw-r--r-- | winsup/cygwin/localtime.cc | 12 |
2 files changed, 13 insertions, 3 deletions
diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 4cf3f783edb..ca28c9b66e1 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,7 @@ +2007-12-11 Corinna Vinschen <corinna@vinschen.de> + + * localtime.cc (tzset): Guard by a muto for thread safety. + 2007-12-11 Dave Korn <dave.korn@artimi.com> * cygheap.cc (_csbrk): Don't request zero bytes from VirtualAlloc, diff --git a/winsup/cygwin/localtime.cc b/winsup/cygwin/localtime.cc index a2f415f0818..3d08cc67b8b 100644 --- a/winsup/cygwin/localtime.cc +++ b/winsup/cygwin/localtime.cc @@ -6,6 +6,7 @@ #include "winsup.h" #include "cygerrno.h" +#include "sync.h" #include <windows.h> #define STD_INSPIRED #define lint @@ -1470,19 +1471,22 @@ tzsetwall P((void)) settzname(); } +static NO_COPY muto tzset_guard; + extern "C" void tzset P((void)) { + tzset_guard.init ("tzset_guard")->acquire (); const char * name = getenv("TZ"); if (name == NULL) { if (!lcl_is_set) tzsetwall(); - return; + goto out; } if (lcl_is_set > 0 && strcmp(lcl_TZname, name) == 0) - return; + goto out; lcl_is_set = (strlen(name) < sizeof (lcl_TZname)); if (lcl_is_set) strcpy(lcl_TZname, name); @@ -1492,7 +1496,7 @@ tzset P((void)) lclptr = (struct state *) malloc(sizeof *lclptr); if (lclptr == NULL) { settzname(); /* all we can do */ - return; + goto out; } } #endif /* defined ALL_STATE */ @@ -1510,6 +1514,8 @@ tzset P((void)) gmtload(lclptr); } settzname(); +out: + tzset_guard.release (); } /* |