error: Use error_report_err() where appropriate

Coccinelle semantic patch:

    @@
    expression E;
    @@
    -    error_report("%s", error_get_pretty(E));
    -    error_free(E);
    +    error_report_err(E);
    @@
    expression E, S;
    @@
    -    error_report("%s", error_get_pretty(E));
    +    error_report_err(E);
    (
         exit(S);
    |
         abort();
    )

Trivial manual touch-ups in block/sheepdog.c.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
This commit is contained in:
Markus Armbruster 2015-02-12 13:55:05 +01:00
parent 2ee2f1e415
commit 565f65d271
31 changed files with 79 additions and 127 deletions

View File

@ -1093,8 +1093,7 @@ static int ram_load(QEMUFile *f, void *opaque, int version_id)
ret = qemu_ram_resize(block->offset, length, &local_err); ret = qemu_ram_resize(block->offset, length, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
} }
break; break;

View File

@ -726,8 +726,7 @@ static coroutine_fn void reconnect_to_sdog(void *opaque)
s->fd = get_sheep_fd(s, &local_err); s->fd = get_sheep_fd(s, &local_err);
if (s->fd < 0) { if (s->fd < 0) {
DPRINTF("Wait for connection to be established\n"); DPRINTF("Wait for connection to be established\n");
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
co_aio_sleep_ns(bdrv_get_aio_context(s->bs), QEMU_CLOCK_REALTIME, co_aio_sleep_ns(bdrv_get_aio_context(s->bs), QEMU_CLOCK_REALTIME,
1000000000ULL); 1000000000ULL);
} }
@ -1283,8 +1282,7 @@ static int reload_inode(BDRVSheepdogState *s, uint32_t snapid, const char *tag)
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
return -EIO; return -EIO;
} }
@ -1292,8 +1290,7 @@ static int reload_inode(BDRVSheepdogState *s, uint32_t snapid, const char *tag)
ret = find_vdi_name(s, s->name, snapid, tag, &vid, false, &local_err); ret = find_vdi_name(s, s->name, snapid, tag, &vid, false, &local_err);
if (ret) { if (ret) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
goto out; goto out;
} }
@ -1785,8 +1782,7 @@ static void sd_close(BlockDriverState *bs)
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
return; return;
} }
@ -1838,8 +1834,7 @@ static int sd_truncate(BlockDriverState *bs, int64_t offset)
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
return fd; return fd;
} }
@ -1912,8 +1907,7 @@ static bool sd_delete(BDRVSheepdogState *s)
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
return false; return false;
} }
@ -1960,8 +1954,7 @@ static int sd_create_branch(BDRVSheepdogState *s)
deleted = sd_delete(s); deleted = sd_delete(s);
ret = do_sd_create(s, &vid, !deleted, &local_err); ret = do_sd_create(s, &vid, !deleted, &local_err);
if (ret) { if (ret) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
goto out; goto out;
} }
@ -1969,8 +1962,7 @@ static int sd_create_branch(BDRVSheepdogState *s)
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
ret = fd; ret = fd;
goto out; goto out;
} }
@ -2218,8 +2210,7 @@ static int sd_snapshot_create(BlockDriverState *bs, QEMUSnapshotInfo *sn_info)
/* refresh inode. */ /* refresh inode. */
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
ret = fd; ret = fd;
goto cleanup; goto cleanup;
} }
@ -2234,8 +2225,7 @@ static int sd_snapshot_create(BlockDriverState *bs, QEMUSnapshotInfo *sn_info)
ret = do_sd_create(s, &new_vid, 1, &local_err); ret = do_sd_create(s, &new_vid, 1, &local_err);
if (ret < 0) { if (ret < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
error_report("failed to create inode for snapshot. %s", error_report("failed to create inode for snapshot. %s",
strerror(errno)); strerror(errno));
goto cleanup; goto cleanup;
@ -2336,8 +2326,7 @@ static int sd_snapshot_list(BlockDriverState *bs, QEMUSnapshotInfo **psn_tab)
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
ret = fd; ret = fd;
goto out; goto out;
} }
@ -2366,8 +2355,7 @@ static int sd_snapshot_list(BlockDriverState *bs, QEMUSnapshotInfo **psn_tab)
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
ret = fd; ret = fd;
goto out; goto out;
} }
@ -2429,8 +2417,7 @@ static int do_load_save_vmstate(BDRVSheepdogState *s, uint8_t *data,
fd = connect_to_sdog(s, &local_err); fd = connect_to_sdog(s, &local_err);
if (fd < 0) { if (fd < 0) {
error_report("%s", error_get_pretty(local_err));; error_report_err(local_err);
error_free(local_err);
return fd; return fd;
} }

View File

@ -728,8 +728,7 @@ DriveInfo *drive_new(QemuOpts *all_opts, BlockInterfaceType block_default_type)
qemu_opt_rename(all_opts, opt_renames[i].from, opt_renames[i].to, qemu_opt_rename(all_opts, opt_renames[i].from, opt_renames[i].to,
&local_err); &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
return NULL; return NULL;
} }
} }
@ -767,8 +766,7 @@ DriveInfo *drive_new(QemuOpts *all_opts, BlockInterfaceType block_default_type)
&error_abort); &error_abort);
qemu_opts_absorb_qdict(legacy_opts, bs_opts, &local_err); qemu_opts_absorb_qdict(legacy_opts, bs_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
goto fail; goto fail;
} }
@ -983,8 +981,7 @@ DriveInfo *drive_new(QemuOpts *all_opts, BlockInterfaceType block_default_type)
bs_opts = NULL; bs_opts = NULL;
if (!blk) { if (!blk) {
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
goto fail; goto fail;
} else { } else {
@ -1978,8 +1975,7 @@ int do_drive_del(Monitor *mon, const QDict *qdict, QObject **ret_data)
aio_context_acquire(aio_context); aio_context_acquire(aio_context);
if (bdrv_op_is_blocked(bs, BLOCK_OP_TYPE_DRIVE_DEL, &local_err)) { if (bdrv_op_is_blocked(bs, BLOCK_OP_TYPE_DRIVE_DEL, &local_err)) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
aio_context_release(aio_context); aio_context_release(aio_context);
return -1; return -1;
} }

View File

@ -158,7 +158,7 @@ Exynos4210State *exynos4210_init(MemoryRegion *system_mem,
if (object_property_find(cpuobj, "has_el3", NULL)) { if (object_property_find(cpuobj, "has_el3", NULL)) {
object_property_set_bool(cpuobj, false, "has_el3", &err); object_property_set_bool(cpuobj, false, "has_el3", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }
@ -168,7 +168,7 @@ Exynos4210State *exynos4210_init(MemoryRegion *system_mem,
"reset-cbar", &error_abort); "reset-cbar", &error_abort);
object_property_set_bool(cpuobj, true, "realized", &err); object_property_set_bool(cpuobj, true, "realized", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }

View File

@ -248,7 +248,7 @@ static void calxeda_init(MachineState *machine, enum cxmachines machine_id)
if (object_property_find(cpuobj, "has_el3", NULL)) { if (object_property_find(cpuobj, "has_el3", NULL)) {
object_property_set_bool(cpuobj, false, "has_el3", &err); object_property_set_bool(cpuobj, false, "has_el3", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }
@ -259,7 +259,7 @@ static void calxeda_init(MachineState *machine, enum cxmachines machine_id)
} }
object_property_set_bool(cpuobj, true, "realized", &err); object_property_set_bool(cpuobj, true, "realized", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
cpu_irq[n] = qdev_get_gpio_in(DEVICE(cpu), ARM_CPU_IRQ); cpu_irq[n] = qdev_get_gpio_in(DEVICE(cpu), ARM_CPU_IRQ);

View File

@ -500,14 +500,14 @@ static void integratorcp_init(MachineState *machine)
if (object_property_find(cpuobj, "has_el3", NULL)) { if (object_property_find(cpuobj, "has_el3", NULL)) {
object_property_set_bool(cpuobj, false, "has_el3", &err); object_property_set_bool(cpuobj, false, "has_el3", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }
object_property_set_bool(cpuobj, true, "realized", &err); object_property_set_bool(cpuobj, true, "realized", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }

View File

@ -108,7 +108,7 @@ static void realview_init(MachineState *machine,
if (object_property_find(cpuobj, "has_el3", NULL)) { if (object_property_find(cpuobj, "has_el3", NULL)) {
object_property_set_bool(cpuobj, false, "has_el3", &err); object_property_set_bool(cpuobj, false, "has_el3", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }
@ -116,14 +116,14 @@ static void realview_init(MachineState *machine,
if (is_pb && is_mpcore) { if (is_pb && is_mpcore) {
object_property_set_int(cpuobj, periphbase, "reset-cbar", &err); object_property_set_int(cpuobj, periphbase, "reset-cbar", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }
object_property_set_bool(cpuobj, true, "realized", &err); object_property_set_bool(cpuobj, true, "realized", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }

View File

@ -213,14 +213,14 @@ static void versatile_init(MachineState *machine, int board_id)
if (object_property_find(cpuobj, "has_el3", NULL)) { if (object_property_find(cpuobj, "has_el3", NULL)) {
object_property_set_bool(cpuobj, false, "has_el3", &err); object_property_set_bool(cpuobj, false, "has_el3", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }
object_property_set_bool(cpuobj, true, "realized", &err); object_property_set_bool(cpuobj, true, "realized", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }

View File

@ -223,7 +223,7 @@ static void init_cpus(const char *cpu_model, const char *privdev,
} }
object_property_set_bool(cpuobj, true, "realized", &err); object_property_set_bool(cpuobj, true, "realized", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }

View File

@ -133,25 +133,25 @@ static void zynq_init(MachineState *machine)
if (object_property_find(OBJECT(cpu), "has_el3", NULL)) { if (object_property_find(OBJECT(cpu), "has_el3", NULL)) {
object_property_set_bool(OBJECT(cpu), false, "has_el3", &err); object_property_set_bool(OBJECT(cpu), false, "has_el3", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
} }
object_property_set_int(OBJECT(cpu), ZYNQ_BOARD_MIDR, "midr", &err); object_property_set_int(OBJECT(cpu), ZYNQ_BOARD_MIDR, "midr", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
object_property_set_int(OBJECT(cpu), MPCORE_PERIPHBASE, "reset-cbar", &err); object_property_set_int(OBJECT(cpu), MPCORE_PERIPHBASE, "reset-cbar", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }
object_property_set_bool(OBJECT(cpu), true, "realized", &err); object_property_set_bool(OBJECT(cpu), true, "realized", &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
exit(1); exit(1);
} }

View File

@ -858,8 +858,7 @@ static void virtio_blk_migration_state_changed(Notifier *notifier, void *data)
virtio_blk_data_plane_create(VIRTIO_DEVICE(s), &s->conf, virtio_blk_data_plane_create(VIRTIO_DEVICE(s), &s->conf,
&s->dataplane, &err); &s->dataplane, &err);
if (err != NULL) { if (err != NULL) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
} }
} }
} }

View File

@ -906,8 +906,7 @@ SerialState *serial_init(int base, qemu_irq irq, int baudbase,
s->chr = chr; s->chr = chr;
serial_realize_core(s, &err); serial_realize_core(s, &err);
if (err != NULL) { if (err != NULL) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
exit(1); exit(1);
} }
@ -970,8 +969,7 @@ SerialState *serial_mm_init(MemoryRegion *address_space,
serial_realize_core(s, &err); serial_realize_core(s, &err);
if (err != NULL) { if (err != NULL) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
exit(1); exit(1);
} }
vmstate_register(NULL, base, &vmstate_serial, s); vmstate_register(NULL, base, &vmstate_serial, s);

View File

@ -953,8 +953,7 @@ static void assigned_dev_update_irq_routing(PCIDevice *dev)
r = assign_intx(assigned_dev, &err); r = assign_intx(assigned_dev, &err);
if (r < 0) { if (r < 0) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
err = NULL; err = NULL;
qdev_unplug(&dev->qdev, &err); qdev_unplug(&dev->qdev, &err);
assert(!err); assert(!err);
@ -1010,8 +1009,7 @@ static void assigned_dev_update_msi(PCIDevice *pci_dev)
assign_intx(assigned_dev, &local_err); assign_intx(assigned_dev, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
} }
} }
@ -1158,8 +1156,7 @@ static void assigned_dev_update_msix(PCIDevice *pci_dev)
assign_intx(assigned_dev, &local_err); assign_intx(assigned_dev, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
} }
} }

View File

@ -414,7 +414,7 @@ void pc_cmos_init(ram_addr_t ram_size, ram_addr_t above_4g_mem_size,
set_boot_dev(s, boot_device, &local_err); set_boot_dev(s, boot_device, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
@ -1040,8 +1040,7 @@ void pc_cpus_init(const char *cpu_model, DeviceState *icc_bridge)
cpu = pc_new_cpu(cpu_model, x86_cpu_apic_id_from_index(i), cpu = pc_new_cpu(cpu_model, x86_cpu_apic_id_from_index(i),
icc_bridge, &error); icc_bridge, &error);
if (error) { if (error) {
error_report("%s", error_get_pretty(error)); error_report_err(error);
error_free(error);
exit(1); exit(1);
} }
} }

View File

@ -908,7 +908,7 @@ void smbios_entry_add(QemuOpts *opts)
qemu_opts_validate(opts, qemu_smbios_file_opts, &local_err); qemu_opts_validate(opts, qemu_smbios_file_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
@ -994,7 +994,7 @@ void smbios_entry_add(QemuOpts *opts)
case 0: case 0:
qemu_opts_validate(opts, qemu_smbios_type0_opts, &local_err); qemu_opts_validate(opts, qemu_smbios_type0_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
save_opt(&type0.vendor, opts, "vendor"); save_opt(&type0.vendor, opts, "vendor");
@ -1014,7 +1014,7 @@ void smbios_entry_add(QemuOpts *opts)
case 1: case 1:
qemu_opts_validate(opts, qemu_smbios_type1_opts, &local_err); qemu_opts_validate(opts, qemu_smbios_type1_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
save_opt(&type1.manufacturer, opts, "manufacturer"); save_opt(&type1.manufacturer, opts, "manufacturer");
@ -1036,7 +1036,7 @@ void smbios_entry_add(QemuOpts *opts)
case 2: case 2:
qemu_opts_validate(opts, qemu_smbios_type2_opts, &local_err); qemu_opts_validate(opts, qemu_smbios_type2_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
save_opt(&type2.manufacturer, opts, "manufacturer"); save_opt(&type2.manufacturer, opts, "manufacturer");
@ -1049,7 +1049,7 @@ void smbios_entry_add(QemuOpts *opts)
case 3: case 3:
qemu_opts_validate(opts, qemu_smbios_type3_opts, &local_err); qemu_opts_validate(opts, qemu_smbios_type3_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
save_opt(&type3.manufacturer, opts, "manufacturer"); save_opt(&type3.manufacturer, opts, "manufacturer");
@ -1061,7 +1061,7 @@ void smbios_entry_add(QemuOpts *opts)
case 4: case 4:
qemu_opts_validate(opts, qemu_smbios_type4_opts, &local_err); qemu_opts_validate(opts, qemu_smbios_type4_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
save_opt(&type4.sock_pfx, opts, "sock_pfx"); save_opt(&type4.sock_pfx, opts, "sock_pfx");
@ -1074,7 +1074,7 @@ void smbios_entry_add(QemuOpts *opts)
case 17: case 17:
qemu_opts_validate(opts, qemu_smbios_type17_opts, &local_err); qemu_opts_validate(opts, qemu_smbios_type17_opts, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
save_opt(&type17.loc_pfx, opts, "loc_pfx"); save_opt(&type17.loc_pfx, opts, "loc_pfx");

View File

@ -172,8 +172,7 @@ static int ide_dev_initfn(IDEDevice *dev, IDEDriveKind kind)
if (kind != IDE_CD) { if (kind != IDE_CD) {
blkconf_geometry(&dev->conf, &dev->chs_trans, 65536, 16, 255, &err); blkconf_geometry(&dev->conf, &dev->chs_trans, 65536, 16, 255, &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
return -1; return -1;
} }
} }

View File

@ -132,8 +132,7 @@ static int scsi_hot_add(Monitor *mon, DeviceState *adapter,
dinfo->unit, false, -1, NULL, dinfo->unit, false, -1, NULL,
&local_err); &local_err);
if (!scsidev) { if (!scsidev) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
return -1; return -1;
} }
dinfo->unit = scsidev->id; dinfo->unit = scsidev->id;

View File

@ -2038,8 +2038,7 @@ int pci_add_capability(PCIDevice *pdev, uint8_t cap_id,
ret = pci_add_capability2(pdev, cap_id, offset, size, &local_err); ret = pci_add_capability2(pdev, cap_id, offset, size, &local_err);
if (local_err) { if (local_err) {
assert(ret < 0); assert(ret < 0);
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} else { } else {
/* success implies a positive offset in config space */ /* success implies a positive offset in config space */
assert(ret > 0); assert(ret > 0);

View File

@ -1399,8 +1399,7 @@ static USBDevice *usb_net_init(USBBus *bus, const char *cmdline)
idx = net_client_init(opts, 0, &local_err); idx = net_client_init(opts, 0, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
return NULL; return NULL;
} }

View File

@ -878,8 +878,7 @@ static int usb_host_open(USBHostDevice *s, libusb_device *dev)
usb_device_attach(udev, &local_err); usb_device_attach(udev, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
goto fail; goto fail;
} }

View File

@ -1273,8 +1273,7 @@ static void usbredir_do_attach(void *opaque)
usb_device_attach(&dev->dev, &local_err); usb_device_attach(&dev->dev, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
WARNING("rejecting device due to speed mismatch\n"); WARNING("rejecting device due to speed mismatch\n");
usbredir_reject_device(dev); usbredir_reject_device(dev);
} }

View File

@ -3753,8 +3753,7 @@ CharDriverState *qemu_chr_new(const char *label, const char *filename, void (*in
chr = qemu_chr_new_from_opts(opts, init, &err); chr = qemu_chr_new_from_opts(opts, init, &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
} }
if (chr && qemu_opt_get_bool(opts, "mux", 0)) { if (chr && qemu_opt_get_bool(opts, "mux", 0)) {
qemu_chr_fe_claim_no_fail(chr); qemu_chr_fe_claim_no_fail(chr);

View File

@ -2004,8 +2004,7 @@ static ImageInfoList *collect_image_info_list(const char *filename,
bdrv_query_image_info(bs, &info, &err); bdrv_query_image_info(bs, &info, &err);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
blk_unref(blk); blk_unref(blk);
goto err; goto err;
} }
@ -3053,8 +3052,7 @@ int main(int argc, char **argv)
qemu_init_exec_dir(argv[0]); qemu_init_exec_dir(argv[0]);
if (qemu_init_main_loop(&local_error)) { if (qemu_init_main_loop(&local_error)) {
error_report("%s", error_get_pretty(local_error)); error_report_err(local_error);
error_free(local_error);
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
} }

View File

@ -464,8 +464,7 @@ int main(int argc, char **argv)
} }
if (qemu_init_main_loop(&local_error)) { if (qemu_init_main_loop(&local_error)) {
error_report("%s", error_get_pretty(local_error)); error_report_err(local_error);
error_free(local_error);
exit(1); exit(1);
} }

View File

@ -228,8 +228,7 @@ static int tcp_socket_incoming(const char *address, uint16_t port)
int fd = inet_listen(address_and_port, NULL, 0, SOCK_STREAM, 0, &local_err); int fd = inet_listen(address_and_port, NULL, 0, SOCK_STREAM, 0, &local_err);
if (local_err != NULL) { if (local_err != NULL) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
return fd; return fd;
} }
@ -240,8 +239,7 @@ static int unix_socket_incoming(const char *path)
int fd = unix_listen(path, NULL, 0, &local_err); int fd = unix_listen(path, NULL, 0, &local_err);
if (local_err != NULL) { if (local_err != NULL) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
return fd; return fd;
} }
@ -252,8 +250,7 @@ static int unix_socket_outgoing(const char *path)
int fd = unix_connect(path, &local_err); int fd = unix_connect(path, &local_err);
if (local_err != NULL) { if (local_err != NULL) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
return fd; return fd;
} }
@ -681,8 +678,7 @@ int main(int argc, char **argv)
} }
if (qemu_init_main_loop(&local_err)) { if (qemu_init_main_loop(&local_err)) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
} }
bdrv_init(); bdrv_init();

View File

@ -71,8 +71,7 @@ CPUState *cpu_generic_init(const char *typename, const char *cpu_model)
out: out:
if (err != NULL) { if (err != NULL) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
object_unref(OBJECT(cpu)); object_unref(OBJECT(cpu));
return NULL; return NULL;
} }

View File

@ -2163,8 +2163,7 @@ X86CPU *cpu_x86_init(const char *cpu_model)
out: out:
if (error) { if (error) {
error_report("%s", error_get_pretty(error)); error_report_err(error);
error_free(error);
if (cpu != NULL) { if (cpu != NULL) {
object_unref(OBJECT(cpu)); object_unref(OBJECT(cpu));
cpu = NULL; cpu = NULL;

View File

@ -111,8 +111,7 @@ static int cpu_sparc_register(SPARCCPU *cpu, const char *cpu_model)
cc->parse_features(CPU(cpu), featurestr, &err); cc->parse_features(CPU(cpu), featurestr, &err);
g_free(s); g_free(s);
if (err) { if (err) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
error_free(err);
return -1; return -1;
} }

View File

@ -41,7 +41,7 @@ void error_set(Error **errp, ErrorClass err_class, const char *fmt, ...)
err->err_class = err_class; err->err_class = err_class;
if (errp == &error_abort) { if (errp == &error_abort) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
abort(); abort();
} }
@ -77,7 +77,7 @@ void error_set_errno(Error **errp, int os_errno, ErrorClass err_class,
err->err_class = err_class; err->err_class = err_class;
if (errp == &error_abort) { if (errp == &error_abort) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
abort(); abort();
} }
@ -122,7 +122,7 @@ void error_set_win32(Error **errp, int win32_err, ErrorClass err_class,
err->err_class = err_class; err->err_class = err_class;
if (errp == &error_abort) { if (errp == &error_abort) {
error_report("%s", error_get_pretty(err)); error_report_err(err);
abort(); abort();
} }
@ -169,7 +169,7 @@ void error_free(Error *err)
void error_propagate(Error **dst_errp, Error *local_err) void error_propagate(Error **dst_errp, Error *local_err)
{ {
if (local_err && dst_errp == &error_abort) { if (local_err && dst_errp == &error_abort) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
abort(); abort();
} else if (dst_errp && !*dst_errp) { } else if (dst_errp && !*dst_errp) {
*dst_errp = local_err; *dst_errp = local_err;

View File

@ -32,8 +32,7 @@ QemuOptsList *qemu_find_opts(const char *group)
ret = find_list(vm_config_groups, group, &local_err); ret = find_list(vm_config_groups, group, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
} }
return ret; return ret;
@ -314,8 +313,7 @@ int qemu_config_parse(FILE *fp, QemuOptsList **lists, const char *fname)
/* group with id */ /* group with id */
list = find_list(lists, group, &local_err); list = find_list(lists, group, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
goto out; goto out;
} }
opts = qemu_opts_create(list, id, 1, NULL); opts = qemu_opts_create(list, id, 1, NULL);
@ -325,8 +323,7 @@ int qemu_config_parse(FILE *fp, QemuOptsList **lists, const char *fname)
/* group without id */ /* group without id */
list = find_list(lists, group, &local_err); list = find_list(lists, group, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
goto out; goto out;
} }
opts = qemu_opts_create(list, NULL, 0, &error_abort); opts = qemu_opts_create(list, NULL, 0, &error_abort);

15
vl.c
View File

@ -2137,8 +2137,7 @@ static int chardev_init_func(QemuOpts *opts, void *opaque)
qemu_chr_new_from_opts(opts, NULL, &local_err); qemu_chr_new_from_opts(opts, NULL, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
return -1; return -1;
} }
return 0; return 0;
@ -2218,8 +2217,7 @@ static void monitor_parse(const char *optarg, const char *mode, bool pretty)
opts = qemu_opts_create(qemu_find_opts("mon"), label, 1, &local_err); opts = qemu_opts_create(qemu_find_opts("mon"), label, 1, &local_err);
if (!opts) { if (!opts) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
exit(1); exit(1);
} }
qemu_opt_set(opts, "mode", mode); qemu_opt_set(opts, "mode", mode);
@ -3766,7 +3764,7 @@ int main(int argc, char **argv, char **envp)
os_daemonize(); os_daemonize();
if (qemu_init_main_loop(&main_loop_err)) { if (qemu_init_main_loop(&main_loop_err)) {
error_report("%s", error_get_pretty(main_loop_err)); error_report_err(main_loop_err);
exit(1); exit(1);
} }
@ -4033,8 +4031,7 @@ int main(int argc, char **argv, char **envp)
Error *local_err = NULL; Error *local_err = NULL;
qtest_init(qtest_chrdev, qtest_log, &local_err); qtest_init(qtest_chrdev, qtest_log, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
error_free(local_err);
exit(1); exit(1);
} }
} }
@ -4056,7 +4053,7 @@ int main(int argc, char **argv, char **envp)
if (order) { if (order) {
validate_bootdevices(order, &local_err); validate_bootdevices(order, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
boot_order = order; boot_order = order;
@ -4066,7 +4063,7 @@ int main(int argc, char **argv, char **envp)
if (once) { if (once) {
validate_bootdevices(once, &local_err); validate_bootdevices(once, &local_err);
if (local_err) { if (local_err) {
error_report("%s", error_get_pretty(local_err)); error_report_err(local_err);
exit(1); exit(1);
} }
normal_boot_order = g_strdup(boot_order); normal_boot_order = g_strdup(boot_order);