On Mon, Nov 14, 2011 at 11:49 PM, Albert Chu <[email protected]> wrote:
> +       if (opt->vlarb_high)
> +               free(opt->vlarb_high);

Those if-statements are superfluous - invoking free(NULL) is safe. See
e.g. http://pubs.opengroup.org/onlinepubs/009695399/functions/free.html.

Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to