From: sobolevn <mail@sobolevn.me>
Date: Mon, 26 May 2025 15:06:33 +0000 (+0300)
Subject: gh-134693: Fix `[-Wmaybe-uninitialized]` warning in `_remote_debugging_module.c`... 
X-Git-Tag: v3.15.0a1~1509
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=806107d7a2fa9baa76d4025f46fab2c8725963f4;p=thirdparty%2FPython%2Fcpython.git

gh-134693: Fix `[-Wmaybe-uninitialized]` warning in `_remote_debugging_module.c` (#134694)
---

diff --git a/Modules/_remote_debugging_module.c b/Modules/_remote_debugging_module.c
index 6c824f9d8d22..86e269f31246 100644
--- a/Modules/_remote_debugging_module.c
+++ b/Modules/_remote_debugging_module.c
@@ -2054,6 +2054,14 @@ parse_async_frame_object(
 
     *previous_frame = GET_MEMBER(uintptr_t, frame, unwinder->debug_offsets.interpreter_frame.previous);
 
+    *code_object = GET_MEMBER(uintptr_t, frame, unwinder->debug_offsets.interpreter_frame.executable);
+    // Strip tag bits for consistent comparison
+    *code_object &= ~Py_TAG_BITS;
+    assert(code_object != NULL);
+    if ((void*)*code_object == NULL) {
+        return 0;
+    }
+
     if (GET_MEMBER(char, frame, unwinder->debug_offsets.interpreter_frame.owner) == FRAME_OWNED_BY_CSTACK ||
         GET_MEMBER(char, frame, unwinder->debug_offsets.interpreter_frame.owner) == FRAME_OWNED_BY_INTERPRETER) {
         return 0;  // C frame
@@ -2067,15 +2075,6 @@ parse_async_frame_object(
         return -1;
     }
 
-    *code_object = GET_MEMBER(uintptr_t, frame, unwinder->debug_offsets.interpreter_frame.executable);
-    // Strip tag bits for consistent comparison
-    *code_object &= ~Py_TAG_BITS;
-
-    assert(code_object != NULL);
-    if ((void*)*code_object == NULL) {
-        return 0;
-    }
-
     uintptr_t instruction_pointer = GET_MEMBER(uintptr_t, frame, unwinder->debug_offsets.interpreter_frame.instr_ptr);
 
     // Get tlbc_index for free threading builds