diff options
| author | Markus Armbruster <armbru@redhat.com> | 2020-07-07 18:05:51 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2020-07-10 15:18:08 +0200 |
| commit | 552d7f49eebdb1c84ab6dc7d93bd152fd885af31 (patch) | |
| tree | ba2ab90bff80624c63ee6c9bef8440e1e47d48f1 /hw/ppc/spapr_hcall.c | |
| parent | 90c69fb9fdd71f5e615145fd1dc43cdef61737b3 (diff) | |
| download | qemu-552d7f49eebdb1c84ab6dc7d93bd152fd885af31.zip qemu-552d7f49eebdb1c84ab6dc7d93bd152fd885af31.tar.gz qemu-552d7f49eebdb1c84ab6dc7d93bd152fd885af31.tar.bz2 | |
qom: Crash more nicely on object_property_get_link() failure
Pass &error_abort instead of NULL where the returned value is
dereferenced or asserted to be non-null. Drop a now redundant
assertion.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20200707160613.848843-24-armbru@redhat.com>
Diffstat (limited to 'hw/ppc/spapr_hcall.c')
| -rw-r--r-- | hw/ppc/spapr_hcall.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c index 0f54988..c1d0122 100644 --- a/hw/ppc/spapr_hcall.c +++ b/hw/ppc/spapr_hcall.c @@ -1655,7 +1655,8 @@ static void spapr_handle_transient_dev_before_cas(SpaprMachineState *spapr) continue; } drc = SPAPR_DR_CONNECTOR(object_property_get_link(drc_container, - prop->name, NULL)); + prop->name, + &error_abort)); if (spapr_drc_transient(drc)) { spapr_drc_reset(drc); |