>>> +
>>> +    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
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to