| commit | 4485263fee62e0f3ee57dcb254921182ae33343a | [log] [tgz] |
|---|---|---|
| author | Lei Zhang <thestig@chromium.org> | Tue Mar 29 00:31:14 2022 +0000 |
| committer | Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue Mar 29 00:31:14 2022 +0000 |
| tree | 7ea043e4ab22cbab6ff8aaff8bca2c404a016880 | |
| parent | 2393614002f553e78fb4bde58c77e5eec0c272ce [diff] |
Fix memory leak in CFX_FontMapper::RawBytesForIndex(). Use the newly added ScopedFontDeleter to call DeleteFont(). Bug: chromium:1310508 Change-Id: I968193652fb32f786a6736d6c3f74a9f6e79b1a2 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/91931 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/core/fxge/cfx_fontmapper.cpp b/core/fxge/cfx_fontmapper.cpp index b85031c..02fbbb5 100644 --- a/core/fxge/cfx_fontmapper.cpp +++ b/core/fxge/cfx_fontmapper.cpp
@@ -739,6 +739,7 @@ if (!hFont) return nullptr; + ScopedFontDeleter scoped_font(m_pFontInfo.get(), hFont); size_t required_size = m_pFontInfo->GetFontData(hFont, 0, {}); if (required_size == 0) return nullptr;