]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
gh-94637: Release GIL in SSLContext.set_default_verify_paths (GH-94658)
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Sat, 9 Jul 2022 16:35:54 +0000 (09:35 -0700)
committerGitHub <noreply@github.com>
Sat, 9 Jul 2022 16:35:54 +0000 (09:35 -0700)
(cherry picked from commit 78307c7dc2352b6633138466debd4c10fae32970)

Co-authored-by: Christian Heimes <christian@python.org>
Misc/NEWS.d/next/Library/2022-07-07-15-46-55.gh-issue-94637.IYEiUM.rst [new file with mode: 0644]
Modules/_ssl.c

diff --git a/Misc/NEWS.d/next/Library/2022-07-07-15-46-55.gh-issue-94637.IYEiUM.rst b/Misc/NEWS.d/next/Library/2022-07-07-15-46-55.gh-issue-94637.IYEiUM.rst
new file mode 100644 (file)
index 0000000..20cbbcd
--- /dev/null
@@ -0,0 +1,3 @@
+:meth:`SSLContext.set_default_verify_paths` now releases the GIL around
+``SSL_CTX_set_default_verify_paths`` call. The function call performs I/O
+and CPU intensive work.
index c8500c57aee757d08e06bbaabb83ff8f7ee27644..51bb2d00e3e8a5e129d33d7866fba82b761ec856 100644 (file)
@@ -4305,7 +4305,11 @@ static PyObject *
 _ssl__SSLContext_set_default_verify_paths_impl(PySSLContext *self)
 /*[clinic end generated code: output=0bee74e6e09deaaa input=35f3408021463d74]*/
 {
-    if (!SSL_CTX_set_default_verify_paths(self->ctx)) {
+    int rc;
+    Py_BEGIN_ALLOW_THREADS
+    rc = SSL_CTX_set_default_verify_paths(self->ctx);
+    Py_END_ALLOW_THREADS
+    if (!rc) {
         _setSSLError(get_state_ctx(self), NULL, 0, __FILE__, __LINE__);
         return NULL;
     }