Get rid of done: label. If the child exits we still need to close

the pty, update utmp and restore the SELinux tty context.

diff --git a/src/exec.c b/src/exec.c
index 3853d347adbcad7b2c088e9ed3707e978f75969d..2d6d8605f470f599b27000f621082ccfaf643382 100644 (file)
--- 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)) {