[Rev 02] RFR: 8212034: Potential memory leaks in jpegLoader.c in error case
Ambarish Rapte
arapte at openjdk.java.net
Mon Feb 10 13:27:27 UTC 2020
> Memory allocated in initDecompressor() and decompressIndirect() is not freed in error case.
> In error case,
> 1. Allocated memory should be freed.
> 2. Appropriate de-initialization jpeg library calls should be added.
>
> Verified that,
> 1. All unit and systems tests pass on three platforms, and
> 2. Memory consumption with and without fix is similar by comparing memory before and after showing 10 jpeg images for 100 times.
The pull request has been updated with a new target base due to a merge or a rebase.
-------------
Commits:
- b91e4236: Merge with master
- 8a48080a: correcting use of scanline_ptr
- 7af932b7: 8212034: Memory leaks in jpegLoader.c in error case
Changes: https://git.openjdk.java.net/jfx/pull/54/files
Webrev: https://webrevs.openjdk.java.net/jfx/54/webrev.02
Stats: 59 lines in 1 file changed: 33 ins; 15 del; 11 mod
Patch: https://git.openjdk.java.net/jfx/pull/54.diff
Fetch: git fetch https://git.openjdk.java.net/jfx pull/54/head:pull/54
PR: https://git.openjdk.java.net/jfx/pull/54
More information about the openjfx-dev
mailing list