From 13a12f869bf2211c7039f9a601ac9e1076535663 Mon Sep 17 00:00:00 2001 From: Pankaj Gupta Date: Wed, 12 Mar 2014 22:24:27 +0530 Subject: tap: Avoid extra iterations while closing file fd Avoid iterations for fd 0, 1 & 2 when we are closing file fds in child process. Signed-off-by: Pankaj Gupta Signed-off-by: Stefan Hajnoczi --- net/tap.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) (limited to 'net') diff --git a/net/tap.c b/net/tap.c index 8847ce1..fc1b865 100644 --- a/net/tap.c +++ b/net/tap.c @@ -367,11 +367,8 @@ static int launch_script(const char *setup_script, const char *ifname, int fd) if (pid == 0) { int open_max = sysconf(_SC_OPEN_MAX), i; - for (i = 0; i < open_max; i++) { - if (i != STDIN_FILENO && - i != STDOUT_FILENO && - i != STDERR_FILENO && - i != fd) { + for (i = 3; i < open_max; i++) { + if (i != fd) { close(i); } } @@ -452,11 +449,8 @@ static int net_bridge_run_helper(const char *helper, const char *bridge) char br_buf[6+IFNAMSIZ] = {0}; char helper_cmd[PATH_MAX + sizeof(fd_buf) + sizeof(br_buf) + 15]; - for (i = 0; i < open_max; i++) { - if (i != STDIN_FILENO && - i != STDOUT_FILENO && - i != STDERR_FILENO && - i != sv[1]) { + for (i = 3; i < open_max; i++) { + if (i != sv[1]) { close(i); } } -- cgit v1.1 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') diff --git a/net/net.c b/net/net.c index a4aadff..bc9ed6d 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.1 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') diff --git a/net/net.c b/net/net.c index bc9ed6d..ccb354a 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.1 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') diff --git a/net/net.c b/net/net.c index ccb354a..9db4dba 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.1