diff options
author | Dan Carpenter <error27@gmail.com> | 2010-08-21 19:24:59 (GMT) |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-08-27 17:27:05 (GMT) |
commit | df1c2986c4685776ed02802c170c7fb6f47d8218 (patch) | |
tree | cef7d77b4d901ecc30ad2734d5110c303080bd49 | |
parent | 903786a5626e7214d97b232bece88ee75e37d021 (diff) | |
download | linux-fsl-qoriq-df1c2986c4685776ed02802c170c7fb6f47d8218.tar.xz |
ath5k: re-order one of the frees on unwind
There was a small misordering here. In the original code, if we were to
go to err_free_ah then it wouldn't free the irq.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | drivers/net/wireless/ath/ath5k/base.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c index f2fceb1..45789c8 100644 --- a/drivers/net/wireless/ath/ath5k/base.c +++ b/drivers/net/wireless/ath/ath5k/base.c @@ -700,10 +700,10 @@ ath5k_pci_probe(struct pci_dev *pdev, return 0; err_ah: ath5k_hw_detach(sc->ah); -err_irq: - free_irq(pdev->irq, sc); err_free_ah: kfree(sc->ah); +err_irq: + free_irq(pdev->irq, sc); err_free: ieee80211_free_hw(hw); err_map: |