execute: account for -o path option count

This always works fine... until your exec() fails and you try to go and free it, you've overwritten the allocator's metadata (and potentially other stuff) and it fails. Signed-off-by: Tycho Andersen <tycho@tycho.ws>

execute: account for -o path option count
15dbf0a0 · Tycho Andersen · Christian Brauner · 37b143df · 15dbf0a0
Unverified Commit 15dbf0a0 authored May 09, 2018 by Tycho Andersen Committed by Christian Brauner Aug 23, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

execute.c src/lxc/execute.c +3 -0

No files found.
--- a/src/lxc/execute.c
+++ b/src/lxc/execute.c
@@ -64,6 +64,9 @@ static int execute_start(struct lxc_handler *handler, void* data)
 	if (lxc_log_has_valid_level())
 		argc_add += 2;

+	if (current_config->logfd != -1 || lxc_log_fd != -1)
+		argc_add += 2;
+
 	argv = malloc((argc + argc_add) * sizeof(*argv));
 	if (!argv)
 		goto out1;