123050 - Memory Leak in nsGIFDecoder2.cpp, HaveDecodedRow

Status: VERIFIED INVALID

(Core :: Graphics: ImageLib, defect)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

According to the above URL, there's a memory leak in the GIF Decoder.
Stack:
__builtin_new new1.cc:77 libxpcom.so 0x403bacb3
__builtin_vec_new new2.cc:39 libxpcom.so 0x403baeaa
nsImageGTK::Init <...> nsImageGTK.cpp:183 libgfx_gtk.so 0x7f7b36ea 
gfxImageFrame::Init <...> gfxImageFrame.cpp:105 libgfx2.so 0x8026bbe6 
HaveDecodedRow <...> nsGIFDecoder2.cpp:375 libimggif.so 0x80943b58 
output_row(gif_struct *) GIF2.cpp:213 libimggif.so 0x8094096c
do_lzw <...> GIF2.cpp:412 libimggif.so 0x80940e5a
gif_write <...> GIF2.cpp:838 libimggif.so 0x80941550
nsGIFDecoder2::ProcessData <...> nsGIFDecoder2.cpp:199 libimggif.so 0x80943543 
ReadDataOut <...> nsGIFDecoder2.cpp:149 libimggif.so 0x80943208

This leak happens 3 times; 109 times the memory is freed.

Comment 1

[Alfred Kayser]

The link mentioned above is no longer valid.
Furthermore, the image code beneath the decoders is replaced by Cairo, so this unconfirmed leak is no longer there in that form.