You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by bo...@apache.org on 2008/07/15 07:23:55 UTC

svn commit: r676801 - /apr/apr-util/branches/1.3.x/misc/apr_reslist.c

Author: bojan
Date: Mon Jul 14 22:23:54 2008
New Revision: 676801

URL: http://svn.apache.org/viewvc?rev=676801&view=rev
Log:
Backport r676800 from the trunk.
Unlock list mutex before destroying it.
Manual page for pthread_mutex_destroy() states:
Attempting to destroy a locked mutex results in undefined behavior.

Modified:
    apr/apr-util/branches/1.3.x/misc/apr_reslist.c

Modified: apr/apr-util/branches/1.3.x/misc/apr_reslist.c
URL: http://svn.apache.org/viewvc/apr/apr-util/branches/1.3.x/misc/apr_reslist.c?rev=676801&r1=676800&r2=676801&view=diff
==============================================================================
--- apr/apr-util/branches/1.3.x/misc/apr_reslist.c (original)
+++ apr/apr-util/branches/1.3.x/misc/apr_reslist.c Mon Jul 14 22:23:54 2008
@@ -158,6 +158,7 @@
     assert(rl->nidle == 0);
     assert(rl->ntotal == 0);
 
+    apr_thread_mutex_unlock(rl->listlock);
     apr_thread_mutex_destroy(rl->listlock);
     apr_thread_cond_destroy(rl->avail);