diff options
author | Roland McGrath <roland@gnu.org> | 1996-07-23 21:40:11 +0000 |
---|---|---|
committer | Roland McGrath <roland@gnu.org> | 1996-07-23 21:40:11 +0000 |
commit | b24be05f19d1471526b4643d96c078d32ec1c692 (patch) | |
tree | 63907e014c30ac36f9ab6714dce8e945da305deb /time | |
parent | e3047adb46007d1c8a81c0f10c74f7d4f8fdbd22 (diff) |
Tue Jul 23 02:49:58 1996 Ulrich Drepper <drepper@cygnus.com>
* locale/iso-4217.def: Add India to comment for symbol "INR ".
Reported by Raja R Harinath <harinath@cs.umn.edu>.
Sat Jul 20 14:03:45 1996 Andreas Schwab <schwab@issan.informatik.uni-dortmund.de>
* sysdeps/m68k/dl-machine.h (elf_machine_rela): Remove bogus
#undef RESOLVE.
* time/tzset.c: Include <libc-lock.h>.
(__tzset_lock): New lock variable.
(tzset): New function (weak defn), no longer weak alias for __tzset.
Call __tzset while holding __tzset_lock.
* time/localtime.c: Include <libc-lock.h>.
(__localtime_r): Hold __tzset_lock while doing the work.
Diffstat (limited to 'time')
-rw-r--r-- | time/localtime.c | 49 | ||||
-rw-r--r-- | time/tzset.c | 16 |
2 files changed, 43 insertions, 22 deletions
diff --git a/time/localtime.c b/time/localtime.c index 4c68ab22dc..9dff89ce74 100644 --- a/time/localtime.c +++ b/time/localtime.c @@ -1,4 +1,5 @@ -/* Copyright (C) 1991, 1992, 1993, 1995 Free Software Foundation, Inc. +/* localtime -- convert `time_t' to `struct tm' in local time zone +Copyright (C) 1991, 92, 93, 95, 96 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -18,6 +19,7 @@ Cambridge, MA 02139, USA. */ #include <errno.h> #include <time.h> +#include <libc-lock.h> /* The C Standard says that localtime and gmtime return the same pointer. */ struct tm _tmbuf; @@ -35,7 +37,11 @@ __localtime_r (timer, tp) const time_t *timer; struct tm *tp; { - extern int __use_tzfile; + /* This lock is defined in tzset.c and locks all the data defined there + and in tzfile.c; the internal functions do no locking themselves. + This lock is only taken here and in `tzset'. */ + __libc_lock_define (extern, __tzset_lock); + extern int __tzset_run, __use_tzfile; extern int __tz_compute __P ((time_t timer, struct tm *tp)); extern int __tzfile_compute __P ((time_t timer, long int *leap_correct, int *leap_hit)); @@ -48,36 +54,37 @@ __localtime_r (timer, tp) return NULL; } - { - /* Make sure the database is initialized. */ - extern int __tzset_run; - if (! __tzset_run) - __tzset (); - } + __libc_lock_lock (__tzset_lock); + + /* Make sure the database is initialized. */ + if (! __tzset_run) + __tzset (); if (__use_tzfile) { if (! __tzfile_compute (*timer, &leap_correction, &leap_extra_secs)) - return NULL; + tp = NULL; } else { - struct tm *gmtp = __gmtime_r (timer, tp); - if (gmtp == NULL) - return NULL; - - if (! __tz_compute (*timer, gmtp)) - return NULL; - + tp = __gmtime_r (timer, tp); + if (tp && ! __tz_compute (*timer, tp)) + tp = NULL; leap_correction = 0L; leap_extra_secs = 0; } - __offtime (timer, __timezone - leap_correction, tp); - tp->tm_sec += leap_extra_secs; - tp->tm_isdst = __daylight; - tp->tm_gmtoff = __timezone; - tp->tm_zone = __tzname[__daylight]; + if (tp) + { + __offtime (timer, __timezone - leap_correction, tp); + tp->tm_sec += leap_extra_secs; + tp->tm_isdst = __daylight; + tp->tm_gmtoff = __timezone; + tp->tm_zone = __tzname[__daylight]; + } + + __libc_lock_unlock (__tzset_lock); + return tp; } weak_alias (__localtime_r, localtime_r) diff --git a/time/tzset.c b/time/tzset.c index d3a33dc1ce..a4fa8e358b 100644 --- a/time/tzset.c +++ b/time/tzset.c @@ -483,5 +483,19 @@ DEFUN(__tz_compute, (timer, tm), return 1; } + +#include <libc-lock.h> + +/* This locks all the state variables in tzfile.c and this file. */ +__libc_lock_define (, __tzset_lock) + +/* Reinterpret the TZ environment variable and set `tzname'. */ -weak_alias (__tzset, tzset) +weak_symbol (tzset) +void +tzset (void) +{ + __libc_lock_lock (__tzset_lock); + __tzset (); + __libc_lock_unlock (__tzset_lock); +} |