diff options
author | Markus Armbruster | 2010-02-18 17:25:24 +0100 |
---|---|---|
committer | Markus Armbruster | 2010-03-16 16:58:32 +0100 |
commit | 1ecda02b24a13f501e747b8442934829d82698ae (patch) | |
tree | 8989913831ca3e48220b57f9104978015e39c5cb /net/tap.c | |
parent | error: Don't abuse qemu_error() for non-error in scsi_hot_add() (diff) | |
download | qemu-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 'net/tap.c')
-rw-r--r-- | net/tap.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -394,7 +394,7 @@ int net_init_tap(QemuOpts *opts, Monitor *mon, const char *name, VLANState *vlan qemu_opt_get(opts, "script") || qemu_opt_get(opts, "downscript") || qemu_opt_get(opts, "vnet_hdr")) { - qemu_error("ifname=, script=, downscript= and vnet_hdr= is invalid with fd=\n"); + error_report("ifname=, script=, downscript= and vnet_hdr= is invalid with fd="); return -1; } |