coverity: #1425818

Dereference after null check Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

coverity: #1425818
1175ca91 · Christian Brauner · c2e55c88 · 1175ca91
Unverified Commit 1175ca91 authored Jun 15, 2018 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 5 deletions

start.c src/lxc/start.c +3 -5

No files found.
--- a/src/lxc/start.c
+++ b/src/lxc/start.c
@@ -751,11 +751,9 @@ int lxc_init(const char *name, struct lxc_handler *handler)
 	TRACE("Set container state to \"STARTING\"");

 	/* Start of environment variable setup for hooks. */
-	if (name) {
-		ret = setenv("LXC_NAME", name, 1);
-		if (ret < 0)
-			SYSERROR("Failed to set environment variable: LXC_NAME=%s", name);
-	}
+	ret = setenv("LXC_NAME", name, 1);
+	if (ret < 0)
+		SYSERROR("Failed to set environment variable: LXC_NAME=%s", name);

 	if (conf->rcfile) {
 		ret = setenv("LXC_CONFIG_FILE", conf->rcfile, 1);