diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-09-04 14:11:23 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-09-05 17:29:22 -0700 |
| commit | b1c849276b5e199f94501101c07ee9cb6ced4935 (patch) | |
| tree | 573b7e5e189ea21351232bef5a611264136138bc /drivers/net | |
| parent | a77f9a282a2290526a6239b17ef822ec75ac793c (diff) | |
hyperv: NULL dereference on error
We try to call free_netvsc_device(net_device) when "net_device" is NULL.
It leads to an Oops.
Fixes: f90251c8a6d0 ('hyperv: Increase the buffer length for netvsc_channel_cb()')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/hyperv/netvsc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c index 5b5644a2233c..977984bc238a 100644 --- a/drivers/net/hyperv/netvsc.c +++ b/drivers/net/hyperv/netvsc.c @@ -1054,10 +1054,8 @@ int netvsc_device_add(struct hv_device *device, void *additional_info) struct net_device *ndev; net_device = alloc_net_device(device); - if (!net_device) { - ret = -ENOMEM; - goto cleanup; - } + if (!net_device) + return -ENOMEM; net_device->ring_size = ring_size; |