Fix the short-circuit in type_ready_set_bases().

ericsnowcurrently · ericsnowcurrently · commit 116959a9bd04 · 2023-05-30T16:55:47.000-06:00
diff --git a/Objects/typeobject.c b/Objects/typeobject.c
@@ -7037,8 +7037,12 @@ type_ready_set_type(PyTypeObject *type)
 static int
 type_ready_set_bases(PyTypeObject *type)
 {
-    if (lookup_tp_bases(type) != NULL) {
-        return 0;
+    if (type->tp_flags & _Py_TPFLAGS_STATIC_BUILTIN) {
+        if (!_Py_IsMainInterpreter(_PyInterpreterState_GET())) {
+            assert(lookup_tp_bases(type) != NULL);
+            return 0;
+        }
+        assert(lookup_tp_bases(type) == NULL);
     }
 
     PyObject *bases = lookup_tp_bases(type);

Original file line number	Diff line number	Diff line change
`@@ -7037,8 +7037,12 @@ type_ready_set_type(PyTypeObject *type)`
`7037`	`7037`	`static int`
`7038`	`7038`	`type_ready_set_bases(PyTypeObject *type)`
`7039`	`7039`	`{`
`7040`		`- if (lookup_tp_bases(type) != NULL) {`
`7041`		`- return 0;`
	`7040`	`+ if (type->tp_flags & _Py_TPFLAGS_STATIC_BUILTIN) {`
	`7041`	`+ if (!_Py_IsMainInterpreter(_PyInterpreterState_GET())) {`
	`7042`	`+ assert(lookup_tp_bases(type) != NULL);`
	`7043`	`+ return 0;`
	`7044`	`+ }`
	`7045`	`+ assert(lookup_tp_bases(type) == NULL);`
`7042`	`7046`	`}`
`7043`	`7047`
`7044`	`7048`	`PyObject *bases = lookup_tp_bases(type);`