diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2024-10-21 11:53:42 +0200 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2024-10-24 09:34:38 +0000 |
| commit | 0b321648703e03ecd7aa098b7b57a479dc7c97d4 (patch) | |
| tree | 2cb0ebd19541776ddc3946e9061b4c8657244d84 /executor | |
| parent | af2638df2c9e396d35a593598d4d30a078d855e7 (diff) | |
executor: don't create new session for sandbox process
It's unclear why we need a new session.
Sessions group process groups, but we don't use that.
Setsid also creates a new process group,
but we don't kill this process group,
so also unclear why this is needed.
Diffstat (limited to 'executor')
| -rw-r--r-- | executor/common_linux.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/executor/common_linux.h b/executor/common_linux.h index a61cd947e..54dbdc8b2 100644 --- a/executor/common_linux.h +++ b/executor/common_linux.h @@ -4044,7 +4044,6 @@ static void loop(); static void sandbox_common() { prctl(PR_SET_PDEATHSIG, SIGKILL, 0, 0, 0); - setsid(); #if SYZ_EXECUTOR || __NR_syz_init_net_socket || SYZ_DEVLINK_PCI || __NR_syz_socket_connect_nvme_tcp int netns = open("/proc/self/ns/net", O_RDONLY); |