>>>> +
>>>> + if (cmd_result) {
>>>> + free(cmd_result);
>>>> + }
>>>> +
>>>> + if (cmd_error) {
>>>> + free(cmd_error);
>>>> + }
>>>
>>> I don't think the cmd_result and cmd_error conditions are required
>>> as free should do nothing when passed NULL.
>>>
>>
>> Isn't free behavior for NULL platform dependent? I was under the impression
>> that it is.
>> But I might be mistaken.
>
> Maybe: not so secret fact is that I'm a Linux person.
It's part of a C standard, so not platform dependent.
And this is also part of a coding style. :)
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
