prod(primes_first_n(64)).divisors() crashes

[sagetrac-asdjughewou9474388]

sage: prod(primes_first_n(64)).divisors()
------------------------------------------------------------------------
Unhandled SIGSEGV: A segmentation fault occurred in Sage.
This probably occurred because a *compiled* component of Sage has a bug
in it and is not properly wrapped with sig_on(), sig_off().
Sage will now terminate.
------------------------------------------------------------------------

The crash is due to the fact that the number has exactly 2^64 divisors and this overflows a C integer.

Component: basic arithmetic

Keywords: integer divisors

Author: Jeroen Demeyer

Branch/Commit: 8c68d72

Reviewer: Ralf Stephan

Issue created by migration from https://trac.sagemath.org/ticket/8573

[sagetrac-asdjughewou9474388]

Changed keywords from product primes 190 to product primes 190 divisors

[mezzarobba]

comment:2

It now (6.1) fails with a MemoryError, which looks reasonable to me.

[jdemeyer]

Reviewer: Marc Mezzarobba, Jeroen Demeyer

[jdemeyer]

Changed reviewer from Marc Mezzarobba, Jeroen Demeyer to none

[jdemeyer]

Changed keywords from product primes 190 divisors to integer divisors

[jdemeyer]

Branch: u/jdemeyer/ticket/8573

[jdemeyer]

Commit: 8c68d72

[jdemeyer]

New commits:

8c68d72

Add overflow check to divisors()

[jdemeyer]

Author: Jeroen Demeyer

[rwst]

Changed branch from u/jdemeyer/ticket/8573 to u/rws/ticket/8573

[rwst]

comment:11

Rebased on 6.2.beta3. Tests OK --long in rings/. Seems to be an uncomplicated change.

---

New commits:

7e1f981

Merge branch 'u/jdemeyer/ticket/8573' of git://trac.sagemath.org/sage into ticket/8573

[rwst]

Changed commit from 8c68d72 to 7e1f981

[rwst]

Reviewer: Ralf Stephan

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:

42a405b

Merge branch 'develop' (6.2.beta4) into ticket/8573

[sagetrac-git]

Changed commit from 7e1f981 to 42a405b

[jdemeyer]

Changed commit from 42a405b to 8c68d72

[jdemeyer]

comment:13

Why change the branch all the time?

---

New commits:

8c68d72

Add overflow check to divisors()

[jdemeyer]

Changed branch from u/rws/ticket/8573 to u/jdemeyer/ticket/8573

[rwst]

comment:14

Replying to @jdemeyer:

Why change the branch all the time?

https://groups.google.com/forum/#!topic/sage-devel/sTLT83d1g14

[jdemeyer]

comment:15

In this ticket, you made no changes, so there is no reason at all to commit/push anything. I'm not complaining about changing the branch, I am complaining about changing the branch without making any changes.

[vbraun]

Changed branch from u/jdemeyer/ticket/8573 to 8c68d72