staging: hv: remove ASSERT()s in RndisFilter.c
authorBill Pemberton <wfp5p@virginia.edu>
Wed, 5 May 2010 19:27:47 +0000 (15:27 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 11 May 2010 18:36:14 +0000 (11:36 -0700)
return -EINVAL instead of calling ASSERT() for these conditionals.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/hv/RndisFilter.c

index f0aacc0c84c9a03e38f12847b03d36dd777b5f6e..597c972f22c2ebb2fb84c53caa33fa2d1333b933 100644 (file)
@@ -391,7 +391,9 @@ static int RndisFilterOnReceive(struct hv_device *Device,
 
        DPRINT_ENTER(NETVSC);
 
-       ASSERT(netDevice);
+       if (!netDevice)
+               return -EINVAL;
+
        /* Make sure the rndis device state is initialized */
        if (!netDevice->Extension) {
                DPRINT_ERR(NETVSC, "got rndis message but no rndis device..."
@@ -492,7 +494,8 @@ static int RndisFilterQueryDevice(struct rndis_device *Device, u32 Oid,
 
        DPRINT_ENTER(NETVSC);
 
-       ASSERT(Result);
+       if (!Result)
+               return -EINVAL;
 
        *ResultSize = 0;
        request = GetRndisRequest(Device, REMOTE_NDIS_QUERY_MSG,
@@ -885,7 +888,9 @@ int RndisFilterOnOpen(struct hv_device *Device)
 
        DPRINT_ENTER(NETVSC);
 
-       ASSERT(netDevice);
+       if (!netDevice)
+               return -EINVAL;
+
        ret = RndisFilterOpenDevice(netDevice->Extension);
 
        DPRINT_EXIT(NETVSC);
@@ -900,7 +905,9 @@ int RndisFilterOnClose(struct hv_device *Device)
 
        DPRINT_ENTER(NETVSC);
 
-       ASSERT(netDevice);
+       if (!netDevice)
+               return -EINVAL;
+
        ret = RndisFilterCloseDevice(netDevice->Extension);
 
        DPRINT_EXIT(NETVSC);