Hi Ryan, Thanks for the comments. I think free() in libc internally will check if remotes_error is NULL or not.
Regards,
William
> > }
> >
> > + free(remotes_error);
>
> Shouldn't this be
>
> if (remotes_error) {
> free(remotes_error);
> }
>
> Ryan Moats
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
