[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] nvdimm acpi: fix g_array_free() with NULL point
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH] nvdimm acpi: fix g_array_free() with NULL pointer |
Date: |
Wed, 11 Jan 2017 17:08:17 +0100 |
On Wed, 11 Jan 2017 09:44:43 +0000
Stefan Hajnoczi <address@hidden> wrote:
> Unlike g_free(), g_array_free() does not accept a NULL pointer argument.
> The following error is logged when an nvdimm device is realized:
>
> GLib-CRITICAL **: g_array_free: assertion 'array' failed
>
> Cc: Xiao Guangrong <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
> ---
> hw/acpi/nvdimm.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/hw/acpi/nvdimm.c b/hw/acpi/nvdimm.c
> index 8e7d6ec..8f0a484 100644
> --- a/hw/acpi/nvdimm.c
> +++ b/hw/acpi/nvdimm.c
> @@ -375,7 +375,9 @@ static void nvdimm_init_fit_buffer(NvdimmFitBuffer
> *fit_buf)
>
> static void nvdimm_build_fit_buffer(NvdimmFitBuffer *fit_buf)
> {
> - g_array_free(fit_buf->fit, true);
> + if (fit_buf->fit) {
> + g_array_free(fit_buf->fit, true);
> + }
> fit_buf->fit = nvdimm_build_device_structure();
> fit_buf->dirty = true;
> }