| From b685f3b1744061aa9ad822548ba9c674de5be7c6 Mon Sep 17 00:00:00 2001 |
| From: Tomasz Nowicki <tomasz.nowicki@linaro.org> |
| Date: Mon, 10 Feb 2014 14:00:11 +0100 |
| Subject: ACPI / PCI: Fix memory leak in acpi_pci_irq_enable() |
| |
| From: Tomasz Nowicki <tomasz.nowicki@linaro.org> |
| |
| commit b685f3b1744061aa9ad822548ba9c674de5be7c6 upstream. |
| |
| acpi_pci_link_allocate_irq() can return negative gsi even if |
| entry != NULL. For that case we have a memory leak, so free |
| entry before returning from acpi_pci_irq_enable() for gsi < 0. |
| |
| Signed-off-by: Tomasz Nowicki <tomasz.nowicki@linaro.org> |
| [rjw: Subject and changelog] |
| Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/acpi/pci_irq.c | 1 + |
| 1 file changed, 1 insertion(+) |
| |
| --- a/drivers/acpi/pci_irq.c |
| +++ b/drivers/acpi/pci_irq.c |
| @@ -432,6 +432,7 @@ int acpi_pci_irq_enable(struct pci_dev * |
| pin_name(pin)); |
| } |
| |
| + kfree(entry); |
| return 0; |
| } |
| |