Fix limits

pablogsal · pablogsal · commit e528bc1ad8f7 · 2025-12-06T21:47:39.000Z
diff --git a/Lib/test/test_external_inspection.py b/Lib/test/test_external_inspection.py
@@ -2724,11 +2724,11 @@ def recurse(n):
             unwinder_no_cache = make_unwinder(cache_frames=False)
 
             frames_cached = self._sample_frames(
-                client_socket, unwinder_cache, b"ready", b"ack", {"recurse"}, 1100
+                client_socket, unwinder_cache, b"ready", b"ack", {"recurse"}, expected_frames=1102
             )
             # Sample again with no cache for comparison
             frames_no_cache = self._sample_frames(
-                client_socket, unwinder_no_cache, b"ready2", b"done", {"recurse"}, 1100
+                client_socket, unwinder_no_cache, b"ready2", b"done", {"recurse"}, expected_frames=1102
             )
 
         self.assertIsNotNone(frames_cached)
diff --git a/Modules/_remote_debugging/frame_cache.c b/Modules/_remote_debugging/frame_cache.c
@@ -221,8 +221,10 @@ frame_cache_store(
     // Clear old frame_list if replacing
     Py_CLEAR(entry->frame_list);
 
-    // Store data - truncate frame_list to match num_addrs
-    entry->frame_list = PyList_GetSlice(frame_list, 0, num_addrs);
+    // Store full frame list (don't truncate to num_addrs - frames beyond the
+    // address array limit are still valid and needed for full cache hits)
+    Py_ssize_t num_frames = PyList_GET_SIZE(frame_list);
+    entry->frame_list = PyList_GetSlice(frame_list, 0, num_frames);
     if (!entry->frame_list) {
         return -1;
     }

Original file line number	Diff line number	Diff line change
`@@ -2724,11 +2724,11 @@ def recurse(n):`
`2724`	`2724`	`unwinder_no_cache = make_unwinder(cache_frames=False)`
`2725`	`2725`
`2726`	`2726`	`frames_cached = self._sample_frames(`
`2727`		`- client_socket, unwinder_cache, b"ready", b"ack", {"recurse"}, 1100`
	`2727`	`+ client_socket, unwinder_cache, b"ready", b"ack", {"recurse"}, expected_frames=1102`
`2728`	`2728`	`)`
`2729`	`2729`	`# Sample again with no cache for comparison`
`2730`	`2730`	`frames_no_cache = self._sample_frames(`
`2731`		`- client_socket, unwinder_no_cache, b"ready2", b"done", {"recurse"}, 1100`
	`2731`	`+ client_socket, unwinder_no_cache, b"ready2", b"done", {"recurse"}, expected_frames=1102`
`2732`	`2732`	`)`
`2733`	`2733`
`2734`	`2734`	`self.assertIsNotNone(frames_cached)`