Commit 830949c
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 <anna@addaleax.net>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 278e8cc commit 830949cCopy full SHA for 830949c
File tree
Expand file treeCollapse file tree
2 files changed
+9
-7
lines changedOpen diff view settings
Filter options
- test/message
Expand file treeCollapse file tree
2 files changed
+9
-7
lines changedOpen diff view settings
Collapse file
test/message/stack_overflow.js
Copy file name to clipboardExpand all lines: test/message/stack_overflow.js+6-4Lines changed: 6 additions & 4 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
5 | 5 | |
6 | 6 | |
7 | 7 | |
8 | | - |
9 | | - |
10 | | - |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
11 | 12 | |
12 | | - |
| 13 | + |
| 14 | + |
13 | 15 | |
14 | 16 | |
Collapse file
test/message/stack_overflow.out
Copy file name to clipboard+3-3Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
1 | 1 | |
2 | | - |
3 | 2 | |
4 | | - |
5 | | - |
| 3 | + |
| 4 | + |
| 5 | + |
6 | 6 | |
0 commit comments