Re: RFR: 8309508: Possible memory leak in JPEG image loader

2023-06-12 Thread Ambarish Rapte
On Fri, 9 Jun 2023 05:18:16 GMT, Jayathirth D V wrote: > On code inspection it is revealed that in jpegloader.c->decompressIndirect() > we are allocating memory for "scanline_ptr", but if we error_exit() from > jpeg_read_scanlines() we are not releasing this memory. > > Added release of "scanl

Re: RFR: 8309508: Possible memory leak in JPEG image loader

2023-06-09 Thread Kevin Rushforth
On Fri, 9 Jun 2023 05:18:16 GMT, Jayathirth D V wrote: > On code inspection it is revealed that in jpegloader.c->decompressIndirect() > we are allocating memory for "scanline_ptr", but if we error_exit() from > jpeg_read_scanlines() we are not releasing this memory. > > Added release of "scanl

RFR: 8309508: Possible memory leak in JPEG image loader

2023-06-08 Thread Jayathirth D V
On code inspection it is revealed that in jpegloader.c->decompressIndirect() we are allocating memory for "scanline_ptr", but if we error_exit() from jpeg_read_scanlines() we are not releasing this memory. Added release of "scanline_ptr" with NULL check at appropriate places. As part of this fix