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 · daf1825 · daf1825
2 parents eb066da + 4fcf282
commit daf1825
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=5f8936dc7a53eea7ffe172d28fb4cf7b3bffcc64
+sha256=980cfc3158929dee7b3aae63b02eb694ee20fcc697917913f52fe30ff7c87b0b
diff --git a/build/pkgs/configure/package-version.txt b/build/pkgs/configure/package-version.txt
@@ -1 +1 @@
-b9cb7bc2559cde857d84d77f0b37a3616ce1eb6c
+6081aa813f0c8187ade0e3352905e56e8748f2e5
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