Hi Godbach,
On Mon, Oct 07, 2013 at 03:37:19PM +0800, Godbach wrote:
> Yes, I have checked the manual of free(), it said that "If ptr is NULL,
> no operation is performed."
In fact it's true on *most* platforms, and all those supported by haproxy.
So we removed all of these "if" a few years ago to make the code more
readable and more maintainable. I remember that we even discovered some
copy-paste mistakes such as :
if (a)
free(a);
if (b)
free(b);
if (b)
free(c);
> Please remove it directly.
Done :-)
Willy