apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r..@covalent.net
Subject Re: cvs commit: apr/lib apr_pools.c
Date Sat, 06 Jan 2001 20:25:41 GMT

> >   @@ -702,6 +706,8 @@
> >    #if APR_HAS_THREADS
> >        apr_destroy_lock(alloc_mutex);
> >        apr_destroy_lock(spawn_mutex);
> >   +    alloc_mutex = NULL;
> >   +    spawn_mutex = NULL;
> >    #endif
> >        apr_destroy_pool(globalp);
> >    }
> 
> To be clear, it doesn't matter what platform it is: nobody can "NULL out
> locks" in the above case. apr_destroy_lock() can't reach out and set
> alloc_mutex to NULL unless the call pattern is &alloc_mutex.
> 
> IOW, this isn't platform dependent. It may simply be that other platforms
> are being a bit more tolerant; that the lock is still sitting around in a
> useful format in free'd memory.

Good point.

Ryan

_______________________________________________________________________________
Ryan Bloom                        	rbb@apache.org
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------


Mime
View raw message