qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 1/1] qom: fix qdict visit in user_creatable_add_


From: Xiao Long Jiang
Subject: Re: [Qemu-devel] [PATCH 1/1] qom: fix qdict visit in user_creatable_add_type
Date: Fri, 23 Sep 2016 14:42:25 +0800
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.3.0

Hi Markus,

On 2016/9/22 下午7:56, Markus Armbruster wrote:
Xiao Long Jiang <address@hidden> writes:

This fixes a Qemu crash that introduced by commit ad73970
("qom: Wrap prop visit in visit_start_struct").

Not all of the callers of user_creatable_add_type always pass qdict,
so we should check qdict against NULL, but not raise an assertion.

The problem can be easily triggered by adding an iothread with:
   $ virsh iothreadadd <domain> <threadid>
In which the <threadid> is newly defined. The result of the command
will return an error and cause the running guest crash.

Cc: Eric Blake <address@hidden>
Cc: Markus Armbruster <address@hidden>
Signed-off-by: Xiao Long Jiang <address@hidden>
Reviewed-by: Dong Jia Shi <address@hidden>
Marc-André already posted a simpler fix:
Subject: [PATCH] qmp: fix object-add assert() without props
Message-Id: <address@hidden>

Would you be willing to test it?  Review would of course also be
appreciated.

Thanks!

Have tested that patch. It works fine on arch s390 system.




reply via email to

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