diff options
| author | Marina Makienko <makienko@ispras.ru> | 2013-02-25 23:26:50 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-02-26 23:33:56 +0100 |
| commit | 114a6f8b52163c0232fbcd7f3808ff04dc61a9b5 (patch) | |
| tree | 43e58fd3d355599e7b9fd1612eea0b1ab55e3d9c /drivers | |
| parent | bond: check if slave count is 0 in case when deciding to take slave's mac (diff) | |
| download | linux-114a6f8b52163c0232fbcd7f3808ff04dc61a9b5.tar.xz linux-114a6f8b52163c0232fbcd7f3808ff04dc61a9b5.zip | |
isdn: hisax: add missing usb_free_urb
Add missing usb_free_urb() on failure path in st5481_setup_usb().
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Marina Makienko <makienko@ispras.ru>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/isdn/hisax/st5481_usb.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/drivers/isdn/hisax/st5481_usb.c b/drivers/isdn/hisax/st5481_usb.c index 017c67ea3f4c..ead0a4fb7448 100644 --- a/drivers/isdn/hisax/st5481_usb.c +++ b/drivers/isdn/hisax/st5481_usb.c @@ -294,13 +294,13 @@ int st5481_setup_usb(struct st5481_adapter *adapter) // Allocate URBs and buffers for interrupt endpoint urb = usb_alloc_urb(0, GFP_KERNEL); if (!urb) { - return -ENOMEM; + goto err1; } intr->urb = urb; buf = kmalloc(INT_PKT_SIZE, GFP_KERNEL); if (!buf) { - return -ENOMEM; + goto err2; } endpoint = &altsetting->endpoint[EP_INT-1]; @@ -313,6 +313,14 @@ int st5481_setup_usb(struct st5481_adapter *adapter) endpoint->desc.bInterval); return 0; +err2: + usb_free_urb(intr->urb); + intr->urb = NULL; +err1: + usb_free_urb(ctrl->urb); + ctrl->urb = NULL; + + return -ENOMEM; } /* |