[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.0] qom: Fix crash with qom-list and link p
From: |
Marcel Apfelbaum |
Subject: |
Re: [Qemu-devel] [PATCH for-2.0] qom: Fix crash with qom-list and link properties |
Date: |
Thu, 10 Apr 2014 22:08:10 +0300 |
On Thu, 2014-04-10 at 14:47 -0400, Cole Robinson wrote:
> Commit 9561fda8d90e176bef598ba87c42a1bd6ad03ef7 changed the type of
> 'opaque' for link properties, but missed updating this call site.
> Reproducer:
>
> ./x86_64-softmmu/qemu-system-x86_64 -qmp unix:./qmp.sock,server &
> ./scripts/qmp/qmp-shell ./qmp.sock
> (QEMU) qom-list path=//machine/i440fx/pci.0/child[2]
>
> Reported-by: Marcin Gibuła <address@hidden>
> Signed-off-by: Cole Robinson <address@hidden>
> ---
> qom/object.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/qom/object.c b/qom/object.c
> index f4de619..9a730e7 100644
> --- a/qom/object.c
> +++ b/qom/object.c
> @@ -1225,7 +1225,8 @@ Object *object_resolve_path_component(Object *parent,
> const gchar *part)
> }
>
> if (object_property_is_link(prop)) {
> - return *(Object **)prop->opaque;
> + LinkProperty *lprop = prop->opaque;
> + return *lprop->child;
Reviewed-by: Marcel Apfelbaum <address@hidden>
You may want another review from maintainers :), but I think
the fix is fine.
Thanks,
Marcel
> } else if (object_property_is_child(prop)) {
> return prop->opaque;
> } else {