[ovs-dev] [PATCH] datapath-windows: cleanup after NdisRegisterDeviceEx()
Nithin Raju
nithin at vmware.com
Tue Apr 5 22:07:54 UTC 2016
The error handling code seems to be wrong.
Signed-off-by: Nithin Raju <nithin at vmware.com>
---
datapath-windows/ovsext/Datapath.c | 12 ++----------
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/datapath-windows/ovsext/Datapath.c b/datapath-windows/ovsext/Datapath.c
index 464fa97..dc6baaa 100644
--- a/datapath-windows/ovsext/Datapath.c
+++ b/datapath-windows/ovsext/Datapath.c
@@ -456,18 +456,10 @@ OvsCreateDeviceObject(NDIS_HANDLE ovsExtDriverHandle)
&deviceAttributes,
&gOvsDeviceObject,
&gOvsDeviceHandle);
- if (status != NDIS_STATUS_SUCCESS) {
- POVS_DEVICE_EXTENSION ovsExt =
- (POVS_DEVICE_EXTENSION)NdisGetDeviceReservedExtension(gOvsDeviceObject);
- ASSERT(gOvsDeviceObject != NULL);
- ASSERT(gOvsDeviceHandle != NULL);
-
- if (ovsExt) {
- ovsExt->numberOpenInstance = 0;
- }
- } else {
+ if (status == NDIS_STATUS_SUCCESS) {
OvsRegisterSystemProvider((PVOID)gOvsDeviceObject);
}
+ ASSERT(status == NDIS_STATUS_SUCCESS || gOvsDeviceObject == NULL);
OVS_LOG_TRACE("DeviceObject: %p", gOvsDeviceObject);
return status;
--
2.6.2
More information about the dev
mailing list