diff options
| author | Laurent Vivier <lvivier@redhat.com> | 2022-10-21 11:09:08 +0200 |
|---|---|---|
| committer | Jason Wang <jasowang@redhat.com> | 2022-10-28 13:28:52 +0800 |
| commit | 21fccb2cbbacdc045f19605915e847de31ca9862 (patch) | |
| tree | e8c5b73130b820a6713e0c21ed57cea2b72caf92 /net/net.c | |
| parent | d63ef17bfcba1126df6bfcb7bca64c96ac4a8d99 (diff) | |
| download | focaccia-qemu-21fccb2cbbacdc045f19605915e847de31ca9862.tar.gz focaccia-qemu-21fccb2cbbacdc045f19605915e847de31ca9862.zip | |
net: simplify net_client_parse() error management
All net_client_parse() callers exit in case of error. Move exit(1) to net_client_parse() and remove error checking from the callers. Suggested-by: Markus Armbruster <armbru@redhat.com> Signed-off-by: Laurent Vivier <lvivier@redhat.com> Reviewed-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: David Gibson <david@gibson.dropbear.id.au> Acked-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Jason Wang <jasowang@redhat.com>
Diffstat (limited to 'net/net.c')
| -rw-r--r-- | net/net.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/net.c b/net/net.c index 5e788802b1..178257abfd 100644 --- a/net/net.c +++ b/net/net.c @@ -1583,13 +1583,11 @@ void net_init_clients(void) &error_fatal); } -int net_client_parse(QemuOptsList *opts_list, const char *optarg) +void net_client_parse(QemuOptsList *opts_list, const char *optarg) { if (!qemu_opts_parse_noisily(opts_list, optarg, true)) { - return -1; + exit(1); } - - return 0; } /* From FreeBSD */ |