diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2014-08-07 22:57:47 +0200 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2014-08-19 09:24:33 -0500 |
commit | 20e7d4653d2b580e490dfbb1dfd138cd9844319c (patch) | |
tree | c9ab22e5ff0e1c9d2159518f974b6e7527d62f97 /drivers/usb | |
parent | bbc66e140babbc7026fa07478b588f2b56f99751 (diff) | |
download | op-kernel-dev-20e7d4653d2b580e490dfbb1dfd138cd9844319c.zip op-kernel-dev-20e7d4653d2b580e490dfbb1dfd138cd9844319c.tar.gz |
usb: gadget: fix error return code
Convert a zero return value on error to a negative one, as returned
elsewhere in the function.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// <smpl>
@@
identifier ret; expression e1,e2;
@@
(
if (\(ret < 0\|ret != 0\))
{ ... return ret; }
|
ret = 0
)
... when != ret = e1
when != &ret
*if(...)
{
... when != ret = e2
when forall
return ret;
}
// </smpl>
Reviewed-by: Jeff Moyer <jmoyer@redhat.com>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/gadget/udc/fusb300_udc.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/usb/gadget/udc/fusb300_udc.c b/drivers/usb/gadget/udc/fusb300_udc.c index d40255f..5c5d1ad 100644 --- a/drivers/usb/gadget/udc/fusb300_udc.c +++ b/drivers/usb/gadget/udc/fusb300_udc.c @@ -1398,13 +1398,17 @@ static int fusb300_probe(struct platform_device *pdev) /* initialize udc */ fusb300 = kzalloc(sizeof(struct fusb300), GFP_KERNEL); - if (fusb300 == NULL) + if (fusb300 == NULL) { + ret = -ENOMEM; goto clean_up; + } for (i = 0; i < FUSB300_MAX_NUM_EP; i++) { _ep[i] = kzalloc(sizeof(struct fusb300_ep), GFP_KERNEL); - if (_ep[i] == NULL) + if (_ep[i] == NULL) { + ret = -ENOMEM; goto clean_up; + } fusb300->ep[i] = _ep[i]; } |