diff options
author | Konrad Rzeszutek <konradr@redhat.com> | 2006-06-15 12:08:30 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-06-21 12:00:01 -0700 |
commit | acc7c2e0b73a46122ec370bf8a3aa9f19065d331 (patch) | |
tree | 6e37db90a6e73aa2095d00e66314b3c3dec7e072 /arch/i386/kernel/acpi | |
parent | 4d15a1779dfdf7e7a111022697d3a43da1745d31 (diff) | |
download | op-kernel-dev-acc7c2e0b73a46122ec370bf8a3aa9f19065d331.zip op-kernel-dev-acc7c2e0b73a46122ec370bf8a3aa9f19065d331.tar.gz |
[PATCH] PCI: fix memory leak in MMCONFIG error path
This a bit late (yours patch was posted about a year ago), but
a co-worker of spotted part of the code that looks like a memory
leak. Looking at the code it seems that pci_mmcfg_config should
be free-ed if MMCONFIG is above 4GB.
From: Konrad Rzeszutek <konradr@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'arch/i386/kernel/acpi')
-rw-r--r-- | arch/i386/kernel/acpi/boot.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/arch/i386/kernel/acpi/boot.c b/arch/i386/kernel/acpi/boot.c index 40e5aba..fbe9308 100644 --- a/arch/i386/kernel/acpi/boot.c +++ b/arch/i386/kernel/acpi/boot.c @@ -202,6 +202,8 @@ int __init acpi_parse_mcfg(unsigned long phys_addr, unsigned long size) if (mcfg->config[i].base_reserved) { printk(KERN_ERR PREFIX "MMCONFIG not in low 4GB of memory\n"); + kfree(pci_mmcfg_config); + pci_mmcfg_config_num = 0; return -ENODEV; } } |