[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v5 03/12] net: simplify net_client_parse() error management
From: |
Laurent Vivier |
Subject: |
[PATCH v5 03/12] net: simplify net_client_parse() error management |
Date: |
Mon, 27 Jun 2022 17:47:40 +0200 |
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>
---
include/net/net.h | 2 +-
net/net.c | 6 ++----
softmmu/vl.c | 12 +++---------
3 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/include/net/net.h b/include/net/net.h
index c53c64ac18c4..e755254443ea 100644
--- a/include/net/net.h
+++ b/include/net/net.h
@@ -214,7 +214,7 @@ extern NICInfo nd_table[MAX_NICS];
extern const char *host_net_devices[];
/* from net.c */
-int net_client_parse(QemuOptsList *opts_list, const char *str);
+void net_client_parse(QemuOptsList *opts_list, const char *str);
void show_netdevs(void);
void net_init_clients(void);
void net_check_clients(void);
diff --git a/net/net.c b/net/net.c
index 15958f881776..f056e8aebfb2 100644
--- a/net/net.c
+++ b/net/net.c
@@ -1579,13 +1579,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 */
diff --git a/softmmu/vl.c b/softmmu/vl.c
index c244e8afc4cc..f6461363d608 100644
--- a/softmmu/vl.c
+++ b/softmmu/vl.c
@@ -2794,21 +2794,15 @@ void qemu_init(int argc, char **argv, char **envp)
break;
case QEMU_OPTION_netdev:
default_net = 0;
- if (net_client_parse(qemu_find_opts("netdev"), optarg) == -1) {
- exit(1);
- }
+ net_client_parse(qemu_find_opts("netdev"), optarg);
break;
case QEMU_OPTION_nic:
default_net = 0;
- if (net_client_parse(qemu_find_opts("nic"), optarg) == -1) {
- exit(1);
- }
+ net_client_parse(qemu_find_opts("nic"), optarg);
break;
case QEMU_OPTION_net:
default_net = 0;
- if (net_client_parse(qemu_find_opts("net"), optarg) == -1) {
- exit(1);
- }
+ net_client_parse(qemu_find_opts("net"), optarg);
break;
#ifdef CONFIG_LIBISCSI
case QEMU_OPTION_iscsi:
--
2.36.1
- [PATCH v5 00/12] qapi: net: add unix socket type support to netdev backend, Laurent Vivier, 2022/06/27
- [PATCH v5 02/12] net: remove the @errp argument of net_client_inits(), Laurent Vivier, 2022/06/27
- [PATCH v5 01/12] net: introduce convert_host_port(), Laurent Vivier, 2022/06/27
- [PATCH v5 03/12] net: simplify net_client_parse() error management,
Laurent Vivier <=
- [PATCH v5 04/12] qapi: net: introduce a way to bypass qemu_opts_parse_noisily(), Laurent Vivier, 2022/06/27
- [PATCH v5 06/12] net: stream: Don't ignore EINVAL on netdev socket connection, Laurent Vivier, 2022/06/27
- [PATCH v5 05/12] qapi: net: add stream and dgram netdevs, Laurent Vivier, 2022/06/27
- [PATCH v5 07/12] net: stream: add unix socket, Laurent Vivier, 2022/06/27