Commit 426aa0a
src: fix Windows segfault with
When specifing a parameter that requries an additional argument on the
command line, node would segfault. This appears to be specific to
Windows, adjusted command line argument parsing to hold a nullptr
terminal.
Adding unit test for crash on missing arguments.
PR-URL: #6938
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>--eval
1 parent 9cccaa3 commit 426aa0aCopy full SHA for 426aa0a
File tree
Expand file treeCollapse file tree
2 files changed
+10
-3
lines changedOpen diff view settings
Filter options
- src
- test/parallel
Expand file treeCollapse file tree
2 files changed
+10
-3
lines changedOpen diff view settings
Collapse file
+2-1Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
3 | 3 | |
4 | 4 | |
5 | 5 | |
6 | | - |
| 6 | + |
7 | 7 | |
8 | 8 | |
9 | 9 | |
| ||
35 | 35 | |
36 | 36 | |
37 | 37 | |
| 38 | + |
38 | 39 | |
39 | 40 | |
40 | 41 | |
|
Collapse file
test/parallel/test-cli-eval.js
Copy file name to clipboardExpand all lines: test/parallel/test-cli-eval.js+8-2Lines changed: 8 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
40 | 40 | |
41 | 41 | |
42 | 42 | |
43 | | - |
| 43 | + |
44 | 44 | |
45 | 45 | |
46 | 46 | |
47 | | - |
| 47 | + |
48 | 48 | |
49 | 49 | |
50 | 50 | |
| ||
59 | 59 | |
60 | 60 | |
61 | 61 | |
| 62 | + |
| 63 | + |
| 64 | + |
| 65 | + |
| 66 | + |
| 67 | + |
62 | 68 | |
63 | 69 | |
64 | 70 | |
|
0 commit comments