From b7f28e0bd92eae7f03afc7f13ac19b5e64902ef8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20=C5=81yszczek?= Date: Tue, 22 Oct 2019 22:09:23 +0200 Subject: [PATCH] ipnetns: do not check netns NAME when -all is specified MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When `-all' argument is specified netns runs cmd on all namespaces and NAME is not used, but netns nevertheless checks if argv[1] is a valid namespace name ignoring the fact that argv[1] contains cmd and not NAME. This results in bug where user cannot specify absolute path to command. # ip -all netns exec /usr/bin/whoami Invalid netns name "/usr/bin/whoami" This forces user to have his command in PATH. Solution is simply to not validate argv[1] when `-all' argument is specified. Signed-off-by: Michał Łyszczek Signed-off-by: Stephen Hemminger --- ip/ipnetns.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ip/ipnetns.c b/ip/ipnetns.c index 20110ef0..fc58a04b 100644 --- a/ip/ipnetns.c +++ b/ip/ipnetns.c @@ -994,7 +994,7 @@ int do_netns(int argc, char **argv) return netns_list(0, NULL); } - if (argc > 1 && invalid_name(argv[1])) { + if (!do_all && argc > 1 && invalid_name(argv[1])) { fprintf(stderr, "Invalid netns name \"%s\"\n", argv[1]); exit(-1); }