node: ensure that streams2 won't .end() stdin

Stdin is purely read-only stream. Although, `net.Socket` might be used to create it if stdin is in fact a Pipe or TCP socket, the `stream.Duplex` should not try to call `.end()` on it. Fix: nodejs#1068
indutny · Mar 21, 2015 · 94975ad · 94975ad
1 parent 99c79f8
commit 94975ad
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/node.js b/src/node.js
@@ -630,6 +630,8 @@
               writable: false
             });
           }
+          // Ensure that Streams2 won't try to `.end()` the stream
+          stdin._writableState.ended = true;
           break;
 
         default:

diff --git a/test/parallel/test-regress-GH-io-1068.js b/test/parallel/test-regress-GH-io-1068.js
@@ -1 +1,2 @@
+debugger;
 process.stdin.emit('end');