qemu-stable
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-stable] [PATCH for-2.3?] qom: Fix object_property_add_alias()


From: Gonglei
Subject: Re: [Qemu-stable] [PATCH for-2.3?] qom: Fix object_property_add_alias() with [*]
Date: Mon, 30 Mar 2015 09:43:43 +0800
User-agent: Mozilla/5.0 (Windows NT 6.1; rv:31.0) Gecko/20100101 Thunderbird/31.4.0

On 2015/3/30 1:19, Andreas Färber wrote:
> Commit 8074264 (qom: Add description field in ObjectProperty struct)
> introduced property descriptions and copied them for alias properties.
> 
> Instead of using the caller-supplied property name, use the returned
> property name for setting the description. This avoids an Error when
> setting a property description for a property with literal "[*]" that
> doesn't exist due to automatic property naming in object_property_add().
> 
> Cc: Gonglei <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Michael S. Tsirkin <address@hidden>
> Signed-off-by: Andreas Färber <address@hidden>
> ---
>  qom/object.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/qom/object.c b/qom/object.c
> index d167038..b8dff43 100644
> --- a/qom/object.c
> +++ b/qom/object.c
> @@ -1761,7 +1761,7 @@ void object_property_add_alias(Object *obj, const char 
> *name,
>      }
>      op->resolve = property_resolve_alias;
>  
> -    object_property_set_description(obj, name,
> +    object_property_set_description(obj, op->name,
>                                      target_prop->description,
>                                      &error_abort);
>  
Looks good to me. I think this is a candidate for 2.3 rc2.

Cc: qemu-stable <address@hidden>

Reviewed-by: Gonglei <address@hidden>

Regards,
-Gonglei




reply via email to

[Prev in Thread] Current Thread [Next in Thread]