diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2012-01-12 10:55:13 +0100 |
---|---|---|
committer | Simone Willett <swillett@nvidia.com> | 2012-02-08 14:06:22 -0800 |
commit | 9369846982cbbf3ccfe90c772ae2415b1af9278d (patch) | |
tree | 92528e09957eaef763ff30276041f26bb128d643 /drivers/usb/host/ehci-fsl.c | |
parent | c6f9dd5bec25ddcd54c74ed5f12930a5277262d5 (diff) |
drivers/usb/host/ehci-fsl.c: add missing iounmap
commit 2492c6e6454ff3edb11e273b071a6ea80a199c71 upstream.
Add missing iounmap in error handling code, in a case where the function
already preforms iounmap on some other execution path.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// <smpl>
@@
expression e;
statement S,S1;
int ret;
@@
e = \(ioremap\|ioremap_nocache\)(...)
... when != iounmap(e)
if (<+...e...+>) S
... when any
when != iounmap(e)
*if (...)
{ ... when != iounmap(e)
return ...; }
... when any
iounmap(e);
// </smpl>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>
Change-Id: Ic3dc56910f445c605ae8ab0c768c9e4820322d0a
Reviewed-on: http://git-master/r/79680
Reviewed-by: Automatic_Commit_Validation_User
Diffstat (limited to 'drivers/usb/host/ehci-fsl.c')
-rw-r--r-- | drivers/usb/host/ehci-fsl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/host/ehci-fsl.c b/drivers/usb/host/ehci-fsl.c index ec39f8b57351..53ca216657d9 100644 --- a/drivers/usb/host/ehci-fsl.c +++ b/drivers/usb/host/ehci-fsl.c @@ -122,7 +122,7 @@ static int usb_hcd_fsl_probe(const struct hc_driver *driver, */ if (pdata->init && pdata->init(pdev)) { retval = -ENODEV; - goto err3; + goto err4; } /* Enable USB controller, 83xx or 8536 */ |