diff options
| author | Tejun Heo <htejun@gmail.com> | 2006-12-11 22:26:25 +0900 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-12-16 10:12:29 -0500 |
| commit | fae07dc389bc32a3638b9d4c61dd3738ca3fb61d (patch) | |
| tree | 1bba7fc5845ddb4add50a85bd7df5a5f172957b0 /drivers | |
| parent | f98b6573f190aff2748894da13a48bab0f10c733 (diff) | |
| download | op-kernel-dev-fae07dc389bc32a3638b9d4c61dd3738ca3fb61d.zip op-kernel-dev-fae07dc389bc32a3638b9d4c61dd3738ca3fb61d.tar.gz | |
[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.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c index 58dba1d..47701b2 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 = { |
