deps: update ChakraCore to chakra-core/ChakraCore@e4493a2fe7

[1.8>1.9] [MERGE #4643 @MikeHolman] fix race condition where multiple threads simultaneously connect Merge pull request #4643 from MikeHolman:connectrace Reviewed-By: chakrabot <chakrabot@users.noreply.github.com>
nodejs · Feb 7, 2018 · d6fe9af · d6fe9af
1 parent 06c6da9
commit d6fe9af
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/deps/chakrashim/core/lib/JITClient/JITManager.cpp b/deps/chakrashim/core/lib/JITClient/JITManager.cpp
@@ -226,20 +226,17 @@ JITManager::IsOOPJITEnabled() const
 HRESULT
 JITManager::ConnectRpcServer(__in HANDLE jitProcessHandle, __in_opt void* serverSecurityDescriptor, __in UUID connectionUuid)
 {
+    // We might be trying to connect from multiple threads simultaneously
+    AutoCriticalSection cs(&m_cs);
+
     Assert(IsOOPJITEnabled());
-    if(m_rpcBindingHandle != nullptr)
+    if (m_rpcBindingHandle != nullptr)
     {
-        // TODO: change this to allow connecting a new JIT process to new ThreadContexts
-        return E_FAIL;
+        return S_OK;
     }
 
     HRESULT hr = E_FAIL;
 
-    if (IsConnected())
-    {
-        Assert(UNREACHED);
-        return E_FAIL;
-    }
 
     hr = CreateBinding(jitProcessHandle, serverSecurityDescriptor, &connectionUuid, &m_rpcBindingHandle);
     if (FAILED(hr))

diff --git a/deps/chakrashim/core/lib/JITClient/JITManager.h b/deps/chakrashim/core/lib/JITClient/JITManager.h
@@ -140,6 +140,7 @@ class JITManager
     bool m_oopJitEnabled;
     bool m_isJITServer;
     HRESULT m_failingHRESULT;
+    CriticalSection m_cs;
 
     static JITManager s_jitManager;