src: fix crash on OnStreamRead on Windows

On Windows it's perfectly possible that the `uv_tcp_t` `read_cb` is
called with an error and a null `uv_buf_t` if it corresponds to a
`UV_HANDLE_ZERO_READ` read. Handle this case without crashing.

Fixes: #40764
PR-URL: #45878
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>

Author: santigimeno

src/stream_base.cc

@@ -583,9 +583,10 @@ void CustomBufferJSListener::OnStreamRead(ssize_t nread, const uv_buf_t& buf) {
   HandleScope handle_scope(env->isolate());
   Context::Scope context_scope(env->context());
 
-  // To deal with the case where POLLHUP is received and UV_EOF is returned, as
-  // libuv returns an empty buffer (on unices only).
-  if (nread == UV_EOF && buf.base == nullptr) {
+  // In the case that there's an error and buf is null, return immediately.
+  // This can happen on unices when POLLHUP is received and UV_EOF is returned
+  // or when getting an error while performing a UV_HANDLE_ZERO_READ on Windows.
+  if (buf.base == nullptr && nread < 0) {
     stream->CallJSOnreadMethod(nread, Local<ArrayBuffer>());
     return;
   }

test/parallel/test-net-child-process-connect-reset.js

@@ -0,0 +1,47 @@
+'use strict';
+
+const common = require('../common');
+const assert = require('assert');
+const { spawn } = require('child_process');
+const net = require('net');
+
+if (process.argv[2] === 'child') {
+  const server = net.createServer(common.mustCall());
+  server.listen(0, common.mustCall(() => {
+    process.send({ type: 'ready', data: { port: server.address().port } });
+  }));
+} else {
+  const cp = spawn(process.execPath,
+                   [__filename, 'child'],
+                   {
+                     stdio: ['ipc', 'inherit', 'inherit']
+                   });
+
+  cp.on('exit', common.mustCall((code, signal) => {
+    assert.strictEqual(code, null);
+    assert.strictEqual(signal, 'SIGKILL');
+  }));
+
+  cp.on('message', common.mustCall((msg) => {
+    const { type, data } = msg;
+    assert.strictEqual(type, 'ready');
+    const port = data.port;
+
+    const conn = net.createConnection({
+      port,
+      onread: {
+        buffer: Buffer.alloc(65536),
+        callback: () => {},
+      }
+    });
+
+    conn.on('error', (err) => {
+      // Error emitted on Windows.
+      assert.strictEqual(err.code, 'ECONNRESET');
+    });
+
+    conn.on('connect', common.mustCall(() => {
+      cp.kill('SIGKILL');
+    }));
+  }));
+}