diff options
author | Sorin Vinturis <svinturis@cloudbasesolutions.com> | 2014-11-18 14:20:39 +0000 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2014-11-24 15:40:03 -0800 |
commit | 21b83c6d6035828dce80774746d9a5b0e50d0981 (patch) | |
tree | 8e827331f55b95b0717bf20c8aa9192f2f1f29e9 | |
parent | aa09ab5d4be6d371107a86fe74b6a2a4184a9045 (diff) | |
download | openvswitch-21b83c6d6035828dce80774746d9a5b0e50d0981.tar.gz |
datapath-windows: Removed duplicate checking of previous dump state
Removed the check for previous dump state before calling FreeUserDumpState
as the check is also performed in the latter function.
Signed-off-by: Sorin Vinturis <svinturis@cloudbasesolutions.com>
Acked-by: Nithin Raju <nithin@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
-rw-r--r-- | datapath-windows/ovsext/Datapath.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/datapath-windows/ovsext/Datapath.c b/datapath-windows/ovsext/Datapath.c index 49a0b68e6..ce56e3266 100644 --- a/datapath-windows/ovsext/Datapath.c +++ b/datapath-windows/ovsext/Datapath.c @@ -783,7 +783,7 @@ OvsDeviceControl(PDEVICE_OBJECT deviceObject, * state in the instance to indicate the command that started the dump * operation. The state can setup 'ovsMsgReadOp' appropriately. Note * 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. @@ -1350,9 +1350,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); return InitUserDumpState(instance, msgIn); } |