diff options
| author | Idan Horowitz <idan.horowitz@gmail.com> | 2021-08-19 22:45:07 +0300 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2021-08-19 23:49:53 +0200 |
| commit | cf271183b4befeba7c96bd6ca25246b899e836bb (patch) | |
| tree | 0361e7378750c2c4e45e5e01e273d6fe416555af /Kernel/Syscalls/ptrace.cpp | |
| parent | 1259dc362376e8011d20d4d9e2fcc49d5b7f9dc9 (diff) | |
| download | serenity-cf271183b4befeba7c96bd6ca25246b899e836bb.zip | |
Kernel: Make Process::current() return a Process& instead of Process*
This has several benefits:
1) We no longer just blindly derefence a null pointer in various places
2) We will get nicer runtime error messages if the current process does
turn out to be null in the call location
3) GCC no longer complains about possible nullptr dereferences when
compiling without KUBSAN
Diffstat (limited to 'Kernel/Syscalls/ptrace.cpp')
| -rw-r--r-- | Kernel/Syscalls/ptrace.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Kernel/Syscalls/ptrace.cpp b/Kernel/Syscalls/ptrace.cpp index 32919cd7d5..f9e9ffed48 100644 --- a/Kernel/Syscalls/ptrace.cpp +++ b/Kernel/Syscalls/ptrace.cpp @@ -20,7 +20,7 @@ static KResultOr<u32> handle_ptrace(const Kernel::Syscall::SC_ptrace_params& par { ScopedSpinLock scheduler_lock(g_scheduler_lock); if (params.request == PT_TRACE_ME) { - if (Process::current()->tracer()) + if (Process::current().tracer()) return EBUSY; caller.set_wait_for_tracer_at_next_execve(true); |