Bug 33464

Summary: errno isn't thread-safe on Solaris without -D_REENTRANT
Product: dbus Reporter: Simon McVittie <smcv>
Component: coreAssignee: Simon McVittie <smcv>
Status: RESOLVED FIXED QA Contact: John (J5) Palmieri <johnp>
Severity: normal    
Priority: medium CC: hp
Version: 1.4.x   
Hardware: All   
OS: Solaris   
Whiteboard:
i915 platform: i915 features:
Attachments: Opt-in to thread safety on Solaris

Description Simon McVittie 2011-01-25 06:41:51 UTC
Solaris' errno.h apparently needs -D_REENTRANT to make it not suck.

I'm tempted to just say "get a better C library", but the fix is easy.
Comment 1 Simon McVittie 2011-01-26 10:24:54 UTC
Created attachment 42541 [details] [review]
Opt-in to thread safety on Solaris
Comment 2 Colin Walters 2011-01-27 09:43:14 UTC
Comment on attachment 42541 [details] [review]
Opt-in to thread safety on Solaris

This looks fine, but can you add a reference to the mailing list thread?  Looks like 
http://lists.freedesktop.org/archives/dbus/2010-December/013902.html

?
Comment 3 Simon McVittie 2011-01-27 10:00:07 UTC
The original report was <http://lists.freedesktop.org/archives/dbus/2010-November/013796.html>. I think in general it's sufficient to reference mailing list threads from Bugzilla and reference the bug in the commit, but I've added a "see also".

I didn't apply the patch from Pavel's December mail, partly because it addressed several unrelated issues and partly because I couldn't find it in my D-Bus mailbox at the time.

Anyway, fixed in git for 1.4.4 (I think this is fine for 1.4).

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.