mirror of
https://gitlab.freedesktop.org/wayland/weston.git
synced 2026-05-07 03:48:02 +02:00
weston: Set CLOEXEC on stdin
We don't want to leak this into apps launched from the panel.
stdout and stderr are left for now because some things launched
by weston - such as weston-keyboard - share weston's log by
printing to those fds.
I'm singling out stdin because it's never needed by a child process
and because it's value is 0, which makes it easy to accidentally
do bad things to (commit 5c611d933f)
Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Quentin Glidic <sardemff7+git@sardemff7.net>
This commit is contained in:
parent
5ef6bd7eee
commit
f0d39b2243
1 changed files with 5 additions and 0 deletions
|
|
@ -1800,6 +1800,11 @@ int main(int argc, char *argv[])
|
||||||
{ WESTON_OPTION_STRING, "config", 'c', &config_file },
|
{ WESTON_OPTION_STRING, "config", 'c', &config_file },
|
||||||
};
|
};
|
||||||
|
|
||||||
|
if (os_fd_set_cloexec(fileno(stdin))) {
|
||||||
|
printf("Unable to set stdin as close on exec().\n");
|
||||||
|
return EXIT_FAILURE;
|
||||||
|
}
|
||||||
|
|
||||||
cmdline = copy_command_line(argc, argv);
|
cmdline = copy_command_line(argc, argv);
|
||||||
parse_options(core_options, ARRAY_LENGTH(core_options), &argc, argv);
|
parse_options(core_options, ARRAY_LENGTH(core_options), &argc, argv);
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue