On Wed, 2013-07-03 at 15:11 +0200, Mark Wielaard wrote:
Indeed it would, as would some other failure paths. Thanks for
spotting
that. I tweaked the patch a little to initialize everything slightly
earlier and do the deallocation at the very end (all success and failure
paths end up at out:). New patch attached. Also tested with valgrind to
make sure there are no leaks or erroneous frees.
I pushed this variant to master now.
Thanks,
Mark