Commit 803fbfb
process: fix uid/gid validation to avoid crash
id |= 0 turns unsigned 32-bit integer values exceeding the unsigned
31-bit range into negative integers, causing a crash. Use id >>>= 0
instead, which works properly for all unsigned 32-bit integers.
Refs: #36786
PR-URL: #44910
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Erick Wendel <erick.workspace@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>1 parent 9f2dd48 commit 803fbfbCopy full SHA for 803fbfb
File tree
Expand file treeCollapse file tree
2 files changed
+8
-12
lines changedOpen diff view settings
Filter options
- lib/internal/bootstrap/switches
- test/parallel
Expand file treeCollapse file tree
2 files changed
+8
-12
lines changedOpen diff view settings
Collapse file
lib/internal/bootstrap/switches/does_own_process_state.js
Copy file name to clipboardExpand all lines: lib/internal/bootstrap/switches/does_own_process_state.js+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
76 | 76 | |
77 | 77 | |
78 | 78 | |
79 | | - |
| 79 | + |
80 | 80 | |
81 | 81 | |
82 | 82 | |
|
Collapse file
test/parallel/test-process-uid-gid.js
Copy file name to clipboardExpand all lines: test/parallel/test-process-uid-gid.js+7-11Lines changed: 7 additions & 11 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
53 | 53 | |
54 | 54 | |
55 | 55 | |
56 | | - |
57 | | - |
58 | | - |
59 | | - |
60 | | - |
61 | | - |
62 | | - |
63 | | - |
64 | | - |
65 | | - |
66 | | - |
| 56 | + |
| 57 | + |
| 58 | + |
| 59 | + |
| 60 | + |
| 61 | + |
| 62 | + |
67 | 63 | |
68 | 64 | |
69 | 65 | |
|
0 commit comments