From 5823dc5c682eb752e89fd8cc5be21bc4fae452a3 Mon Sep 17 00:00:00 2001 From: David Gibson Date: Wed, 28 Sep 2022 14:33:14 +1000 Subject: clang-tidy: Fix spurious null pointer warning in pasta_start_ns() clang-tidy isn't quite clever enough to figure out that getenv("SHELL") will return the same thing both times here, which makes it conclude that shell could be NULL, causing problems later. It's a bit ugly that we call getenv() twice in any case, so rework this in a way that clang-tidy can figure out shell won't be NULL. Signed-off-by: David Gibson Signed-off-by: Stefano Brivio --- pasta.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/pasta.c b/pasta.c index e233762..1dd8267 100644 --- a/pasta.c +++ b/pasta.c @@ -184,7 +184,7 @@ void pasta_start_ns(struct ctx *c, int argc, char *argv[]) struct pasta_setup_ns_arg arg = { .argv = argv, }; - char *shell = getenv("SHELL") ? getenv("SHELL") : "/bin/sh"; + char *shell = getenv("SHELL"); char *sh_argv[] = { shell, NULL }; char *bash_argv[] = { shell, "-l", NULL }; char ns_fn_stack[NS_FN_STACK_SIZE]; @@ -193,6 +193,9 @@ void pasta_start_ns(struct ctx *c, int argc, char *argv[]) if (!c->debug) c->quiet = 1; + if (!shell) + shell = "/bin/sh"; + if (argc == 0) { if (strstr(shell, "/bash")) { arg.argv = bash_argv; -- cgit v1.2.3