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

Posted to commits@apr.apache.org by mt...@apache.org on 2009/02/20 19:01:10 UTC

svn commit: r746310 - /apr/apr/trunk/file_io/win32/open.c

Author: mturk
Date: Fri Feb 20 18:01:09 2009
New Revision: 746310

URL: http://svn.apache.org/viewvc?rev=746310&view=rev
Log:
Do not close the original if the apr_os_file_put fails.
This ensures the same behavior as on other platforms

Modified:
    apr/apr/trunk/file_io/win32/open.c

Modified: apr/apr/trunk/file_io/win32/open.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/file_io/win32/open.c?rev=746310&r1=746309&r2=746310&view=diff
==============================================================================
--- apr/apr/trunk/file_io/win32/open.c (original)
+++ apr/apr/trunk/file_io/win32/open.c Fri Feb 20 18:01:09 2009
@@ -645,9 +645,6 @@
         rv = apr_thread_mutex_create(&(*file)->mutex, 
                                      APR_THREAD_MUTEX_DEFAULT, pool);
         if (rv) {
-            if (file_cleanup(*file) == APR_SUCCESS) {
-                apr_pool_cleanup_kill(pool, *file, file_cleanup);
-            }
             return rv;
         }
     }