src: prevent hard coding stack trace limit

Refer to Environment::stack_trace_limit() while printing fresh
stacktraces in c++ land.

PR-URL: #30752
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Denys Otrishko <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>

Author: legendecas

src/env.cc

@@ -592,9 +592,9 @@ void Environment::PrintSyncTrace() const {
 
   fprintf(
       stderr, "(node:%d) WARNING: Detected use of sync API\n", uv_os_getpid());
-  PrintStackTrace(
-      isolate(),
-      StackTrace::CurrentStackTrace(isolate(), 10, StackTrace::kDetailed));
+  PrintStackTrace(isolate(),
+                  StackTrace::CurrentStackTrace(
+                      isolate(), stack_trace_limit(), StackTrace::kDetailed));
 }
 
 void Environment::RunCleanup() {
@@ -949,9 +949,9 @@ void Environment::Exit(int exit_code) {
 
     fprintf(
         stderr, "WARNING: Exited the environment with code %d\n", exit_code);
-    PrintStackTrace(
-        isolate(),
-        StackTrace::CurrentStackTrace(isolate(), 10, StackTrace::kDetailed));
+    PrintStackTrace(isolate(),
+                    StackTrace::CurrentStackTrace(
+                        isolate(), stack_trace_limit(), StackTrace::kDetailed));
   }
   if (is_main_thread()) {
     stop_sub_worker_contexts();

src/env.h

@@ -1234,6 +1234,8 @@ class Environment : public MemoryRetainer {
   inline void modify_base_object_count(int64_t delta);
   inline int64_t base_object_count() const;
 
+  inline int32_t stack_trace_limit() const { return 10; }
+
 #if HAVE_INSPECTOR
   void set_coverage_connection(
       std::unique_ptr<profiler::V8CoverageConnection> connection);

src/node_errors.cc

@@ -145,7 +145,7 @@ static std::string GetErrorSource(Isolate* isolate,
 }
 
 void PrintStackTrace(Isolate* isolate, Local<StackTrace> stack) {
-  for (int i = 0; i < stack->GetFrameCount() - 1; i++) {
+  for (int i = 0; i < stack->GetFrameCount(); i++) {
     Local<StackFrame> stack_frame = stack->GetFrame(isolate, i);
     node::Utf8Value fn_name_s(isolate, stack_frame->GetFunctionName());
     node::Utf8Value script_name(isolate, stack_frame->GetScriptName());

test/message/throw_error_with_getter_throw_traced.out

@@ -10,3 +10,4 @@ Thrown at:
     at Module.load (internal/modules/cjs/loader.js:*:*)
     at Module._load (internal/modules/cjs/loader.js:*:*)
     at executeUserEntryPoint (internal/modules/run_main.js:*:*)
+    at internal/main/run_main_module.js:*:*

test/message/throw_null_traced.out

@@ -10,3 +10,4 @@ Thrown at:
     at Module.load (internal/modules/cjs/loader.js:*:*)
     at Module._load (internal/modules/cjs/loader.js:*:*)
     at executeUserEntryPoint (internal/modules/run_main.js:*:*)
+    at internal/main/run_main_module.js:*:*

test/message/throw_undefined_traced.out

@@ -10,3 +10,4 @@ Thrown at:
     at Module.load (internal/modules/cjs/loader.js:*:*)
     at Module._load (internal/modules/cjs/loader.js:*:*)
     at executeUserEntryPoint (internal/modules/run_main.js:*:*)
+    at internal/main/run_main_module.js:*:*