coverity: #1426126

Unchecked return value Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

coverity: #1426126
5b7f756a · Christian Brauner · fa456191 · 5b7f756a
Unverified Commit 5b7f756a authored Feb 08, 2018 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

start.c src/lxc/start.c +4 -1

No files found.
--- a/src/lxc/start.c
+++ b/src/lxc/start.c
@@ -1657,7 +1657,10 @@ static int lxc_spawn(struct lxc_handler *handler)
 		DEBUG("Preserved cgroup namespace via fd %d", ret);
 	}

-	snprintf(pidstr, 20, "%d", handler->pid);
+	ret = snprintf(pidstr, 20, "%d", handler->pid);
+	if (ret < 0 || ret >= 20)
+		goto out_delete_net;
+
 	if (setenv("LXC_PID", pidstr, 1))
 		SYSERROR("Failed to set environment variable: LXC_PID=%s.", pidstr);