From: Rafal Somla
Date: May 2 2011 10:24am
Subject: Re: bzr push into mysql-trunk branch (rafal.somla:3345 to 3346) Bug#11766631
 Bug#11879051
List-Archive: http://lists.mysql.com/commits/136510
Message-Id: <4DBE8646.2010509@oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Hi again,

On 29/04/2011 21:56, Davi Arnaut wrote:
> On 4/29/11 4:45 PM, Rafal Somla wrote:
>> + if (m_buf)
>> + free(m_buf);
>
> free() handles NULL pointers just fine.

Correct, but above code has 2 slight advantages:

1. It might be more efficient than calling free() in case m_buf is NULL.

2. It will always work, even if on some exotic platform/configuration, 
free() does not handle NULL pointers.

Rafal