[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RESEND 2/2] hw/machine: Free old values of strin
From: |
Amos Kong |
Subject: |
Re: [Qemu-devel] [PATCH RESEND 2/2] hw/machine: Free old values of string properties |
Date: |
Thu, 7 Aug 2014 10:24:48 +0800 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Aug 06, 2014 at 03:18:21PM -0300, Eduardo Habkost wrote:
> Reviewed-by: Markus Armbruster <address@hidden>
> Reviewed-by: Marcel Apfelbaum <address@hidden>
> Signed-off-by: Eduardo Habkost <address@hidden>
Reviewed-by: Amos Kong <address@hidden>
> ---
> hw/core/machine.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/hw/core/machine.c b/hw/core/machine.c
> index 7a66c57..d145aca 100644
> --- a/hw/core/machine.c
> +++ b/hw/core/machine.c
> @@ -24,6 +24,7 @@ static void machine_set_accel(Object *obj, const char
> *value, Error **errp)
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->accel);
> ms->accel = g_strdup(value);
> }
>
> @@ -79,6 +80,7 @@ static void machine_set_kernel(Object *obj, const char
> *value, Error **errp)
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->kernel_filename);
> ms->kernel_filename = g_strdup(value);
> }
>
> @@ -93,6 +95,7 @@ static void machine_set_initrd(Object *obj, const char
> *value, Error **errp)
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->initrd_filename);
> ms->initrd_filename = g_strdup(value);
> }
>
> @@ -107,6 +110,7 @@ static void machine_set_append(Object *obj, const char
> *value, Error **errp)
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->kernel_cmdline);
> ms->kernel_cmdline = g_strdup(value);
> }
>
> @@ -121,6 +125,7 @@ static void machine_set_dtb(Object *obj, const char
> *value, Error **errp)
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->dtb);
> ms->dtb = g_strdup(value);
> }
>
> @@ -135,6 +140,7 @@ static void machine_set_dumpdtb(Object *obj, const char
> *value, Error **errp)
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->dumpdtb);
> ms->dumpdtb = g_strdup(value);
> }
>
> @@ -176,6 +182,7 @@ static void machine_set_dt_compatible(Object *obj, const
> char *value, Error **er
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->dt_compatible);
> ms->dt_compatible = g_strdup(value);
> }
>
> @@ -232,6 +239,7 @@ static void machine_set_firmware(Object *obj, const char
> *value, Error **errp)
> {
> MachineState *ms = MACHINE(obj);
>
> + g_free(ms->firmware);
> ms->firmware = g_strdup(value);
> }
>
> --
> 1.9.3
>
--
Amos.