Ticket #651 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

[with patch] memory leak in gmp_globals.c

Reported by: mabshoff Owned by: mabshoff
Priority: major Milestone: sage-2.8.5
Component: memleak Keywords:
Cc: Author(s):
Report Upstream: Reviewer(s):
Merged in: Work issues:

Description

In gmp_globals.c we init tmp again instead of clearing it. This leaks about 16 bytes. For a patch see

 http://fsmath.mathematik.uni-dortmund.de/~mabshoff/patches/Sage-2.8.4.2-fix-small-mpq-leak-in-gmp_globals.c.patch

Without the patch:

==32229== LEAK SUMMARY:
==32229==    definitely lost: 2,548 bytes in 7 blocks.
==32229==      possibly lost: 364,814 bytes in 1,127 blocks.
==32229==    still reachable: 137,021,540 bytes in 18,327 blocks.
==32229==         suppressed: 0 bytes in 0 blocks.

With the patch:

==14532== LEAK SUMMARY:
==14532==    definitely lost: 2,532 bytes in 5 blocks.
==14532==      possibly lost: 364,878 bytes in 1,128 blocks.
==14532==    still reachable: 137,021,460 bytes in 18,324 blocks.
==14532==         suppressed: 0 bytes in 0 blocks.

Every byte counts ;)

Cheers,

Michael

Change History

Changed 3 years ago by mabshoff

  • status changed from new to assigned

Changed 3 years ago by was

  • status changed from assigned to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.