summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorTejun Heo2006-12-11 14:26:25 +0100
committerJeff Garzik2006-12-16 16:12:29 +0100
commitfae07dc389bc32a3638b9d4c61dd3738ca3fb61d (patch)
tree1bba7fc5845ddb4add50a85bd7df5a5f172957b0 /drivers
parent[PATCH] ata_piix: IDE mode SATA patch for Intel ICH9 (diff)
downloadkernel-qcow2-linux-fae07dc389bc32a3638b9d4c61dd3738ca3fb61d.tar.gz
kernel-qcow2-linux-fae07dc389bc32a3638b9d4c61dd3738ca3fb61d.tar.xz
kernel-qcow2-linux-fae07dc389bc32a3638b9d4c61dd3738ca3fb61d.zip
[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')
-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 = {
.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 = {