coverity: #1425867

do not pass NULL pointer to chdir() Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

coverity: #1425867
0b01bd4d · Christian Brauner · ae65a0f7 · 0b01bd4d
Unverified Commit 0b01bd4d authored Dec 09, 2017 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

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

No files found.
--- a/src/lxc/attach.c
+++ b/src/lxc/attach.c
@@ -1186,9 +1186,11 @@ int lxc_attach(const char *name, const char *lxcpath,
 		new_cwd = options->initial_cwd;
 	else
 		new_cwd = cwd;
-	ret = chdir(new_cwd);
-	if (ret < 0)
-		WARN("Could not change directory to \"%s\".", new_cwd);
+	if (new_cwd) {
+		ret = chdir(new_cwd);
+		if (ret < 0)
+			WARN("Could not change directory to \"%s\"", new_cwd);
+	}
 	free(cwd);

 	/* Now create the real child process. */