Re: svn commit: rev 3701 - trunk/subversion/libsvn_subr

Justin Erenkrantz <jerenkrantz@apache.org> writes:

> No. r3701 is the correct solution. Nothing else makes any sense.
> What we had before was just blantantly wrong. The only thing I can
> think of is that there might be an iconv() library that isn't
> thread-safe and doesn't support multiple open buffers. But, that'd be
> a flaw in the iconv() implementation not the core iconv() API (as it
> worked on Darwin).

What happens is that the pool gets destroyed, the pool cleanup
handlers run, one of which is the apr_xlate_t handler which calls
iconv_close(). However the userdata still points to the apr_xlate_t.
A second, independent, pool cleanup handler then runs which attempts
to do a UTF8 conversion. This looks up the userdata and gets the
address of the apr_xlate_t object for which iconv_close has been
called.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org