standardize some apr_foo_close() functions (call apr_pool_cleanup_run())

a couple of these functions didn't kill the cleanup if it failed; we might
as well; the error isn't going to magically disappear next time we try


git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@63272 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 1 insertions, 7 deletions

diff --git a/mmap/unix/mmap.c b/mmap/unix/mmap.c
index 3f081308e..4a54b860e 100644
--- a/mmap/unix/mmap.c
+++ b/mmap/unix/mmap.c
@@ -199,13 +199,7 @@ APR_DECLARE(apr_status_t) apr_mmap_dup(apr_mmap_t **new_mmap,
 
 APR_DECLARE(apr_status_t) apr_mmap_delete(apr_mmap_t *mm)
 {
-    apr_status_t rv;
-
-    if ((rv = mmap_cleanup(mm)) == APR_SUCCESS) {
-        apr_pool_cleanup_kill(mm->cntxt, mm, mmap_cleanup);
-        return APR_SUCCESS;
-    }
-    return rv;
+    return apr_pool_cleanup_run(mm->cntxt, mm, mmap_cleanup);
 }
 
 #endif