CJPEGDecoder::Reset leaks memory if previously initialized

[Steve_Browne]

This bug has been in the IPP sample code since 6.0.1 or earlier. Its definitely still there in the IPP 7 sample code as well. Basically the Reset call on CJPEGDecoder zeroes everything out but doesn't free any of the allocated buffers. Obviously that's the goal of Reset and Clean would be the more appropriate call, but Reset is public and Clean is not. That's probably whyMJPEGVideoDecoder uses Reset rather than Clean. See umc_mjpeg_video_decoder.cpp line 229 which calls its own Reset which in turn calls CJPEGDecoder::Reset causing the leak.

[Chao_Y_Intel]

Hello,

Thanks for your report. I looked that code. It looks that both the Clean and Reset function in the CJPEGDecoder needs to be called to reset the status. I will track this problem, so it will be reviewed by the engineer owner for future fix.

Thanks,
Chao

[Chao_Y_Intel]

Hello,

Could you check IPP 7.1 Beta release? This problem were planed to be fixed since this release.

Thanks,
Cho