This project is read-only.

Memory leaks in revision 2014-06-02



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.


file attachments


drolon wrote Aug 3, 2014 at 10:14 PM

file attached

christoph_hausner wrote Aug 3, 2014 at 10: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 1:37 AM


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


christoph_hausner wrote Aug 4, 2014 at 2:35 AM

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

drolon wrote Aug 5, 2014 at 7:44 PM


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


christoph_hausner wrote Aug 6, 2014 at 1: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 5:28 PM


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


christoph_hausner wrote Sep 7, 2014 at 7: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.