Changeset 102380 in vbox for trunk/src

Timestamp:

Nov 29, 2023 1:45:05 PM (14 months ago)

Author:

vboxsync

Message:

libs/xpcom/python: Attempt at fixing python crashes on darwin.arm64 when using the limited API, the inherited tp_is_gc() method is crashing because it works on a different object type so we define our own, also limit comparison for different object types, bugref:10544

Location:

trunk/src/libs/xpcom18a4/python/src

Files:

• PyIID.cpp (modified) (3 diffs)
• PyXPCOM.h (modified) (1 diff)

trunk/src/libs/xpcom18a4/python/src/PyIID.cpp

@@ -237 +237 @@
                 { Py_tp_str,            (void *)(uintptr_t)&Py_nsIID::PyTypeMethod_str },
                 { Py_tp_richcompare,    (void *)(uintptr_t)&Py_nsIID::PyTypeMethod_richcompare },
+                { Py_tp_is_gc,          (void *)(uintptr_t)&Py_nsIID::PyTypeMethod_is_gc },
                 { 0, NULL } /* terminator */
         };
@@ -320 +321 @@
 Py_nsIID::PyTypeMethod_richcompare(PyObject *self, PyObject *other, int op)
 {
+    if (self->ob_type != other->ob_type) /* Different types can't be compared. */
+    {
+        if (op == Py_EQ)
+            Py_RETURN_FALSE;
+        else if (op == Py_NE)
+            Py_RETURN_TRUE;
+
+       /* different are not comparable. */
+       PyErr_SetString(PyExc_TypeError, "Comparison between different object types is not defined");
+       return NULL;
+    }
+
     PyObject *result = NULL;
         Py_nsIID *s_iid = (Py_nsIID *)self;
@@ -409 +422 @@
         delete (Py_nsIID *)ob;
 }
+
+#ifdef Py_LIMITED_API
+/* static */ int
+Py_nsIID::PyTypeMethod_is_gc(PyObject *self)
+{
+        return 1;
+}
+#endif

trunk/src/libs/xpcom18a4/python/src/PyXPCOM.h

@@ -586 +586 @@
         static NS_EXPORT_STATIC_MEMBER_(PyTypeObject *) s_pType;
         static PyTypeObject *GetTypeObject(void);
+        static int PyTypeMethod_is_gc(PyObject *self);
 #endif
         static NS_EXPORT_STATIC_MEMBER_(PyMethodDef) methods[];