fix(hooks): require page to be at-least readable

Curve · May 9, 2024 · aeec3f6 · aeec3f6
1 parent f6eee3b
commit aeec3f6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/include/lime/hooks/hook.hpp b/include/lime/hooks/hook.hpp
@@ -46,6 +46,7 @@ namespace lime
         protect,
         relocate,
         bad_page,
+        bad_prot,
     };
 
     class hook_base

diff --git a/src/hooks.hook.cpp b/src/hooks.hook.cpp
@@ -76,6 +76,11 @@ namespace lime
             return tl::make_unexpected(hook_error::bad_page);
         }
 
+        if (!(page->prot() & lime::protection::read))
+        {
+            return tl::make_unexpected(hook_error::bad_prot);
+        }
+
         auto start = instruction::unsafe(source);
 
         if (auto follow = start.follow(); follow)