ACPICA: Tables: Fix multiple ACPI_FREE()s around acpi_tb_add_table().
authorLv Zheng <lv.zheng@intel.com>
Fri, 4 Apr 2014 04:38:35 +0000 (12:38 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Sun, 20 Apr 2014 20:59:38 +0000 (22:59 +0200)
commit55df23f0d620c5194ecbd3b68ecdb2798778bf93
tree22d088acce0e908142f0d8bef2323849f192677f
parentdc156adf0d1b9d9ea54a86ca803ac5202ab139c7
ACPICA: Tables: Fix multiple ACPI_FREE()s around acpi_tb_add_table().

Currently there are following issues in acpi_tb_add_table():
Following logic is currently correct:
1. When a table is allocated in acpi_ex_load_op(), if a reloading happens,
   the allocated memory is freed by acpi_tb_add_table() and AE_OK is
   returned to the caller to avoid the caller to free it again.
Following logic is currently incorrect:
1. When a table is allocated in acpi_ex_load_op() or by the
   acpi_load_table() caller, if the table is already loaded, there will be
   twice ACPI_FREE() called for the same pointer when acpi_tb_add_table()
   returns AE_ALREADY_EXISTS.

This patch only fixes the above incorrect logic in acpi_tb_add_table():
1. Only invoke acpi_tb_delete_table() if AE_OK is going to be returned.
2. After doing so, we do not invoke ACPI_FREE() when returning AE_OK;

Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Bob Moore <robert.moore@intel.com>
[rjw: Subject]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/acpica/exconfig.c
drivers/acpi/acpica/tbinstal.c