From ab75dd2e306a92e491bc4fae9876c84d311a166c Mon Sep 17 00:00:00 2001 From: "Todd C. Miller" Date: Mon, 24 Oct 2011 11:14:20 -0400 Subject: [PATCH] Get rid of done: label. If the child exits we still need to close the pty, update utmp and restore the SELinux tty context. --- src/exec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/exec.c b/src/exec.c index 3853d347a..2d6d8605f 100644 --- a/src/exec.c +++ b/src/exec.c @@ -341,7 +341,7 @@ sudo_execve(struct command_details *details, struct command_status *cstat) n = handle_signals(signal_pipe[0], child, log_io, cstat); if (n == 0) { /* Child has exited, cstat is set, we are done. */ - goto done; + break; } if (n == -1) { /* Error reading signal_pipe[0], should not happen. */ @@ -405,7 +405,6 @@ sudo_execve(struct command_details *details, struct command_status *cstat) } #endif -done: efree(fdsr); efree(fdsw); while (!tq_empty(&sigfwd_list)) { -- 2.40.0