[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[BUG] Mutex is destroyed when clearing top level pool, WAS: RE: svn commit: rev 2017 - trunk/subversion/libsvn_subr

From: Sander Striker <striker_at_apache.org>
Date: 2002-05-28 00:10:43 CEST

> From: Sander Striker [mailto:striker@apache.org]
> Sent: 27 May 2002 03:51

>> From: Philip Martin [mailto:pm@localhost]On Behalf Of Philip Martin
>> Sent: 27 May 2002 03:13
>> striker@tigris.org writes:
>>
>>> Author: striker
>>> Date: 2002-05-26 09:27 GMT
>>> New Revision: 2017

[...]
>> Since this change the Subversion regression tests are failing when
>> compiled with APR_POOL_DEBUG=1. They hang in svn_pool_clear() when it
>> recreates the Subversion error pool. Essentially the Subversion code
>> does this
>>
>> apr_pool_clear_debug (some_pool, file_line);
>> apr_pool_create_ex(error_pool, some_pool, abort_on_pool_failure, NULL);
>>
>> The problem is that when some_pool was originally created by
>> apr_pool_create_ex() a mutex was created which was allocated within
>> some_pool itself. When apr_pool_clear_debug is called it free's all
>> the allocated memory including the memory used for mutex. Thus any
>> further attempt to use some_pool, such as for the parent of the new
>> error_pool, may fail when attempting the lock some_pool's mutex. The
>> Subversion regression tests hang attempting to lock this mutex.
>
> Correct. I'll look into a fix somewhere tonight.

Hmmm, well, it _is_ evening now, but I don't really have the energy
to patch it up. If someone beats me to it, fine, otherwise just don't
run with apr pool debugging enabled for now.

Sander

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 28 00:04:13 2002

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.