>>> +
>>> + PWSTR wsName = OvsAllocateMemory(wstrSize);
>>> + if (!wsName) {
>>> + vport = NULL;
>>> + goto Cleanup;
>>
>> If we jump to 'Cleanup', and call OvsFreeMemory(wsName) while wsName ==
>> NULL, we'll hit an ASSERT in OvsFreeMemory() that checks for a valid pointer.
>>
> Well I wanted to ASSERT there but I can just return NULL if you think is
> better.
I think we should return NULL.
-- Nithin
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev