[JSC] GetByIdStatus::m_wasSeenInJIT is touched in GetByIdStatus::slowVersion
authoryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Aug 2018 17:46:35 +0000 (17:46 +0000)
committeryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Aug 2018 17:46:35 +0000 (17:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=188560

Reviewed by Keith Miller.

While GetByIdStatus() / GetByIdStatus(status) constructors do not set m_wasSeenInJIT,
it is loaded unconditionally in GetByIdStatus::slowVersion. This access to the
uninitialized member field is caught in UBSan. This patch fixes it by adding an initializer
`m_wasSeenInJIT { false }`.

* bytecode/GetByIdStatus.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@234855 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/bytecode/GetByIdStatus.h

index 044d264..7112d15 100644 (file)
@@ -1,5 +1,19 @@
 2018-08-14  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
 
+        [JSC] GetByIdStatus::m_wasSeenInJIT is touched in GetByIdStatus::slowVersion
+        https://bugs.webkit.org/show_bug.cgi?id=188560
+
+        Reviewed by Keith Miller.
+
+        While GetByIdStatus() / GetByIdStatus(status) constructors do not set m_wasSeenInJIT,
+        it is loaded unconditionally in GetByIdStatus::slowVersion. This access to the
+        uninitialized member field is caught in UBSan. This patch fixes it by adding an initializer
+        `m_wasSeenInJIT { false }`.
+
+        * bytecode/GetByIdStatus.h:
+
+2018-08-14  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
+
         [DFG] DFGPredictionPropagation should set PrimaryPass when processing invariants
         https://bugs.webkit.org/show_bug.cgi?id=188557
 
index e057fb3..56ff40e 100644 (file)
@@ -158,7 +158,7 @@ private:
     
     State m_state;
     Vector<GetByIdVariant, 1> m_variants;
-    bool m_wasSeenInJIT;
+    bool m_wasSeenInJIT { false };
     JSModuleNamespaceObject* m_moduleNamespaceObject { nullptr };
     JSModuleEnvironment* m_moduleEnvironment { nullptr };
     ScopeOffset m_scopeOffset { };