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


Reply via email to