On Nov 18, 2014, at 6:20 AM, Sorin Vinturis <[email protected]>
wrote:
> * that 'ovsMsgReadOp' is needed only in this function to call into
> the
> - * appropraite handler. The handler itself can access the state in
> the
> + * appropriate handler. The handler itself can access the state in
> the
> * instance.
> *
> * In the absence of a dump start, return 0 bytes.
> @@ -1344,9 +1344,7 @@ OvsSetupDumpStart(POVS_USER_PARAMS_CONTEXT usrParamsCtx)
> * This operation should be setting up the dump state. If there's any
> * previous state, clear it up so as to set it up afresh.
> */
> - if (instance->dumpState.ovsMsg != NULL) {
> - FreeUserDumpState(instance);
> - }
> + FreeUserDumpState(instance);
Thanks for the cleanup.
Acked-by: Nithin Raju <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
