2
Vote

Memory leaks in revision 2014-06-02

description

Hi,

I found memory leaks trying to read a JXR image coming from the conformance test suite : "JXR_ConformanceSuite\Alpha_Interleaved\Maui-32bppBGRA_64x64_Interleaved.jxr"

Attached is an extract from a memory leak detector, using the JXRDecApp.c program.
I don't know enough the library to tell you where these variables should be freed.

Hervé

file attachments

comments

drolon wrote Aug 3, 2014 at 9:14 PM

file attached

christoph_hausner wrote Aug 3, 2014 at 9:41 PM

Please test with the latest code in MSOpenTech's fork. It could be that those leaks have already been fixed there.

drolon wrote Aug 4, 2014 at 12:37 AM

Hi,

I've tried MSOpenTech's fork but nothing changed (strdec.c didn't change).

Hervé

christoph_hausner wrote Aug 4, 2014 at 1:35 AM

Sorry, should have mentioned that the changes are in a special branch.

drolon wrote Aug 5, 2014 at 6:44 PM

Hi,

Using the MSOpenTech's fork (mergejpegxr branch), the memory leaks are still here.
See attached report.

Hervé

christoph_hausner wrote Aug 6, 2014 at 12:04 PM

Thanks Hervé. Some of the memory leaks are small factory objects that don't get cleaned up in the jxrdecapp, those are rather insignificant as they are not in the library itself. The other three leaks seem to all result from the fact that the decoding context for one of the image planes (alpha?) is not freed.

I am not familiar enough with the code to know exactly how to fix it, maybe Ted or one of the maintainers can help?

drolon wrote Sep 6, 2014 at 4:28 PM

Hi,

Any news on this bug ?
The master branch wasn't updated since june 02, 2014. Is this master branch still maintained ?

Hervé

christoph_hausner wrote Sep 7, 2014 at 6:05 PM

This issue tracker does not seem to be monitored by the maintainers, unfortenuately. I know that a lot of bugs have been fixed lately in the mergejpegxr branch of MSOpenTech's fork (including better progressive decoding support) and there has been a corresponding pull request since May, but it hasn't been merged into the master branch, yet.