async_hooks: move restoreTmpHooks call to init

This fixes an error that could occure by nesting async_hooks calls PR-URL: #14054 Ref: #13755 (comment) Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Trevor Norris <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Andreas Madsen <[email protected]>

Author: BridgeAR

lib/async_hooks.js

@@ -335,11 +335,6 @@ function emitInitS(asyncId, type, triggerAsyncId, resource){
 thrownewRangeError('triggerAsyncId must be an unsigned integer');
 
 init(asyncId,type,triggerAsyncId,resource);
-
-// Isn't null if hooks were added/removed while the hooks were running.
-if(tmp_active_hooks_array!==null){
-restoreTmpHooks();
-}
 }
 
 functionemitHookFactory(symbol,name){
@@ -442,6 +437,11 @@ function init(asyncId, type, triggerAsyncId, resource){
 fatalError(e);
 }
 processing_hook=false;
+
+// Isn't null if hooks were added/removed while the hooks were running.
+if(tmp_active_hooks_array!==null){
+restoreTmpHooks();
+}
 }
 
 

test/parallel/test-async-hooks-enable-recursive.js

@@ -0,0 +1,19 @@
+'use strict';
+
+constcommon=require('../common');
+constasync_hooks=require('async_hooks');
+constfs=require('fs');
+
+constnestedHook=async_hooks.createHook({
+init: common.mustCall()
+});
+
+async_hooks.createHook({
+init: common.mustCall((id,type)=>{
+nestedHook.enable();
+},2)
+}).enable();
+
+fs.access(__filename,common.mustCall(()=>{
+fs.access(__filename,common.mustCall());
+}));