-
Notifications
You must be signed in to change notification settings - Fork 29.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unreachable code reached when using {stdio: "overlapped"} #48476
Labels
child_process
Issues and PRs related to the child_process subsystem.
repro-exists
Issues with reproductions.
Comments
bnoordhuis
added
the
child_process
Issues and PRs related to the child_process subsystem.
label
Jun 17, 2023
That's an oversight in #29412. It added the 'overlapped' flag to src/process_wrap.cc but not src/spawn_sync.cc. Pull request welcome. |
bnoordhuis
added a commit
to bnoordhuis/io.js
that referenced
this issue
Jun 17, 2023
Fix an oversight in the pull request that introduced the 'overlapped' option to the asynchronous child process methods, and also add it to the synchronous methods, like child_process.spawnSync(). Fixes: nodejs#48476 Refs: nodejs#29412
aduh95
pushed a commit
to bnoordhuis/io.js
that referenced
this issue
May 11, 2024
Fix an oversight in the pull request that introduced the 'overlapped' option to the asynchronous child process methods, and also add it to the synchronous methods, like child_process.spawnSync(). Fixes: nodejs#48476 Refs: nodejs#29412
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
child_process
Issues and PRs related to the child_process subsystem.
repro-exists
Issues with reproductions.
Version
18.16.0
Platform
Linux fedora 6.3.5-200.fc38.x86_64 #1 SMP PREEMPT_DYNAMIC Tue May 30 15:44:17 UTC 2023 x86_64 GNU/Linux
Subsystem
No response
What steps will reproduce the bug?
How often does it reproduce? Is there a required condition?
Always
What is the expected behavior? Why is that the expected behavior?
Execute the subprocess, and return some object.
According to the documentation,
pipe
should be used on non-Windows systems.What do you see instead?
A segfault.
Additional information
No response
The text was updated successfully, but these errors were encountered: