diff options
author | Brian Gianforcaro <bgianf@serenityos.org> | 2021-07-17 06:07:02 -0700 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-07-17 16:22:59 +0200 |
commit | c0987453e62632d3102aea524b4b825959442aa0 (patch) | |
tree | e842952aca47fa5e6b6688cef9fe10acd83be0eb /Kernel/Thread.cpp | |
parent | d879709ec78d75e8d0ab80a0d67d0b778876a732 (diff) | |
download | serenity-c0987453e62632d3102aea524b4b825959442aa0.zip |
Kernel: Remove double RedBlackTree lookup in VM/Space region removal
We should never request a regions removal that we don't currently
own. We currently assert this everywhere else by all callers.
Instead lets just push the assert down into the RedBlackTree removal
and assume that we will always successfully remove the region.
Diffstat (limited to 'Kernel/Thread.cpp')
-rw-r--r-- | Kernel/Thread.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/Kernel/Thread.cpp b/Kernel/Thread.cpp index fc1fec3bdb..40abdba261 100644 --- a/Kernel/Thread.cpp +++ b/Kernel/Thread.cpp @@ -404,9 +404,7 @@ void Thread::exit(void* exit_value) [[maybe_unused]] auto rc = unlock_process_if_locked(unlock_count); if (m_thread_specific_range.has_value()) { auto* region = process().space().find_region_from_range(m_thread_specific_range.value()); - VERIFY(region); - if (!process().space().deallocate_region(*region)) - dbgln("Failed to unmap TLS range, exiting thread anyway."); + process().space().deallocate_region(*region); } die_if_needed(); } |