Add super().__init__ calls and early returns to limit recursion.

junkmd · junkmd · commit 59968701ad75 · 2024-10-30T14:11:11.000Z
diff --git a/Lib/test/test_ctypes/test_c_simple_type_meta.py b/Lib/test/test_ctypes/test_c_simple_type_meta.py
@@ -87,11 +87,16 @@ class Sub(CtBase):
         self.assertTrue(issubclass(POINTER(Sub), Sub))
 
     def test_creating_pointer_in_dunder_init_1(self):
-        # Test for modern metaclass initialization that does not
-        # require recursion avoidance.
-
         class ct_meta(type):
             def __init__(self, name, bases, namespace):
+                super().__init__(name, bases, namespace)
+
+                # Avoid recursion.
+                # (See test_creating_pointer_in_dunder_new_1)
+                if bases == (c_void_p,):
+                    return
+                if issubclass(self, PtrBase):
+                    return
                 if bases == (object,):
                     ptr_bases = (self, PtrBase)
                 else:
@@ -120,10 +125,14 @@ class Sub2(Sub):
         self.assertTrue(issubclass(POINTER(Sub), POINTER(CtBase)))
 
     def test_creating_pointer_in_dunder_init_2(self):
-        # A simpler variant of the above.
-
         class ct_meta(type):
             def __init__(self, name, bases, namespace):
+                super().__init__(name, bases, namespace)
+
+                # Avoid recursion.
+                # (See test_creating_pointer_in_dunder_new_2)
+                if isinstance(self, p_meta):
+                    return self
                 p = p_meta(f"POINTER({self.__name__})", (self, c_void_p), {})
                 ctypes._pointer_type_cache[self] = p
 
