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
Lint Skipped
Unit
Unit Tests Skipped
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.