sagemathgh-38628: Fix hypellfrob.pyx calling Python inside sig_on

set_ntl_matrix_modn_dense uses isinstance so there is a chance for Python garbage collection to run. Presumably you get the SystemError: calling remove_from_pari_stack() inside sig_on() if that happens. Fixes sagemath#33304 URL: sagemath#38628 Reported by: Volker Braun Reviewer(s):
vbraun · Sep 6, 2024 · 6081aa8 · 6081aa8
2 parents eb066da + 4fcf282
commit 6081aa8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/build/pkgs/configure/checksums.ini b/build/pkgs/configure/checksums.ini
@@ -1,3 +1,3 @@
 tarball=configure-VERSION.tar.gz
-sha1=0f6355fc136bb6619585863b9e4bc954cc6e0e3d
-sha256=5b618581d51997afa78b5e6647584f7ef4e6d5844823dd44e607f2accd7abba5
+sha1=29f9d73909ebb57e1dce82fd9eee09707171b0cf
+sha256=02d0d43bb7c272412ceef2b414d01770bb639c99557d5dcc35568fb9ca368f21
diff --git a/build/pkgs/configure/package-version.txt b/build/pkgs/configure/package-version.txt
@@ -1 +1 @@
-b9cb7bc2559cde857d84d77f0b37a3616ce1eb6c
+cf40a329a3e13b5ed39788bf705c3480a551e82e
diff --git a/src/sage/schemes/hyperelliptic_curves/hypellfrob.pyx b/src/sage/schemes/hyperelliptic_curves/hypellfrob.pyx
@@ -127,8 +127,10 @@ def interval_products(M0, M1, target):
     cdef long dim = M0.nrows()
     sig_on()
     c.restore_c()
+    sig_off()
     set_ntl_matrix_modn_dense(mm0, c, M0)
     set_ntl_matrix_modn_dense(mm1, c, M1)
+    sig_on()
     for t in target:
         targ.push_back(ntl_ZZ(t).x)
     numintervals = len(target)/2