Merge pull request #17202 from FRRouting/mergify/bp/stable/10.1/pr-17198

Revert "lib: Attach stdout to child only if --log=stdout and stdout F… (backport #17198)
This commit is contained in:
Donatas Abraitis 2024-10-23 12:40:07 +03:00 committed by GitHub
commit be50cd1ee0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -1115,12 +1115,9 @@ static void frr_terminal_close(int isexit)
* don't redirect when stdout is set with --log stdout
*/
for (fd = 2; fd >= 0; fd--)
if (logging_to_stdout && isatty(fd) &&
fd == STDOUT_FILENO) {
/* Do nothing. */
} else {
if (isatty(fd) &&
(fd != STDOUT_FILENO || !logging_to_stdout))
dup2(nullfd, fd);
}
close(nullfd);
}
}
@ -1206,12 +1203,9 @@ void frr_run(struct event_loop *master)
* stdout
*/
for (fd = 2; fd >= 0; fd--)
if (logging_to_stdout && isatty(fd) &&
fd == STDOUT_FILENO) {
/* Do nothing. */
} else {
if (isatty(fd) &&
(fd != STDOUT_FILENO || !logging_to_stdout))
dup2(nullfd, fd);
}
close(nullfd);
}