]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
closes bpo-31903: Release the GIL when calling into SystemConfiguration (GH-4178)
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Wed, 12 Sep 2018 15:38:36 +0000 (08:38 -0700)
committerGitHub <noreply@github.com>
Wed, 12 Sep 2018 15:38:36 +0000 (08:38 -0700)
(cherry picked from commit 4859ba0d2ce4506fddc3f55f90f8dce031b3804f)

Co-authored-by: Max Bélanger <aeromax@gmail.com>
Misc/NEWS.d/next/macOS/2017-11-01-16-53-12.bpo-31903.K6jCVG.rst [new file with mode: 0644]
Modules/_scproxy.c

diff --git a/Misc/NEWS.d/next/macOS/2017-11-01-16-53-12.bpo-31903.K6jCVG.rst b/Misc/NEWS.d/next/macOS/2017-11-01-16-53-12.bpo-31903.K6jCVG.rst
new file mode 100644 (file)
index 0000000..3788112
--- /dev/null
@@ -0,0 +1,2 @@
+In :mod:`_scproxy`, drop the GIL when calling into ``SystemConfiguration`` to avoid
+deadlocks.
index 1ce4b776f3e26908353a9a1f80cb501c3cd28b74..e37515ee6552023c2690d40f0316296b1fdd3191 100644 (file)
@@ -62,7 +62,10 @@ get_proxy_settings(PyObject* mod __attribute__((__unused__)))
     PyObject* v;
     int r;
 
+    Py_BEGIN_ALLOW_THREADS
     proxyDict = SCDynamicStoreCopyProxies(NULL);
+    Py_END_ALLOW_THREADS
+
     if (!proxyDict) {
         Py_INCREF(Py_None);
         return Py_None;
@@ -173,7 +176,10 @@ get_proxies(PyObject* mod __attribute__((__unused__)))
     int r;
     CFDictionaryRef proxyDict = NULL;
 
+    Py_BEGIN_ALLOW_THREADS
     proxyDict = SCDynamicStoreCopyProxies(NULL);
+    Py_END_ALLOW_THREADS
+
     if (proxyDict == NULL) {
         return PyDict_New();
     }