Re: Problem with last commit

[Willy Tarreau]

Hi Thomas,

On Sat, May 17, 2014 at 03:12:40PM +0200, Thomas Heil wrote:
> The patch works. No crashes anymore. Thanks a lot,

Thanks for confirming, however the fix is still not good (regfree
may only be called with a non-null pointer, and still needs free()
to be run afterwards). Could you please check with this one instead :

        free(errmsg);
-       free(preg);
+       if (preg) {
+               regfree(preg);
+               free(preg);
+       }

Thanks,
Willy