> -----Original Message-----
> From: hwri...@apache.org [mailto:hwri...@apache.org]
> Sent: dinsdag 6 december 2011 22:22
> To: comm...@subversion.apache.org
> Subject: svn commit: r1211162 -
> /subversion/trunk/subversion/libsvn_wc/context.c
>
> Author: hwright
> Date: Tue Dec 6 21:22:08 2011
> New Revision: 1211162
>
> URL: http://svn.apache.org/viewvc?rev=1211162&view=rev
> Log:
> Fix a potential memory cleanup ordering bug.
This introduces too many other cleanup problems. This makes the context and db
unavailable from the normal cleanup handlers in the same pool, that will run as
normal cleanup.
The wc_db is carefull to not make assumptions and allocates everything in the
same pool and *this* change breaks that by closing the db earlier than those
cleanups.
-0.5 (open for discussion).
If not extremely necessary we should never use a pre-cleanup handler in our
code as it just *starts* a race on who can cleanup first and thereby pushes the
segfaults in other code
There should be no race: everything should be implemented as a proper last-in,
first out.
Bert
