dgram: fix unhandled exception aborting a closed udp socket

Fixes: nodejs#46750
vramana · Feb 22, 2023 · ba3fa6c · ba3fa6c
1 parent fda0de4
commit ba3fa6c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/dgram.js b/lib/dgram.js
@@ -139,7 +139,7 @@ function Socket(type, listener) {
     const { signal } = options;
     validateAbortSignal(signal, 'options.signal');
     const onAborted = () => {
-      this.close();
+      if (this[kStateSymbol].handle) this.close();
     };
     if (signal.aborted) {
       onAborted();

diff --git a/test/parallel/test-dgram-abort-closed.js b/test/parallel/test-dgram-abort-closed.js
@@ -0,0 +1,10 @@
+'use strict';
+
+const dgram = require('dgram');
+
+const controller = new AbortController();
+const socket = dgram.createSocket({ type: 'udp4', signal: controller.signal });
+
+socket.close();
+
+controller.abort();