From 6103e374cc57bac0f9bb827259f4c16dce76e2e9 Mon Sep 17 00:00:00 2001
From: Andreas Kling <awesomekling@gmail.com>
Date: Sun, 10 Feb 2019 12:42:54 +0100
Subject: [PATCH] Kernel: Drop the "helper" region after ELF load is finished.

The individual segment mappings will keep the VMObject alive.
---
 Kernel/Process.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/Kernel/Process.cpp b/Kernel/Process.cpp
index 0b33f78221..95f5ff2963 100644
--- a/Kernel/Process.cpp
+++ b/Kernel/Process.cpp
@@ -316,7 +316,7 @@ int Process::do_exec(const String& path, Vector<String>&& arguments, Vector<Stri
 
     auto vmo = VMObject::create_file_backed(descriptor->inode());
     vmo->set_name(descriptor->absolute_path());
-    RetainPtr<Region> region = allocate_region_with_vmo(LinearAddress(), descriptor->metadata().size, vmo.copy_ref(), 0, "helper", true, false);
+    RetainPtr<Region> region = allocate_region_with_vmo(LinearAddress(), descriptor->metadata().size, vmo.copy_ref(), 0, "executable", true, false);
 
     // FIXME: Should we consider doing on-demand paging here? Is it actually useful?
     bool success = region->page_in();
@@ -362,8 +362,6 @@ int Process::do_exec(const String& path, Vector<String>&& arguments, Vector<Stri
         }
     }
 
-    m_regions.append(move(region));
-
     m_signal_stack_kernel_region = nullptr;
     m_signal_stack_user_region = nullptr;
     m_display_framebuffer_region = nullptr;