gh-124621: Emscripten: Fix regression in use-after-close error handli…

…ng (#136837)
python · ambv · Jul 22, 2025 · Jul 19, 2025 · Jul 19, 2025 · Jul 22, 2025
commit 4d7d5075091125c027263dc6d094ac52cdd8d814
diff --git a/Python/emscripten_syscalls.c b/Python/emscripten_syscalls.c
@@ -148,10 +148,17 @@ EM_JS_MACROS(__externref_t, __maybe_fd_read_async, (
     size_t iovcnt,
     __wasi_size_t *nread
 ), {
-    var stream = SYSCALLS.getStreamFromFD(fd);
     if (!WebAssembly.promising) {
         return null;
     }
+    var stream;
+    try {
+        stream = SYSCALLS.getStreamFromFD(fd);
+    } catch (e) {
+        // If the fd was already closed or never existed, getStreamFromFD()
+        // raises. We'll let fd_read_orig() handle setting errno.
+        return null;
+    }
     if (!stream.stream_ops.readAsync) {
         // Not an async device. Fall back to __wasi_fd_read_orig().
         return null;