On Nov 25, 2014, at 9:09 AM, Alin Serdean <aserd...@cloudbasesolutions.com>
 wrote:

> +    if (datapath->flowTable == NULL) {
> +        status = STATUS_INVALID_PARAMETER;
> +        goto dp_unlock;
> +    }

Alin,
We allocate the flow table in OvsAllocateFlowTable() when the datapath is 
allocated, which is when the 'gOvsSwitchContext' is also initialized.

The flow dump command handler which eventually calls into 
_FlowNlDumpCmdHandler() does check to see if 'gOvsSwitchContext' is valid or 
not since 'validateDpIndex' is TRUE in ValidateNetlinkCmd().

I was wondering how we'd end up exercising this code.

Thanks,
-- Nithin

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to