summaryrefslogtreecommitdiff
path: root/Kernel
diff options
context:
space:
mode:
authorAndreas Kling <kling@serenityos.org>2021-03-11 13:28:50 +0100
committerAndreas Kling <kling@serenityos.org>2021-03-11 14:21:49 +0100
commitb7b7a48c66b56850b3ee8b5493f9964ef96859d2 (patch)
treedd5d3929a2bdc404b97a7677f4d47b48ba6995dd /Kernel
parent08e0e2eb417d7c9a50b4309b4cdcef70b8554537 (diff)
downloadserenity-b7b7a48c66b56850b3ee8b5493f9964ef96859d2.zip
Kernel: Move process signal trampoline address into protected data
Diffstat (limited to 'Kernel')
-rw-r--r--Kernel/Process.h2
-rw-r--r--Kernel/Syscalls/execve.cpp3
-rw-r--r--Kernel/Syscalls/fork.cpp2
3 files changed, 4 insertions, 3 deletions
diff --git a/Kernel/Process.h b/Kernel/Process.h
index 6ff695547b..0b6be73611 100644
--- a/Kernel/Process.h
+++ b/Kernel/Process.h
@@ -118,6 +118,7 @@ protected:
bool m_has_execpromises { false };
u32 m_execpromises { 0 };
mode_t m_umask { 022 };
+ VirtualAddress m_signal_trampoline;
};
class ProcessBase : public ProtectedProcessBase {
@@ -544,7 +545,6 @@ private:
String m_name;
OwnPtr<Space> m_space;
- VirtualAddress m_signal_trampoline;
RefPtr<ProcessGroup> m_pg;
diff --git a/Kernel/Syscalls/execve.cpp b/Kernel/Syscalls/execve.cpp
index 803f610746..63fb04c5e6 100644
--- a/Kernel/Syscalls/execve.cpp
+++ b/Kernel/Syscalls/execve.cpp
@@ -530,7 +530,6 @@ KResult Process::do_exec(NonnullRefPtr<FileDescription> main_program_description
}
signal_trampoline_region.value()->set_syscall_region(true);
- m_signal_trampoline = signal_trampoline_region.value()->vaddr();
m_executable = main_program_description->custody();
m_arguments = arguments;
@@ -605,6 +604,8 @@ KResult Process::do_exec(NonnullRefPtr<FileDescription> main_program_description
m_execpromises = 0;
m_has_execpromises = false;
+ m_signal_trampoline = signal_trampoline_region.value()->vaddr();
+
// FIXME: PID/TID ISSUE
m_pid = new_main_thread->tid().value();
}
diff --git a/Kernel/Syscalls/fork.cpp b/Kernel/Syscalls/fork.cpp
index 4016206d18..102345800b 100644
--- a/Kernel/Syscalls/fork.cpp
+++ b/Kernel/Syscalls/fork.cpp
@@ -44,7 +44,6 @@ KResultOr<pid_t> Process::sys$fork(RegisterState& regs)
child->m_veil_state = m_veil_state;
child->m_unveiled_paths = m_unveiled_paths.deep_copy();
child->m_fds = m_fds;
- child->m_signal_trampoline = m_signal_trampoline;
child->m_pg = m_pg;
{
@@ -56,6 +55,7 @@ KResultOr<pid_t> Process::sys$fork(RegisterState& regs)
child->m_sid = m_sid;
child->m_extra_gids = m_extra_gids;
child->m_umask = m_umask;
+ child->m_signal_trampoline = m_signal_trampoline;
}
dbgln_if(FORK_DEBUG, "fork: child={}", child);