[Rev 02] RFR: 8212034: Potential memory leaks in jpegLoader.c in error case
arapte at openjdk.java.net
Tue Feb 11 06:37:43 UTC 2020
On Fri, 7 Feb 2020 10:06:59 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:
>> In general, this makes sense. I need to look into more detail that the additional calls for freeing resources (in case of errors) don't cause e.g. segmentation violations and lead to a crash -- which would be worse than throwing an Exception.
>> I expect memory consumption to be similar before and after this patch if you don't run into errors, but did you check memory consumption before/after this patch in case of errors?
> The PR was outdated. It is now merged with latest master.
> Please take a look.
@johanvos , @arajkumar
Could you please take a re-look at the updated PRs.
More information about the openjfx-dev