diff options
| author | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2010-08-26 09:14:58 +0200 |
|---|---|---|
| committer | Matthew Garrett <mjg@redhat.com> | 2010-10-21 15:36:47 +0200 |
| commit | 64ed0ab8d060d28a7787de29c76178c2efc1dd65 (patch) | |
| tree | 0fb02e7d696f310886a9e57c01130322fc60e3e8 /drivers/platform | |
| parent | WMI: fix wmi_gtoa() to actully terminate the string (diff) | |
| download | linux-64ed0ab8d060d28a7787de29c76178c2efc1dd65.tar.xz linux-64ed0ab8d060d28a7787de29c76178c2efc1dd65.zip | |
WMI: do not leak memory in parse_wdg()
If we _WDG returned object that is not buffer we were forgetting
to free memory allocated for that object.
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Diffstat (limited to 'drivers/platform')
| -rw-r--r-- | drivers/platform/x86/wmi.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/platform/x86/wmi.c b/drivers/platform/x86/wmi.c index 88ce87c74e27..a24a4b52c919 100644 --- a/drivers/platform/x86/wmi.c +++ b/drivers/platform/x86/wmi.c @@ -825,8 +825,10 @@ static acpi_status parse_wdg(acpi_handle handle) obj = (union acpi_object *) out.pointer; - if (obj->type != ACPI_TYPE_BUFFER) - return AE_ERROR; + if (obj->type != ACPI_TYPE_BUFFER) { + status = AE_ERROR; + goto out_free_pointer; + } total = obj->buffer.length / sizeof(struct guid_block); |