From 97ce5e0b4b614872ced69ba800bc726d464e92f2 Mon Sep 17 00:00:00 2001
From: Anna Henningsen <anna@addaleax.net>
Date: Tue, 5 Mar 2019 20:16:33 +0100
Subject: [PATCH] src: guard against calling `Init()` multiple times

This function should only be called once. Calling it multiple times
would currently break Node.js (e.g. re-registering builtin modules
would break the linked list for them).

PR-URL: https://github.com/nodejs/node/pull/26458
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 src/node.cc | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/node.cc b/src/node.cc
index 4def46ee167dbd..89217e47bc1070 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -608,9 +608,14 @@ int ProcessGlobalArgs(std::vector<std::string>* args,
   return 0;
 }
 
+static std::atomic_bool init_called{false};
+
 int Init(std::vector<std::string>* argv,
          std::vector<std::string>* exec_argv,
          std::vector<std::string>* errors) {
+  // Make sure Init() is called only once.
+  CHECK(!init_called.exchange(true));
+
   // Register built-in modules
   binding::RegisterBuiltinModules();