subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Philip Martin <philip.mar...@wandisco.com>
Subject Re: svn commit: r1325478 - in /subversion/trunk/subversion: include/svn_pools.h libsvn_subr/pool.c
Date Fri, 13 Apr 2012 16:44:52 GMT
Stefan Fuhrmann <stefanfuhrmann@alice-dsl.de> writes:

> Am 13.04.2012 16:35, schrieb Daniel Shahaf:
>> stefan2@apache.org wrote on Thu, Apr 12, 2012 at 20:02:37 -0000:

>>> + * @since: New in 1.8
>>> + */
>>> +apr_allocator_t *
>>> +svn_pool_create_allocator(svn_boolean_t thread_safe);
>> Should this be an svn-private API?  Should it take a parameter
>> that allows overriding the SVN_ALLOCATOR_RECOMMENDED_MAX_FREE default?
>>
> Not too long ago, I spent a couple of days hunting
> memory corruptions. The reason turned out to be
> the allocator not being thread-safe by default while
> the default allocator was.

If that's correct then this bug is present in all Subversion releases;
does it affect threaded Apache or threaded svnserve?  Do we need to
backport to 1.7 and 1.6?

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com

Mime
View raw message