After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 129983 - libexslt date functions are not thread safe.
libexslt date functions are not thread safe.
Status: VERIFIED FIXED
Product: libxslt
Classification: Platform
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2003-12-25 09:06 UTC by Vasily Chekalkin
Modified: 2009-08-15 18:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch. (745 bytes, patch)
2003-12-25 09:08 UTC, Vasily Chekalkin
none Details | Review

Description Vasily Chekalkin 2003-12-25 09:06:51 UTC
libexslt date functions are not thread safe because exsltDateCurrent use
localtime, not localtime_r, in thread enabled builds.
Comment 1 Vasily Chekalkin 2003-12-25 09:08:26 UTC
Created attachment 22705 [details] [review]
Proposed patch.
Comment 2 William M. Brack 2004-02-26 17:09:40 UTC
There was a bit more work required (checking during library 
configuration that localtime_r is present on the system), plus the 
macro LIBXML_THREAD_ENABLED is not present within date.c, but I hope 
that the changes I have made will accomplish what you wanted.  The 
changed routine (libexslt/date.c) is in CVS.
Comment 3 Daniel Veillard 2004-03-25 11:38:17 UTC
This should be closed by release of libxslt-1.1.5,
                                                                                
  thanks,
                                                                                
Daniel
Comment 4 Christopher R. Palmer 2004-09-18 13:52:59 UTC
This function still uses gmtime which is not thread-safe.  gmtime_r is a
thread-safe version.
Comment 5 William M. Brack 2004-10-15 05:37:52 UTC
OK, I extended the previous modification to also use gmtime_r.  Fix is in CVS (libexslt/date.c, 
configure.in, config.h.in).  Thanks for the report.
Comment 6 Daniel Veillard 2005-09-05 09:44:09 UTC
This should be closed by release of libxslt-1.1.15

  thanks,

Daniel