From 7887ce2a7891f27dbcbbdc0538890771a2546cf3 Mon Sep 17 00:00:00 2001
From: Robey Pointer <robeypointer@gmail.com>
Date: Tue, 14 Jul 2020 20:41:11 -0700
Subject: [PATCH] net: don't return the stream object from onStreamRead

CallJSOnreadMethod expects the return value to be undefined or
a new buffer, so make sure to return nothing, even when an error
causes us to destroy the stream.

Fixes: https://github.com/nodejs/node/issues/34346

PR-URL: https://github.com/nodejs/node/pull/34375
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
---
 lib/internal/stream_base_commons.js | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/lib/internal/stream_base_commons.js b/lib/internal/stream_base_commons.js
index 55b9c3d4bb03e0..233741285d2c72 100644
--- a/lib/internal/stream_base_commons.js
+++ b/lib/internal/stream_base_commons.js
@@ -204,7 +204,9 @@ function onStreamRead(arrayBuffer) {
   }
 
   if (nread !== UV_EOF) {
-    return stream.destroy(errnoException(nread, 'read'));
+    // #34375 CallJSOnreadMethod expects the return value to be a buffer.
+    stream.destroy(errnoException(nread, 'read'));
+    return;
   }
 
   // Defer this until we actually emit end
@@ -221,8 +223,11 @@ function onStreamRead(arrayBuffer) {
     // test-https-truncate test.
     if (handle.readStop) {
       const err = handle.readStop();
-      if (err)
-        return stream.destroy(errnoException(err, 'read'));
+      if (err) {
+        // #34375 CallJSOnreadMethod expects the return value to be a buffer.
+        stream.destroy(errnoException(err, 'read'));
+        return;
+      }
     }
 
     // Push a null to signal the end of data.