diff options
author | Markus Armbruster <armbru@redhat.com> | 2015-12-18 16:35:24 +0100 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2016-01-13 15:16:18 +0100 |
commit | 433672b0d5cfdd4acbf269e6aef079e162af5bad (patch) | |
tree | 36ace9e1e89177cb024e58f3057c3f1c48d9cde0 /qemu-nbd.c | |
parent | bf89e87427fb99b994eb0dfb710bb4b45785f733 (diff) | |
download | qemu-433672b0d5cfdd4acbf269e6aef079e162af5bad.zip qemu-433672b0d5cfdd4acbf269e6aef079e162af5bad.tar.gz qemu-433672b0d5cfdd4acbf269e6aef079e162af5bad.tar.bz2 |
error: Clean up errors with embedded newlines (again)
The arguments of error_report() should yield a short error string
without newlines.
A few places try to print additional help after the error message by
embedding newlines in the error string. That's nice, but let's do it
the right way. Commit 474c213 cleaned up some, but they keep coming
back. Offenders tracked down with the Coccinelle semantic patch from
commit 312fd5f.
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Pavel Fedin <p.fedin@samsung.com>
Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'qemu-nbd.c')
-rw-r--r-- | qemu-nbd.c | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -599,9 +599,8 @@ int main(int argc, char **argv) } if ((argc - optind) != 1) { - error_report("Invalid number of argument.\n" - "Try `%s --help' for more information.", - argv[0]); + error_report("Invalid number of arguments"); + error_printf("Try `%s --help' for more information.\n", argv[0]); exit(EXIT_FAILURE); } |