Hi Jean-Frédéric, On Tue, Jun 19, 2018 at 10:01 AM, <jfcl...@apache.org> wrote: > Author: jfclere > Date: Tue Jun 19 08:01:45 2018 > New Revision: 1833786 > > URL: http://svn.apache.org/viewvc?rev=1833786&view=rev > Log: > proposal for review.
Usually APR backports are CTR (Commit Then Review), but it does not prevent to propose them first ;) I don't find the corresponding trunk commit here, though. > > + * make sure we don't unlock mutex when we haven't locked it. > + 1.6.x patch: http://people.apache.org/~jfclere/patch.180618.txt (assuming this is http://people.apache.org/~jfclere/patches/patch.180618.txt, above is 404). In 1.6.x, the mutex is locked unconditionally, I don't think we can unlock it when ->buffered only. Don't you want the whole http://svn.apache.org/r1808457 instead? Regards, Yann.