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:26:12 UTC
svn commit: r468056 - in /apr/apr/branches/1.2.x: CHANGES
file_io/unix/mktemp.c
Author: rooneg
Date: Thu Oct 26 09:26:12 2006
New Revision: 468056
URL: http://svn.apache.org/viewvc?view=rev&rev=468056
Log:
Merge r468055 from trunk.
Portably check for EEXIST in the mktemp code.
Modified:
apr/apr/branches/1.2.x/CHANGES
apr/apr/branches/1.2.x/file_io/unix/mktemp.c
Modified: apr/apr/branches/1.2.x/CHANGES
URL: http://svn.apache.org/viewvc/apr/apr/branches/1.2.x/CHANGES?view=diff&rev=468056&r1=468055&r2=468056
==============================================================================
--- apr/apr/branches/1.2.x/CHANGES (original)
+++ apr/apr/branches/1.2.x/CHANGES Thu Oct 26 09:26:12 2006
@@ -1,4 +1,6 @@
Changes for APR 1.2.8
+ *) Portably check for EEXIST in mktemp code. PR 40818
+ [Kenneth Golomb <KGolomb TradeCard.com>]
*) Fix apr_snprintf/apr_vsnprintf return value to not count the
NUL terminator in the overflow case. PR 39996.
Modified: apr/apr/branches/1.2.x/file_io/unix/mktemp.c
URL: http://svn.apache.org/viewvc/apr/apr/branches/1.2.x/file_io/unix/mktemp.c?view=diff&rev=468056&r1=468055&r2=468056
==============================================================================
--- apr/apr/branches/1.2.x/file_io/unix/mktemp.c (original)
+++ apr/apr/branches/1.2.x/file_io/unix/mktemp.c Thu Oct 26 09:26:12 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 */