From b925965294e8cf370a922ca0504c21877e748e70 Mon Sep 17 00:00:00 2001 From: Igor Ryzhov Date: Wed, 16 Apr 2014 17:43:07 +0400 Subject: net/net.c: Remove unnecessary semicolon Signed-off-by: Igor Ryzhov Reviewed-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- net/net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'net/net.c') diff --git a/net/net.c b/net/net.c index a4aadffc11..bc9ed6def0 100644 --- a/net/net.c +++ b/net/net.c @@ -473,7 +473,7 @@ ssize_t qemu_deliver_packet(NetClientState *sender, if (ret == 0) { nc->receive_disabled = 1; - }; + } return ret; } -- cgit v1.2.3 From 638fb14169ad96cf9bc0dd5f61460daaecee5bb1 Mon Sep 17 00:00:00 2001 From: Markus Armbruster Date: Thu, 24 Apr 2014 15:44:17 +0200 Subject: net: Make qmp_query_rx_filter() with name argument more obvious With a client name, the QMP command is specified to return a list of one element. This isn't locally obvious in the code. Make it so. Signed-off-by: Markus Armbruster Signed-off-by: Stefan Hajnoczi --- net/net.c | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'net/net.c') diff --git a/net/net.c b/net/net.c index bc9ed6def0..ccb354aee8 100644 --- a/net/net.c +++ b/net/net.c @@ -1066,6 +1066,10 @@ RxFilterInfoList *qmp_query_rx_filter(bool has_name, const char *name, " rx-filter querying", name); break; } + + if (has_name) { + break; + } } if (filter_list == NULL && !error_is_set(errp) && has_name) { -- cgit v1.2.3 From 9083da1d4c9dfff30d411f8c73ea494e9d78de1b Mon Sep 17 00:00:00 2001 From: Markus Armbruster Date: Thu, 24 Apr 2014 15:44:18 +0200 Subject: net: Don't use error_is_set() to suppress additional errors Using error_is_set(errp) that way can sweep programming errors under the carpet when we get called incorrectly with an error set. qmp_query_rx_filter() breaks its loop when it detects an error. It needs to set another error when the loop completes normally. Return right away instead of merely breaking the loop. Signed-off-by: Markus Armbruster Signed-off-by: Stefan Hajnoczi --- net/net.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'net/net.c') diff --git a/net/net.c b/net/net.c index ccb354aee8..9db4dba769 100644 --- a/net/net.c +++ b/net/net.c @@ -1045,7 +1045,7 @@ RxFilterInfoList *qmp_query_rx_filter(bool has_name, const char *name, if (nc->info->type != NET_CLIENT_OPTIONS_KIND_NIC) { if (has_name) { error_setg(errp, "net client(%s) isn't a NIC", name); - break; + return NULL; } continue; } @@ -1064,7 +1064,7 @@ RxFilterInfoList *qmp_query_rx_filter(bool has_name, const char *name, } else if (has_name) { error_setg(errp, "net client(%s) doesn't support" " rx-filter querying", name); - break; + return NULL; } if (has_name) { @@ -1072,7 +1072,7 @@ RxFilterInfoList *qmp_query_rx_filter(bool has_name, const char *name, } } - if (filter_list == NULL && !error_is_set(errp) && has_name) { + if (filter_list == NULL && has_name) { error_setg(errp, "invalid net client name: %s", name); } -- cgit v1.2.3