[3.14] gh-144194: Fix mmap failure check in perf_jit_trampoline.c (GH-143713) (#144301)

gh-144194: Fix mmap failure check in perf_jit_trampoline.c (GH-143713)

mmap() returns MAP_FAILED ((void*)-1) on error, not NULL. The current
check never detects mmap failures, so jitdump initialization proceeds
even when the memory mapping fails.
(cherry picked from commit 8fe8a94a7c050bc16cac9ec300f89c0f389f9a44)

Co-authored-by: stratakis <cstratak@redhat.com>

diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2026-01-23-20-20-42.gh-issue-144194.IbXfxd.rst b/Misc/NEWS.d/next/Core_and_Builtins/2026-01-23-20-20-42.gh-issue-144194.IbXfxd.rst
new file mode 100644 (file)
index 0000000..1f33284
--- /dev/null
+++ b/Misc/NEWS.d/next/Core_and_Builtins/2026-01-23-20-20-42.gh-issue-144194.IbXfxd.rst
@@ -0,0 +1 @@
+Fix error handling in perf jitdump initialization on memory allocation failure.

diff --git a/Python/perf_jit_trampoline.c b/Python/perf_jit_trampoline.c
index c0f314a7f6522b06533d2f9c86cd27da19d6fa26..acfd27fa29526ea2ca69d08d875f580de8f45fb0 100644 (file)
--- a/Python/perf_jit_trampoline.c
+++ b/Python/perf_jit_trampoline.c
@@ -1054,7 +1054,8 @@ static void* perf_map_jit_init(void) {
         0                        // Offset 0 (first page)
     );
 
-    if (perf_jit_map_state.mapped_buffer == NULL) {
+    if (perf_jit_map_state.mapped_buffer == MAP_FAILED) {
+        perf_jit_map_state.mapped_buffer = NULL;
         close(fd);
         return NULL;  // Memory mapping failed
     }