Changeset 249857 in webkit

Timestamp:

Sep 13, 2019 4:18:51 PM (5 years ago)

Author:

mark.lam@apple.com

Message:

performJITMemcpy() should do its !Gigacage assertion on exit.
​https://bugs.webkit.org/show_bug.cgi?id=201780
<rdar://problem/55354867>

Reviewed by Robin Morisset.

• jit/ExecutableAllocator.h:

(JSC::GigacageAssertScope::GigacageAssertScope):
(JSC::GigacageAssertScope::~GigacageAssertScope):
(JSC::performJITMemcpy):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• jit/ExecutableAllocator.h (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2019-09-13  Mark Lam  <mark.lam@apple.com>
+
+        performJITMemcpy() should do its !Gigacage assertion on exit.
+        https://bugs.webkit.org/show_bug.cgi?id=201780
+        <rdar://problem/55354867>
+
+        Reviewed by Robin Morisset.
+
+        * jit/ExecutableAllocator.h:
+        (JSC::GigacageAssertScope::GigacageAssertScope):
+        (JSC::GigacageAssertScope::~GigacageAssertScope):
+        (JSC::performJITMemcpy):
+
 2019-09-13  Yusuke Suzuki  <ysuzuki@apple.com>
 

trunk/Source/JavaScriptCore/jit/ExecutableAllocator.h

@@ -117 +117 @@
 JS_EXPORT_PRIVATE void dumpJITMemory(const void*, const void*, size_t);
 
+struct GigacageAssertScope {
+    GigacageAssertScope(const void *src)
+        : src(src)
+    { }
+    ~GigacageAssertScope() { RELEASE_ASSERT(!Gigacage::contains(src)); }
+    const void* src;
+};
+
 static ALWAYS_INLINE void* performJITMemcpy(void *dst, const void *src, size_t n)
 {
-    RELEASE_ASSERT(!Gigacage::contains(src));
+    GigacageAssertScope assertScope(src);
 #if CPU(ARM64)
     static constexpr size_t instructionSize = sizeof(unsigned);