diff options
author | Anna Henningsen <anna@addaleax.net> | 2018-12-04 12:39:30 +0100 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2019-01-12 20:10:04 +0100 |
commit | 4da7e6e228766454912a4de5dc6f037da8223e53 (patch) | |
tree | eb60d4a391fc3ed6cad314cb2662ebb9b7b1d9a6 | |
parent | 92e95f17b64838d4cf77343c1a814d4ebd795217 (diff) | |
download | android-node-v8-4da7e6e228766454912a4de5dc6f037da8223e53.tar.gz android-node-v8-4da7e6e228766454912a4de5dc6f037da8223e53.tar.bz2 android-node-v8-4da7e6e228766454912a4de5dc6f037da8223e53.zip |
src: dispose of V8 platform in `process.exit()`
Calling `process.exit()` calls the C `exit()` function, which in turn
calls the destructors of static C++ objects. This can lead to race
conditions with other concurrently executing threads; disposing of all
Worker threads and then the V8 platform instance helps with this
(although it might not be a full solution for all problems of
this kind).
Refs: https://github.com/nodejs/node/issues/24403
Refs: https://github.com/nodejs/node/issues/25007
PR-URL: https://github.com/nodejs/node/pull/25061
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
-rw-r--r-- | src/env.cc | 7 | ||||
-rw-r--r-- | src/node.cc | 4 | ||||
-rw-r--r-- | src/node_internals.h | 1 |
3 files changed, 10 insertions, 2 deletions
diff --git a/src/env.cc b/src/env.cc index d2d5d5bf35..50c718f9e9 100644 --- a/src/env.cc +++ b/src/env.cc @@ -859,10 +859,13 @@ void Environment::AsyncHooks::grow_async_ids_stack() { uv_key_t Environment::thread_local_env = {}; void Environment::Exit(int exit_code) { - if (is_main_thread()) + if (is_main_thread()) { + stop_sub_worker_contexts(); + DisposePlatform(); exit(exit_code); - else + } else { worker_context_->Exit(exit_code); + } } void Environment::stop_sub_worker_contexts() { diff --git a/src/node.cc b/src/node.cc index d5f3f4e6aa..7a585646f6 100644 --- a/src/node.cc +++ b/src/node.cc @@ -345,6 +345,10 @@ tracing::AgentWriterHandle* GetTracingAgentWriter() { return v8_platform.GetTracingAgentWriter(); } +void DisposePlatform() { + v8_platform.Dispose(); +} + #ifdef __POSIX__ static const unsigned kMaxSignal = 32; #endif diff --git a/src/node_internals.h b/src/node_internals.h index 0b0c97f8c8..fc27b384ec 100644 --- a/src/node_internals.h +++ b/src/node_internals.h @@ -345,6 +345,7 @@ int ThreadPoolWork::CancelWork() { } tracing::AgentWriterHandle* GetTracingAgentWriter(); +void DisposePlatform(); static inline const char* errno_string(int errorno) { #define ERRNO_CASE(e) case e: return #e; |