From 0b321648703e03ecd7aa098b7b57a479dc7c97d4 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Mon, 21 Oct 2024 11:53:42 +0200 Subject: 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. --- executor/common_linux.h | 1 - 1 file changed, 1 deletion(-) (limited to 'executor/common_linux.h') 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); -- cgit mrf-deployment