search

lucasb-eyer / libheatmap

High performance C heatmap generation library. Supposed to be wrapped by higher-level languages.
MIT License
302 stars 60 forks source link

there is a small memory leak in examples/show_colorschemes #16

Closed rodan closed 1 year ago

rodan commented 1 year ago

==6332== 484 bytes in 1 blocks are indirectly lost in loss record 2 of 3 ==6332== at 0x48465EF: calloc (vg_replace_malloc.c:1340) ==6332== by 0x1252A5: heatmap_stamp_gen_nonlinear (heatmap.c:270) ==6332== by 0x125274: heatmap_stamp_gen (heatmap.c:262) ==6332== by 0x10D618: main (show_colorschemes.cpp:196) ==6332== ==6332== 500 (16 direct, 484 indirect) bytes in 1 blocks are definitely lost in loss record 3 of 3 ==6332== at 0x48417B5: malloc (vg_replace_malloc.c:393) ==6332== by 0x1251BF: heatmap_stamp_new_with (heatmap.c:243) ==6332== by 0x125429: heatmap_stamp_gen_nonlinear (heatmap.c:288) ==6332== by 0x125274: heatmap_stamp_gen (heatmap.c:262) ==6332== by 0x10D618: main (show_colorschemes.cpp:196) ==6332== ==6332== LEAK SUMMARY: ==6332== definitely lost: 16 bytes in 1 blocks ==6332== indirectly lost: 484 bytes in 1 blocks ==6332== possibly lost: 0 bytes in 0 blocks ==6332== still reachable: 8 bytes in 1 blocks ==6332== suppressed: 0 bytes in 0 blocks

rodan commented 1 year ago

I have no idea why my second commit on my fork got added to this pull request, it was not intentional. the memory leak is fixed by the one liner from the first commit.