diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2008-06-13 06:08:54 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2008-06-13 06:08:54 +0000 |
| commit | 5bcc6c0f96789e52caa4ffdded7856472bb895e2 (patch) | |
| tree | 0dd661d0982012366bfd3335249c178af8c77a5c /time | |
| parent | f854efd7224944b15ba3068de28c0d00550f88b7 (diff) | |
| download | glibc-5bcc6c0f96789e52caa4ffdded7856472bb895e2.tar.gz glibc-5bcc6c0f96789e52caa4ffdded7856472bb895e2.tar.xz glibc-5bcc6c0f96789e52caa4ffdded7856472bb895e2.zip | |
[BZ #6612] cvs/fedora-glibc-20080613T1601
* time/strftime.c: Pass reference to tzset_called around to handle recursive calls. [BZ #6612] * time/strftime.c (__strftime_internal): Call tzset() only when printing timezone-dependent values. Based on a patch by Petr Baudis <pasky@suse.cz>.
Diffstat (limited to 'time')
| -rw-r--r-- | time/strftime_l.c | 43 |
1 files changed, 28 insertions, 15 deletions
diff --git a/time/strftime_l.c b/time/strftime_l.c index 4a57b1599d..f1d33034d9 100644 --- a/time/strftime_l.c +++ b/time/strftime_l.c @@ -1,4 +1,4 @@ -/* Copyright (C) 2002, 2004, 2007 Free Software Foundation, Inc. +/* Copyright (C) 2002, 2004, 2007, 2008 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 @@ -455,7 +455,8 @@ static CHAR_T const month_name[][10] = #endif static size_t __strftime_internal (CHAR_T *, size_t, const CHAR_T *, - const struct tm *, bool ut_argument_spec_iso + const struct tm *, bool * + ut_argument_spec_iso LOCALE_PARAM_PROTO) __THROW; /* Write information from TP into S according to the format @@ -481,7 +482,8 @@ my_strftime (s, maxsize, format, tp ut_argument LOCALE_PARAM) tmcopy = *tp; tp = &tmcopy; #endif - return __strftime_internal (s, maxsize, format, tp, false + bool tzset_called = false; + return __strftime_internal (s, maxsize, format, tp, &tzset_called ut_argument LOCALE_ARG); } #ifdef _LIBC @@ -495,7 +497,7 @@ __strftime_internal (s, maxsize, format, tp, tzset_called ut_argument size_t maxsize; const CHAR_T *format; const struct tm *tp; - bool tzset_called; + bool *tzset_called; ut_argument_spec LOCALE_PARAM_DECL { @@ -563,16 +565,6 @@ __strftime_internal (s, maxsize, format, tp, tzset_called ut_argument if (! (zone && *zone)) zone = "GMT"; } - else - { - /* POSIX.1 requires that local time zone information is used as - though strftime called tzset. */ -# if HAVE_TZSET - if (!tzset_called) - tzset (); - tzset_called = true; -# endif - } #endif if (hour12 > 12) @@ -1325,7 +1317,18 @@ __strftime_internal (s, maxsize, format, tp, tzset_called ut_argument #if HAVE_TZNAME /* The tzset() call might have changed the value. */ if (!(zone && *zone) && tp->tm_isdst >= 0) - zone = tzname[tp->tm_isdst]; + { + /* POSIX.1 requires that local time zone information is used as + though strftime called tzset. */ +# if HAVE_TZSET + if (!*tzset_called) + { + tzset (); + *tzset_called = true; + } +# endif + zone = tzname[tp->tm_isdst]; + } #endif if (! zone) zone = ""; @@ -1361,6 +1364,16 @@ __strftime_internal (s, maxsize, format, tp, tzset_called ut_argument struct tm ltm; time_t lt; + /* POSIX.1 requires that local time zone information is used as + though strftime called tzset. */ +# if HAVE_TZSET + if (!*tzset_called) + { + tzset (); + *tzset_called = true; + } +# endif + ltm = *tp; lt = mktime (<m); |