>>> + >>> + 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