diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2017-07-13 10:52:39 +1000 |
|---|---|---|
| committer | David Gibson <david@gibson.dropbear.id.au> | 2017-07-17 15:07:05 +1000 |
| commit | ba50822ff835c197563423b688afc6d0fe50f7d0 (patch) | |
| tree | db49ff48c6290ae9bab6236a3741560f93b89af8 | |
| parent | 765d1bdda5282cb38445c6dd82fa8aa0802cc904 (diff) | |
| download | qemu-ba50822ff835c197563423b688afc6d0fe50f7d0.zip qemu-ba50822ff835c197563423b688afc6d0fe50f7d0.tar.gz qemu-ba50822ff835c197563423b688afc6d0fe50f7d0.tar.bz2 | |
spapr: Abort on delete failure in spapr_drc_release()
We currently ignore errors from the object_property_del() in
spapr_drc_release(). But the only way that could fail is if the property
doesn't exist, in which case it's a bug that we're in spapr_drc_release()
at all. So change from ignoring to abort()ing on errors.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
| -rw-r--r-- | hw/ppc/spapr_drc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/ppc/spapr_drc.c b/hw/ppc/spapr_drc.c index 7f872a4..fd56140 100644 --- a/hw/ppc/spapr_drc.c +++ b/hw/ppc/spapr_drc.c @@ -367,7 +367,7 @@ static void spapr_drc_release(sPAPRDRConnector *drc) g_free(drc->fdt); drc->fdt = NULL; drc->fdt_start_offset = 0; - object_property_del(OBJECT(drc), "device", NULL); + object_property_del(OBJECT(drc), "device", &error_abort); drc->dev = NULL; } |