aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/ata/ata_piix.c
diff options
context:
space:
mode:
authorTejun Heo <htejun@gmail.com>2006-12-11 08:26:25 -0500
committerJeff Garzik <jeff@garzik.org>2006-12-16 10:12:29 -0500
commitfae07dc389bc32a3638b9d4c61dd3738ca3fb61d (patch)
tree1bba7fc5845ddb4add50a85bd7df5a5f172957b0 /drivers/ata/ata_piix.c
parentf98b6573f190aff2748894da13a48bab0f10c733 (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/ata_piix.c')
-rw-r--r--drivers/ata/ata_piix.c2
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
348static const struct ata_port_operations piix_sata_ops = { 348static const struct ata_port_operations piix_sata_ops = {