summaryrefslogtreecommitdiffstats
path: root/hw/scsi-bus.c
diff options
context:
space:
mode:
authorMarkus Armbruster2010-02-18 17:25:24 +0100
committerMarkus Armbruster2010-03-16 16:58:32 +0100
commit1ecda02b24a13f501e747b8442934829d82698ae (patch)
tree8989913831ca3e48220b57f9104978015e39c5cb /hw/scsi-bus.c
parenterror: Don't abuse qemu_error() for non-error in scsi_hot_add() (diff)
downloadqemu-1ecda02b24a13f501e747b8442934829d82698ae.tar.gz
qemu-1ecda02b24a13f501e747b8442934829d82698ae.tar.xz
qemu-1ecda02b24a13f501e747b8442934829d82698ae.zip
error: Replace qemu_error() by error_report()
error_report() terminates the message with a newline. Strip it it from its arguments. This fixes a few error messages lacking a newline: net_handle_fd_param()'s "No file descriptor named %s found", and tap_open()'s "vnet_hdr=1 requested, but no kernel support for IFF_VNET_HDR available" (all three versions). There's one place that passes arguments without newlines intentionally: load_vmstate(). Fix it up.
Diffstat (limited to 'hw/scsi-bus.c')
-rw-r--r--hw/scsi-bus.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/scsi-bus.c b/hw/scsi-bus.c
index c41ce9b32a..383240bc07 100644
--- a/hw/scsi-bus.c
+++ b/hw/scsi-bus.c
@@ -41,7 +41,7 @@ static int scsi_qdev_init(DeviceState *qdev, DeviceInfo *base)
}
}
if (dev->id >= bus->ndev) {
- qemu_error("bad scsi device id: %d\n", dev->id);
+ error_report("bad scsi device id: %d", dev->id);
goto err;
}