Commit cfc769b
committed
async_hooks: fix resource stack for deep stacks
460c81d introduced a bug where the execution resource was not
stored properly if we needed to call into C++ to extend the stack size.
Fix that bug by always storing the resource.
Refs: #34319
Fixes: #34556
PR-URL: #34573
Reviewed-By: Andrey Pechkurov <apechkurov@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Gus Caplan <me@gus.host>1 parent a2141d3 commit cfc769bCopy full SHA for cfc769b
File tree
Expand file treeCollapse file tree
2 files changed
+16
-1
lines changedOpen diff view settings
Filter options
- lib/internal
- test/parallel
Expand file treeCollapse file tree
2 files changed
+16
-1
lines changedOpen diff view settings
Collapse file
lib/internal/async_hooks.js
Copy file name to clipboardExpand all lines: lib/internal/async_hooks.js+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
434 | 434 | |
435 | 435 | |
436 | 436 | |
| 437 | + |
437 | 438 | |
438 | 439 | |
439 | 440 | |
440 | 441 | |
441 | | - |
442 | 442 | |
443 | 443 | |
444 | 444 | |
|
Collapse file
test/parallel/test-async-local-storage-deep-stack.js
Copy file name to clipboard+15Lines changed: 15 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
0 commit comments