From 299528668c759f40b2cc78914e2ca2c832f82834 Mon Sep 17 00:00:00 2001 From: Luiz Capitulino Date: Thu, 12 Apr 2012 11:58:57 -0300 Subject: qemu-option: qemu_opts_validate(): use error_set() net_client_init() propagates the error up by calling qerror_report_err(), because its users expect QError semantics. Signed-off-by: Luiz Capitulino Reviewed-By: Laszlo Ersek --- net.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'net.c') diff --git a/net.c b/net.c index 1922d8a..f5d9cc7 100644 --- a/net.c +++ b/net.c @@ -1136,10 +1136,14 @@ int net_client_init(Monitor *mon, QemuOpts *opts, int is_netdev) for (i = 0; i < NET_CLIENT_TYPE_MAX; i++) { if (net_client_types[i].type != NULL && !strcmp(net_client_types[i].type, type)) { + Error *local_err = NULL; VLANState *vlan = NULL; int ret; - if (qemu_opts_validate(opts, &net_client_types[i].desc[0]) == -1) { + qemu_opts_validate(opts, &net_client_types[i].desc[0], &local_err); + if (error_is_set(&local_err)) { + qerror_report_err(local_err); + error_free(local_err); return -1; } -- cgit v1.1