public inbox for glibc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libc/10696] New: tzset_internal doesn't handle changes of timezone file
@ 2009-09-25 17:17 cfriesen at nortel dot com
  2009-09-25 17:18 ` [Bug libc/10696] " cfriesen at nortel dot com
  2009-09-26 19:00 ` drepper at redhat dot com
  0 siblings, 2 replies; 3+ messages in thread
From: cfriesen at nortel dot com @ 2009-09-25 17:17 UTC (permalink / raw)
  To: glibc-bugs

If the TZ environment variable is changed while a program is running, the next
call to tzset_internal will result in picking up the modified timzeone.

However, if the contents of the timezone file are updated (due to a package
upgrade, for instance) or if it's a symlink and it is changed to point to a
different file while a program is running, the tzset_internal function will not
pick up the modifications because the filename is still the same.

I propose modifying this function to also stat() and lstat() the timezone file
and compare the mtime values against values stored along with the filename.  If
either of the timestamps doesn't match the stored value, the timezone file is
re-read.

-- 
           Summary: tzset_internal doesn't handle changes of timezone file
           Product: glibc
           Version: 2.10
            Status: NEW
          Severity: normal
          Priority: P2
         Component: libc
        AssignedTo: drepper at redhat dot com
        ReportedBy: cfriesen at nortel dot com
                CC: glibc-bugs at sources dot redhat dot com
 GCC build triplet: n/a
  GCC host triplet: n/a
GCC target triplet: n/a


http://sourceware.org/bugzilla/show_bug.cgi?id=10696

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug libc/10696] tzset_internal doesn't handle changes of timezone file
  2009-09-25 17:17 [Bug libc/10696] New: tzset_internal doesn't handle changes of timezone file cfriesen at nortel dot com
@ 2009-09-25 17:18 ` cfriesen at nortel dot com
  2009-09-26 19:00 ` drepper at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: cfriesen at nortel dot com @ 2009-09-25 17:18 UTC (permalink / raw)
  To: glibc-bugs


------- Additional Comments From cfriesen at nortel dot com  2009-09-25 17:18 -------
Created an attachment (id=4231)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=4231&action=view)
proposed patch

I propose a patch something along these lines.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=10696

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug libc/10696] tzset_internal doesn't handle changes of timezone file
  2009-09-25 17:17 [Bug libc/10696] New: tzset_internal doesn't handle changes of timezone file cfriesen at nortel dot com
  2009-09-25 17:18 ` [Bug libc/10696] " cfriesen at nortel dot com
@ 2009-09-26 19:00 ` drepper at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: drepper at redhat dot com @ 2009-09-26 19:00 UTC (permalink / raw)
  To: glibc-bugs


------- Additional Comments From drepper at redhat dot com  2009-09-26 19:00 -------
No, this is deliberately like this.  The code is already slow enough.  I'm not
going to add more code to slow things down for a case that never should happen
in the first case.  Restart apps if you need to handle this.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WONTFIX


http://sourceware.org/bugzilla/show_bug.cgi?id=10696

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-09-26 19:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-09-25 17:17 [Bug libc/10696] New: tzset_internal doesn't handle changes of timezone file cfriesen at nortel dot com
2009-09-25 17:18 ` [Bug libc/10696] " cfriesen at nortel dot com
2009-09-26 19:00 ` drepper at redhat dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).