net: fix connect crash when call destroy in lookup handler

PR-URL: #51826 Fixes: #50841 Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Jiawen Geng <[email protected]>
nodejs · Feb 29, 2024 · c6cc3ed · c6cc3ed
1 parent db30428
commit c6cc3ed
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/lib/net.js b/lib/net.js
@@ -1435,7 +1435,10 @@ function lookupAndConnectMultiple(
         const address = addresses[i];
         const { address: ip, family: addressType } = address;
         self.emit('lookup', err, ip, addressType, host);
-
+        // It's possible we were destroyed while looking this up.
+        if (!self.connecting) {
+          return;
+        }
         if (isIP(ip) && (addressType === 4 || addressType === 6)) {
           if (!destinations) {
             destinations = addressType === 6 ? { 6: 0, 4: 1 } : { 4: 0, 6: 1 };

diff --git a/test/parallel/test-destroy-socket-in-lookup.js b/test/parallel/test-destroy-socket-in-lookup.js
@@ -0,0 +1,16 @@
+'use strict';
+const common = require('../common');
+const net = require('net');
+
+// Test that the process does not crash.
+const socket = net.connect({
+  port: 12345,
+  host: 'localhost',
+  // Make sure autoSelectFamily is true
+  // so that lookupAndConnectMultiple is called.
+  autoSelectFamily: true,
+});
+// DNS resolution fails or succeeds
+socket.on('lookup', common.mustCall(() => {
+  socket.destroy();
+}));