On 20.09.2011 21:30, Daniel Shahaf wrote:
ack, did you intend to CC the list too?

Ups. Yes, of course. CC now.


Stefan Küng wrote on Tue, Sep 20, 2011 at 21:17:57 +0200:
On 20.09.2011 20:53, Daniel Shahaf wrote:
Mattias Engdegård (CC'd) reports on IRC that apr_atomic_xchgptr() in
recent APR's is broken, and the brokenness can affect Subversion under
certain circumstances:

<@danielsh>   wayita: t
<   wayita>   gcc 4.0.x or older
<   wayita>   apr 1.3.x or newer
<   wayita>   x86-64 only
<   wayita>   WORKAROUND: --disable-nonportable-atomics to APR's configure
<   wayita>   OS: any unix
<   wayita>   nature of the bug: apr_atomic_xchgptr() does not initialize the 
return value
<   wayita>   https://issues.apache.org/bugzilla/show_bug.cgi?id=51851
<   wayita>   manifests as a segfault in svn.

I suppose we could recommend --disable-nonportable-atomics in the
release notes.  It is undetermined at this time whether Windows 64-bit
is affected.

At least for the Visual Studio compiler, it definitely is not
affected for x64 since there's no inline assembler for x64:
http://msdn.microsoft.com/en-us/library/wbk4z78b.aspx

Stefan

--
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>     The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net


--
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net

Reply via email to