Skip to content

Commit f84a5e1

Browse files
hashseedevanlucas
authored andcommitted
test: use JSON.stringify to trigger stack overflow
V8's interpreter performs stack checks both at the call site and at the function entry. A recursive function could therefore trigger stack overflow at two different source locations. Instead of recursion, call JSON.stringify on a deeply nested array. PR-URL: #12481 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent e77f1e2 commit f84a5e1

File tree

2 files changed

+9
-7
lines changed

2 files changed

+9
-7
lines changed

test/message/stack_overflow.js

+6-4
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,12 @@ Error.stackTraceLimit = 0;
55

66
console.error('before');
77

8-
// stack overflow
9-
function stackOverflow() {
10-
stackOverflow();
8+
// Trigger stack overflow by stringifying a deeply nested array.
9+
let array = [];
10+
for (let i = 0; i < 100000; i++) {
11+
array = [ array ];
1112
}
12-
stackOverflow();
13+
14+
JSON.stringify(array);
1315

1416
console.error('after');

test/message/stack_overflow.out

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
before
2-
32
*test*message*stack_overflow.js:*
4-
function stackOverflow() {
5-
^
3+
JSON.stringify(array);
4+
^
5+
66
RangeError: Maximum call stack size exceeded

0 commit comments

Comments
 (0)