Disallow mixing objects from different contexts.

Author: PetterS

module.c

@@ -137,6 +137,11 @@ static PyObject *object_call(ObjectData *self, PyObject *args, PyObject *kwds) {
 		} else if (item == Py_None) {
 		} else if (PyUnicode_Check(item)) {
 		} else if (PyObject_IsInstance(item, (PyObject *)&Object)) {
+			ObjectData *object = (ObjectData *)item;
+			if (object->context != self->context) {
+				PyErr_Format(PyExc_ValueError, "Can not mix JS objects from different contexts.");
+				return NULL;
+			}
 		} else {
 			PyErr_Format(PyExc_ValueError,
 			             "Unsupported type of argument %d when calling quickjs object: %s.",

setup.py

@@ -3,18 +3,21 @@
 
 from setuptools import setup, Extension
 
+
 def get_c_sources(include_headers=False):
-      sources=['module.c'] + glob.glob("third-party/*.c")
-      if include_headers:
-            sources += glob.glob("third-party/*.h")
-      return sources
+    sources = ['module.c'] + glob.glob("third-party/*.c")
+    if include_headers:
+        sources += glob.glob("third-party/*.h")
+    return sources
+
 
-_quickjs = Extension('_quickjs',
-                     define_macros=[('CONFIG_VERSION', '"2019-07-09"')],
-                     # HACK.
-                     # See https://github.com/pypa/packaging-problems/issues/84.
-                     sources=get_c_sources(include_headers=("sdist" in sys.argv)),
-                     headers=glob.glob("third-party/*.h"))
+_quickjs = Extension(
+    '_quickjs',
+    define_macros=[('CONFIG_VERSION', '"2019-07-09"')],
+    # HACK.
+    # See https://github.com/pypa/packaging-problems/issues/84.
+    sources=get_c_sources(include_headers=("sdist" in sys.argv)),
+    headers=glob.glob("third-party/*.h"))
 
 long_description = """
 Thin Python wrapper around https://bellard.org/quickjs/ .
@@ -24,7 +27,7 @@ def get_c_sources(include_headers=False):
       author_email="petter.strandmark@gmail.com",
       name='quickjs',
       url='https://github.com/PetterS/quickjs',
-      version='1.1.1',
+      version='1.1.2',
       description='Wrapping the quickjs C library.',
       long_description=long_description,
       packages=["quickjs"],

test_quickjs.py

@@ -263,6 +263,14 @@ def test_time_limit(self):
         f.set_time_limit(-1)
         f()
 
+    def test_wrong_context(self):
+        context1 = quickjs.Context()
+        context2 = quickjs.Context()
+        f = context1.eval("(function(x) { return x.a; })")
+        d = context2.eval("({a: 1})")
+        with self.assertRaisesRegex(ValueError, "Can not mix JS objects from different contexts."):
+            f(d)
+
 
 class Strings(unittest.TestCase):
     def test_unicode(self):