readline: clean up event listener in onNewListener

Once the Readline interface is closed, the 'data' event listener should be removed. Refs: nodejs#9447 (comment)
gibfahn · May 28, 2017 · 6d27376 · 6d27376
1 parent 5b50501
commit 6d27376
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/lib/readline.js b/lib/readline.js
@@ -1030,6 +1030,7 @@ function emitKeypressEvents(stream, iface) {
   function onNewListener(event) {
     if (event === 'keypress') {
       stream.on('data', onData);
+      iface.once('close', () => { stream.removeListener('data', onData); });
       stream.removeListener('newListener', onNewListener);
     }
   }