Opened 13 years ago

Closed 13 years ago

#2497 closed defect (fixed)

[with patch, positive review] crash in polynomial remainder

Reported by: dmharvey Owned by: somebody
Priority: major Milestone: sage-3.0
Component: basic arithmetic Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description 

sage: R.<x> = PolynomialRing(Integers(4))
sage: f = x^2 + 3
sage: f % 2
InvMod: inverse undefined
/Users/david/sage/local/bin/sage-sage: line 222: 11351 Abort trap              sage-ipython "$@" -c "$SAGE_STARTUP_COMMAND;"

This is an NTL error message, which is not being trapped or something.

Attachments (1)

2497.patch​ (1.5 KB) - added by dmharvey 13 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 13 years ago by dfdeshom

This also happens in __florrdiv__(), quo_rem()

comment:2 Changed 13 years ago by gfurnish

  • Milestone set to sage-2.11

Changed 13 years ago by dmharvey

comment:3 Changed 13 years ago by mabshoff

  • Summary changed from crash in polynomial remainder to [with patch, needs review] crash in polynomial remainder

comment:4 Changed 13 years ago by mabshoff

#2592 seems related and is not fixed by this patch.

Cheers,

Michael

comment:5 Changed 13 years ago by mabshoff

  • Summary changed from [with patch, needs review] crash in polynomial remainder to [with patch, positive review] crash in polynomial remainder

The patch fixes the issue, adds a doctest for the crash and passes doctests. Positive review.

Cheers,

Michaek

comment:6 Changed 13 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in Sage 3.0.alpha2

Note: See TracTickets for help on using tickets.