ACPI: watchdog: Put the watchdog action table after parsing

The mapped watchdog action table should be released after the
successfully parsing, and the failure path in the driver
initialization.

Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

diff --git a/drivers/acpi/acpi_watchdog.c b/drivers/acpi/acpi_watchdog.c
index 6e9ec6e3fe47d63ea9adf07e540ac2a223379a06..5c1e9ea43123e94de355cc4cc5fd906aab6fd8ea 100644 (file)
--- a/drivers/acpi/acpi_watchdog.c
+++ b/drivers/acpi/acpi_watchdog.c
@@ -73,6 +73,7 @@ static const struct acpi_table_wdat *acpi_watchdog_get_wdat(void)
        }
 
        if (acpi_watchdog_uses_rtc(wdat)) {
+               acpi_put_table((struct acpi_table_header *)wdat);
                pr_info("Skipping WDAT on this system because it uses RTC SRAM\n");
                return NULL;
        }
@@ -117,12 +118,12 @@ void __init acpi_watchdog_init(void)
 
        /* Watchdog disabled by BIOS */
        if (!(wdat->flags & ACPI_WDAT_ENABLED))
-               return;
+               goto fail_put_wdat;
 
        /* Skip legacy PCI WDT devices */
        if (wdat->pci_segment != 0xff || wdat->pci_bus != 0xff ||
            wdat->pci_device != 0xff || wdat->pci_function != 0xff)
-               return;
+               goto fail_put_wdat;
 
        INIT_LIST_HEAD(&resource_list);
 
@@ -188,4 +189,6 @@ void __init acpi_watchdog_init(void)
 
 fail_free_resource_list:
        resource_list_free(&resource_list);
+fail_put_wdat:
+       acpi_put_table((struct acpi_table_header *)wdat);
 }