Opened 10 years ago

Closed 10 years ago

#14067 closed defect (fixed)

generic_graph_pyx.pyx invalid free() with MALLOC_CHECK_=3

Reported by: Jeroen Demeyer Owned by: Robert Miller
Priority: blocker Milestone: sage-5.7
Component: memleak Keywords:
Cc: Jean-Pierre Flori, Simon King Merged in: sage-5.7.beta4
Authors: Jeroen Demeyer Reviewers: Jean-Pierre Flori
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Jeroen Demeyer)

$ env MALLOC_CHECK_=3 ./sage -c 'from sage.graphs.generic_graph_pyx import SubgraphSearch; SubgraphSearch(Graph(5),Graph(1))'
*** glibc detected *** python: free(): invalid pointer: 0x00000000030838e0 ***

Crash log: sage_crash_SHRqxx.log

Attachments (2)

sage_crash_SHRqxx.log (13.0 KB) - added by Jeroen Demeyer 10 years ago.
14067_invalid_free.patch (2.2 KB) - added by Jeroen Demeyer 10 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 10 years ago by Jeroen Demeyer

Cc: Jean-Pierre Flori Simon King added

comment:2 Changed 10 years ago by Jeroen Demeyer

Description: modified (diff)

comment:3 Changed 10 years ago by Jeroen Demeyer

Description: modified (diff)

Changed 10 years ago by Jeroen Demeyer

Attachment: sage_crash_SHRqxx.log added

comment:4 Changed 10 years ago by Jeroen Demeyer

Description: modified (diff)

comment:5 Changed 10 years ago by Jeroen Demeyer

Description: modified (diff)

comment:6 Changed 10 years ago by Jean-Pierre Flori

The problem occurs when deallocating self.stack.

comment:7 Changed 10 years ago by Jeroen Demeyer

Got it, we are assigning self.stack[1] even if self.stack is an array with just one element.

Changed 10 years ago by Jeroen Demeyer

Attachment: 14067_invalid_free.patch added

comment:8 Changed 10 years ago by Jeroen Demeyer

Authors: Jeroen Demeyer
Status: newneeds_review

comment:9 Changed 10 years ago by Jean-Pierre Flori

Reviewers: Jean-Pierre Flori
Status: needs_reviewpositive_review

Works well.

comment:10 Changed 10 years ago by Jeroen Demeyer

Merged in: sage-5.7.beta4
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.