Fix memory leak in CMS
ClosedPublic

Authored by sandsmark on Mar 27 2020, 1:55 PM.

Details

Summary

Quoting iccjpeg.h:

* IMPORTANT: the data at **icc_data_ptr has been allocated with malloc()
* and must be freed by the caller with free() when the caller no longer
* needs it.  (Alternatively, we could write this routine to use the
* IJG library's memory allocator, so that the data would be freed implicitly
* at jpeg_finish_decompress() time.  But it seems likely that many apps
* will prefer to have the data stick around after decompression finishes.)
Test Plan

Not reported by asan anymore

Diff Detail

Repository
R260 Gwenview
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sandsmark requested review of this revision.Mar 27 2020, 1:55 PM
sandsmark created this revision.

Again, fairly trivial, I hope.

ngraham accepted this revision.Apr 14 2020, 3:27 PM

Yup.

This revision is now accepted and ready to land.Apr 14 2020, 3:27 PM
This revision was automatically updated to reflect the committed changes.