diff mbox

hw/acpi/aml-build: Fix memory leak

Message ID 1432538077-15712-1-git-send-email-zhaoshenglong@huawei.com
State New
Headers show

Commit Message

Shannon Zhao May 25, 2015, 7:14 a.m. UTC 
From: Shannon Zhao <shannon.zhao@linaro.org>

Signed-off-by: Shannon Zhao <zhaoshenglong@huawei.com>
Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org>
---
 hw/acpi/aml-build.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Igor Mammedov May 25, 2015, 1:50 p.m. UTC | #1 
On Mon, 25 May 2015 15:14:37 +0800
Shannon Zhao <zhaoshenglong@huawei.com> wrote:

> From: Shannon Zhao <shannon.zhao@linaro.org>
> 
> Signed-off-by: Shannon Zhao <zhaoshenglong@huawei.com>
> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>

> ---
>  hw/acpi/aml-build.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
> index 323b7bc..9efef70 100644
> --- a/hw/acpi/aml-build.c
> +++ b/hw/acpi/aml-build.c
> @@ -306,6 +306,7 @@ static void aml_free(gpointer data, gpointer user_data)
>  {
>      Aml *var = data;
>      build_free_array(var->buf);
> +    g_free(var);
>  }
>  
>  Aml *init_aml_allocator(void)
Igor Mammedov May 25, 2015, 1:52 p.m. UTC | #2 
On Mon, 25 May 2015 15:14:37 +0800
Shannon Zhao <zhaoshenglong@huawei.com> wrote:

CCing stable.
> From: Shannon Zhao <shannon.zhao@linaro.org>
> 
> Signed-off-by: Shannon Zhao <zhaoshenglong@huawei.com>
> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org>
> ---
>  hw/acpi/aml-build.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
> index 323b7bc..9efef70 100644
> --- a/hw/acpi/aml-build.c
> +++ b/hw/acpi/aml-build.c
> @@ -306,6 +306,7 @@ static void aml_free(gpointer data, gpointer user_data)
>  {
>      Aml *var = data;
>      build_free_array(var->buf);
> +    g_free(var);
>  }
>  
>  Aml *init_aml_allocator(void)
diff mbox

Patch

diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
index 323b7bc..9efef70 100644
--- a/hw/acpi/aml-build.c
+++ b/hw/acpi/aml-build.c
@@ -306,6 +306,7 @@  static void aml_free(gpointer data, gpointer user_data)
 {
     Aml *var = data;
     build_free_array(var->buf);
+    g_free(var);
 }
 
 Aml *init_aml_allocator(void)