repl: fix eval return value

In case no error has occurred during the evaluation of some code,
`undefined` has been returned in some cases as error argument instead
of `null`. This is fixed by this patch.

PR-URL: https://github.com/nodejs/node/pull/25731
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

1 files changed, 6 insertions, 5 deletions

diff --git a/lib/repl.js b/lib/repl.js
index df861843d1..e0f5373c8c 100644
--- a/lib/repl.js
+++ b/lib/repl.js
@@ -215,10 +215,11 @@ function REPLServer(prompt,
   }
 
   function defaultEval(code, context, file, cb) {
-    var err, result, script, wrappedErr;
-    var wrappedCmd = false;
-    var awaitPromise = false;
-    var input = code;
+    let result, script, wrappedErr;
+    let err = null;
+    let wrappedCmd = false;
+    let awaitPromise = false;
+    const input = code;
 
     if (/^\s*\{/.test(code) && /\}\s*$/.test(code)) {
       // It's confusing for `{ a : 1 }` to be interpreted as a block
@@ -362,7 +363,7 @@ function REPLServer(prompt,
         }
 
         promise.then((result) => {
-          finishExecution(undefined, result);
+          finishExecution(null, result);
         }, (err) => {
           if (err && process.domain) {
             debug('not recoverable, send to domain');