From: Bill Pemberton <[email protected]>

return -EINVAL instead of calling ASSERT()

Signed-off-by: Bill Pemberton <[email protected]>
Cc: Hank Janssen <[email protected]>
Cc: Haiyang Zhang <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
---
 drivers/staging/hv/NetVsc.c |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/hv/NetVsc.c b/drivers/staging/hv/NetVsc.c
index ddde391..f852984 100644
--- a/drivers/staging/hv/NetVsc.c
+++ b/drivers/staging/hv/NetVsc.c
@@ -354,7 +354,11 @@ static int NetVscInitializeSendBufferWithNetVsp(struct 
hv_device *Device)
                DPRINT_EXIT(NETVSC);
                return -1;
        }
-       ASSERT(netDevice->SendBufferSize > 0);
+       if (netDevice->SendBufferSize <= 0) {
+               ret = -EINVAL;
+               goto Cleanup;
+       }
+
        /* page-size grandularity */
        /* ASSERT((netDevice->SendBufferSize & (PAGE_SIZE - 1)) == 0); */
 
-- 
1.7.0.3

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to