From d53e54f8bf1b3bb77ae35750eb465ed4389b5f2a Mon Sep 17 00:00:00 2001 From: Robin Burchell Date: Tue, 16 Jul 2019 15:06:34 +0200 Subject: [PATCH] SharedBuffer: fix nullptr dereference on seal If the buffer hadn't been used by all participants yet, there wouldn't be a region, which leads to bad things. --- Kernel/Process.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Kernel/Process.cpp b/Kernel/Process.cpp index 4c18348958..06e5e30418 100644 --- a/Kernel/Process.cpp +++ b/Kernel/Process.cpp @@ -2479,8 +2479,10 @@ public: LOCKER(m_refs.lock()); m_writable = false; for (auto& ref : m_refs.resource()) { - ref.region->set_writable(false); - MM.remap_region(*ref.region->page_directory(), *ref.region); + if (ref.region) { + ref.region->set_writable(false); + MM.remap_region(*ref.region->page_directory(), *ref.region); + } } }