[PATCH] ata_piix: use piix_host_stop() in ich_pata_ops
authorTejun Heo <htejun@gmail.com>
Mon, 11 Dec 2006 13:26:25 +0000 (22:26 +0900)
committerJeff Garzik <jeff@garzik.org>
Sat, 16 Dec 2006 15:12:29 +0000 (10:12 -0500)
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>
drivers/ata/ata_piix.c

index 58dba1df2b7b03207ed94d08d55aee73f821e09c..47701b286f8b6e2a124392c8b8b2befe27051139 100644 (file)
@@ -342,7 +342,7 @@ static const struct ata_port_operations ich_pata_ops = {
 
        .port_start             = ata_port_start,
        .port_stop              = ata_port_stop,
-       .host_stop              = ata_host_stop,
+       .host_stop              = piix_host_stop,
 };
 
 static const struct ata_port_operations piix_sata_ops = {