readline: fix to not access a property on an undefined value

nodejs · Jun 23, 2022 · 83c879a · 83c879a
1 parent f5ce6b1
commit 83c879a
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/lib/internal/readline/utils.js b/lib/internal/readline/utils.js
@@ -366,6 +366,9 @@ function* emitKeys(stream) {
 
 // This runs in O(n log n).
 function commonPrefix(strings) {
+  if (strings.length === 0) {
+    return '';
+  }
   if (strings.length === 1) {
     return strings[0];
   }

diff --git a/test/parallel/test-repl-tab-complete-editor-mode.js b/test/parallel/test-repl-tab-complete-editor-mode.js
@@ -0,0 +1,21 @@
+'use strict';
+
+require('../common');
+const ArrayStream = require('../common/arraystream');
+const repl = require('repl');
+
+const stream = new ArrayStream();
+const replServer = repl.start({
+  input: stream,
+  output: stream,
+  terminal: true
+});
+
+// Editor mode
+replServer.write('.editor\n');
+
+// Regression test for https://github.com/nodejs/node/issues/43528
+replServer.write('a');
+replServer.write(null, { name: 'tab' }); // Should not throw
+
+replServer.close();