diff options
author | Laszlo Ersek <lersek@redhat.com> | 2013-03-21 00:23:19 +0100 |
---|---|---|
committer | Anthony Liguori <aliguori@us.ibm.com> | 2013-04-04 19:23:08 -0500 |
commit | 23084327dc5c2f42e8a1c3e39ec565e344c7f52e (patch) | |
tree | 80045f2e756bbe9c9fc0394153b7c58024c92e83 /hw/acpi.c | |
parent | e980f2bf0a39cb524259bb70084e0f75b92c8f39 (diff) |
like acpi_table_install(), acpi_table_add() should propagate Errors
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Anthony Liguori <aliguori@us.ibm.com>
Message-id: 1363821803-3380-8-git-send-email-lersek@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'hw/acpi.c')
-rw-r--r-- | hw/acpi.c | 9 |
1 files changed, 2 insertions, 7 deletions
@@ -229,7 +229,7 @@ static void acpi_table_install(const char unsigned *blob, size_t bloblen, ACPI_TABLE_PFX_SIZE, acpi_payload_size); } -int acpi_table_add(const QemuOpts *opts) +void acpi_table_add(const QemuOpts *opts, Error **errp) { AcpiTableOptions *hdrs = NULL; Error *err = NULL; @@ -306,12 +306,7 @@ out: qapi_dealloc_visitor_cleanup(dv); } - if (err) { - fprintf(stderr, "%s\n", error_get_pretty(err)); - error_free(err); - return -1; - } - return 0; + error_propagate(errp, err); } static void acpi_notify_wakeup(Notifier *notifier, void *data) |