You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@apr.apache.org by ro...@apache.org on 2006/10/26 18:27:18 UTC

svn commit: r468057 - in /apr/apr/branches/0.9.x: CHANGES file_io/unix/mktemp.c

Author: rooneg
Date: Thu Oct 26 09:27:17 2006
New Revision: 468057

URL: http://svn.apache.org/viewvc?view=rev&rev=468057
Log:
Merge r468055 from trunk.

Portably check for EEXIST in the mktemp code.

Modified:
    apr/apr/branches/0.9.x/CHANGES
    apr/apr/branches/0.9.x/file_io/unix/mktemp.c

Modified: apr/apr/branches/0.9.x/CHANGES
URL: http://svn.apache.org/viewvc/apr/apr/branches/0.9.x/CHANGES?view=diff&rev=468057&r1=468056&r2=468057
==============================================================================
--- apr/apr/branches/0.9.x/CHANGES (original)
+++ apr/apr/branches/0.9.x/CHANGES Thu Oct 26 09:27:17 2006
@@ -1,4 +1,6 @@
 Changes with APR 0.9.13
+  *) Portably check for EEXIST in mktemp code.  PR 40818
+     [Kenneth Golomb <KGolomb TradeCard.com>]
 
   *) Fix apr_atomic_cas on platforms with 64 bit longs.
      [Philip Martin <philip codematters.co.uk>]

Modified: apr/apr/branches/0.9.x/file_io/unix/mktemp.c
URL: http://svn.apache.org/viewvc/apr/apr/branches/0.9.x/file_io/unix/mktemp.c?view=diff&rev=468057&r1=468056&r2=468057
==============================================================================
--- apr/apr/branches/0.9.x/file_io/unix/mktemp.c (original)
+++ apr/apr/branches/0.9.x/file_io/unix/mktemp.c Thu Oct 26 09:27:17 2006
@@ -142,7 +142,7 @@
         if ((rv = apr_file_open(doopen, path, flags,
                                 APR_UREAD | APR_UWRITE, p)) == APR_SUCCESS)
             return APR_SUCCESS;
-        if (rv != APR_EEXIST)
+        if (!APR_STATUS_IS_EEXIST(rv))
             return rv;
 
         /* If we have a collision, cycle through the space of filenames */