diff options
author | Tejun Heo <htejun@gmail.com> | 2006-12-11 08:26:25 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2006-12-16 10:12:29 -0500 |
commit | fae07dc389bc32a3638b9d4c61dd3738ca3fb61d (patch) | |
tree | 1bba7fc5845ddb4add50a85bd7df5a5f172957b0 /drivers/ata | |
parent | f98b6573f190aff2748894da13a48bab0f10c733 (diff) |
[PATCH] ata_piix: use piix_host_stop() in ich_pata_ops
piix_init_one() allocates host private data which should be freed by
piix_host_stop(). ich_pata_ops wasn't converted to piix_host_stop()
while merging, leaking 4 bytes on driver detach. Fix it.
This was spotted using Kmemleak by Catalin Marinas.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Cc: Catalin Marinas <catalin.marinas@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/ata')
-rw-r--r-- | drivers/ata/ata_piix.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c index 58dba1df2b7b..47701b286f8b 100644 --- a/drivers/ata/ata_piix.c +++ b/drivers/ata/ata_piix.c | |||
@@ -342,7 +342,7 @@ static const struct ata_port_operations ich_pata_ops = { | |||
342 | 342 | ||
343 | .port_start = ata_port_start, | 343 | .port_start = ata_port_start, |
344 | .port_stop = ata_port_stop, | 344 | .port_stop = ata_port_stop, |
345 | .host_stop = ata_host_stop, | 345 | .host_stop = piix_host_stop, |
346 | }; | 346 | }; |
347 | 347 | ||
348 | static const struct ata_port_operations piix_sata_ops = { | 348 | static const struct ata_port_operations piix_sata_ops = { |