summaryrefslogtreecommitdiffstats
path: root/drivers/usb/host/ehci-ppc-of.c
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2010-08-14 11:06:19 +0200
committerGreg Kroah-Hartman <gregkh@suse.de>2010-09-03 17:33:40 -0700
commit08a3b3b1c2e622e378d9086aee9e2e42ce37591d (patch)
treefe52d1eba9ec20201de1ea723e51628eb9522894 /drivers/usb/host/ehci-ppc-of.c
parent0f1312b260499b34bb92fc9bd8ca1560dda2c4da (diff)
downloadop-kernel-dev-08a3b3b1c2e622e378d9086aee9e2e42ce37591d.zip
op-kernel-dev-08a3b3b1c2e622e378d9086aee9e2e42ce37591d.tar.gz
USB: ehci-ppc-of: problems in unwind
The iounmap(ehci->ohci_hcctrl_reg); should be the first thing we do because the ioremap() was the last thing we did. Also if we hit any of the goto statements in the original code then it would have led to a NULL dereference of "ehci". This bug was introduced in: 796bcae7361c "USB: powerpc: Workaround for the PPC440EPX USBH_23 errata [take 3]" I modified the few lines in front a little so that my code didn't obscure the return success code path. Signed-off-by: Dan Carpenter <error27@gmail.com> Reviewed-by: Grant Likely <grant.likely@secretlab.ca> Cc: stable <stable@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/host/ehci-ppc-of.c')
-rw-r--r--drivers/usb/host/ehci-ppc-of.c12
1 files changed, 7 insertions, 5 deletions
diff --git a/drivers/usb/host/ehci-ppc-of.c b/drivers/usb/host/ehci-ppc-of.c
index 335ee69..ba52be4 100644
--- a/drivers/usb/host/ehci-ppc-of.c
+++ b/drivers/usb/host/ehci-ppc-of.c
@@ -192,17 +192,19 @@ ehci_hcd_ppc_of_probe(struct platform_device *op, const struct of_device_id *mat
}
rv = usb_add_hcd(hcd, irq, 0);
- if (rv == 0)
- return 0;
+ if (rv)
+ goto err_ehci;
+
+ return 0;
+err_ehci:
+ if (ehci->has_amcc_usb23)
+ iounmap(ehci->ohci_hcctrl_reg);
iounmap(hcd->regs);
err_ioremap:
irq_dispose_mapping(irq);
err_irq:
release_mem_region(hcd->rsrc_start, hcd->rsrc_len);
-
- if (ehci->has_amcc_usb23)
- iounmap(ehci->ohci_hcctrl_reg);
err_rmr:
usb_put_hcd(hcd);
OpenPOWER on IntegriCloud