diff options
author | Tejun Heo <htejun@gmail.com> | 2007-02-27 08:33:21 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-03-01 20:16:28 -0500 |
commit | 1aa56cca5bfa396c791e93b38e4776ef39b590a7 (patch) | |
tree | c4e88e2195dfc1db88c932e5c2b9adb151bb9d65 /drivers/ata/pata_isapnp.c | |
parent | 7b965e0884cee430ffe5dc81cdb117b9316b0549 (diff) |
libata: clear drvdata in ata_host_release(), take#2
Clearing drvdata in ->remove_one causes NULL pointer deference. Clear
drvdata only in ata_host_release() after all resources are freed.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/ata/pata_isapnp.c')
-rw-r--r-- | drivers/ata/pata_isapnp.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/ata/pata_isapnp.c b/drivers/ata/pata_isapnp.c index d5f2e85e28f3..1a61cc891741 100644 --- a/drivers/ata/pata_isapnp.c +++ b/drivers/ata/pata_isapnp.c | |||
@@ -128,7 +128,6 @@ static void isapnp_remove_one(struct pnp_dev *idev) | |||
128 | struct ata_host *host = dev_get_drvdata(dev); | 128 | struct ata_host *host = dev_get_drvdata(dev); |
129 | 129 | ||
130 | ata_host_detach(host); | 130 | ata_host_detach(host); |
131 | dev_set_drvdata(dev, NULL); | ||
132 | } | 131 | } |
133 | 132 | ||
134 | static struct pnp_device_id isapnp_devices[] = { | 133 | static struct pnp_device_id isapnp_devices[] = { |