Repository navigation
win: pipe UV_HANDLE_EMULATE_IOCP implementation broken #629
Description
observed at JuliaLang/julia#11727 (comment)
but can also be triggered in nodejs with the following command (from a shell such as Cygwin mintty):
(/cygdrive/C/Program\ Files/nodejs/node -e "process.stdout.write('a')" &&
/cygdrive/C/Program\ Files/nodejs/node -e "process.stdout.write('b')") | cat -(although node itself appears to drop the error and simply hangs)
as mentioned at the above link, the error appears to be that overlapped->hEvent is not initialized prior to a call to ReadFile or WriteFile in the pipe.c code (I think the tty.c code equivalents look correct in this regard.)
Activity
saghul commentedon Nov 24, 2015
/cc @piscisaureus
vtjnash commentedon Jun 11, 2018
Fixed by JuliaAttic/libuv-archive#44. I'm just waiting for some of my other PRs to be merged before I open more.
stale commentedon Aug 10, 2019
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
vtjnash commentedon Aug 12, 2019
It's still broken, and I still need to turn our patch into a PR...
tests
tests
vtjnash commentedon Jan 13, 2020
The nodejs reproducer seems to have stopped functioning between v8.11.4 and v8.12.0, though I don't know which commit. Maybe nodejs/node#19551 fixed it incidentally or nodejs/node#18019?
win,pipe: ensure `req->event_handle` is defined
win,pipe: ensure `req->event_handle` is defined
win,pipe: ensure `req->event_handle` is defined
win,pipe: ensure `req->event_handle` is defined
win,pipe: ensure `req->event_handle` is defined
win,pipe: ensure `req->event_handle` is defined