src: use v8::Isolate::TryGetCurrent() in DumpJavaScriptBacktrace()

It was using Isolate::GetCurrent() which DCHECK on nullptr, even though what we wanted was to return early if it is nullptr. PR-URL: nodejs#50518 Refs: nodejs#50242 Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
anonrig · Nov 9, 2023 · 4691144 · 4691144
1 parent 24afbb9
commit 4691144
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/debug_utils.cc b/src/debug_utils.cc
@@ -319,7 +319,7 @@ void DumpNativeBacktrace(FILE* fp) {
 }
 
 void DumpJavaScriptBacktrace(FILE* fp) {
-  v8::Isolate* isolate = v8::Isolate::GetCurrent();
+  v8::Isolate* isolate = v8::Isolate::TryGetCurrent();
   if (isolate == nullptr) {
     return;
   }

diff --git a/test/cctest/test_util.cc b/test/cctest/test_util.cc
@@ -299,3 +299,7 @@ TEST(UtilTest, SPrintF) {
   const std::string with_zero = std::string("a") + '\0' + 'b';
   EXPECT_EQ(SPrintF("%s", with_zero), with_zero);
 }
+
+TEST(UtilTest, DumpJavaScriptStackWithNoIsolate) {
+  node::DumpJavaScriptBacktrace(stderr);
+}