diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-05-10 17:05:26 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-05-10 17:05:26 +0000 |
| commit | 2db7eda9f51a4aa19df529ab12530542513c22a1 (patch) | |
| tree | ed92f627dcf4862dab0546a771b7f633305b41d7 /core/fpdfapi/render/cpdf_dibsource.cpp | |
| parent | 401e618415d424f8a8b48f15e6710fa0e75d0615 (diff) | |
| download | pdfium-2db7eda9f51a4aa19df529ab12530542513c22a1.tar.xz | |
Fix destruction order in CPDF_Dibsource.
The order of the elements in the header is correct, but we
were clearing it early in the destructor itself.
Bug: 840695
Change-Id: I1585722fed8dc672ffd5e1dc9a1eea1c7c80f310
Reviewed-on: https://pdfium-review.googlesource.com/32311
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'core/fpdfapi/render/cpdf_dibsource.cpp')
| -rw-r--r-- | core/fpdfapi/render/cpdf_dibsource.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/core/fpdfapi/render/cpdf_dibsource.cpp b/core/fpdfapi/render/cpdf_dibsource.cpp index bbf91da868..c76ae82bce 100644 --- a/core/fpdfapi/render/cpdf_dibsource.cpp +++ b/core/fpdfapi/render/cpdf_dibsource.cpp @@ -106,7 +106,6 @@ CPDF_DIBSource::CPDF_DIBSource() {} CPDF_DIBSource::~CPDF_DIBSource() { FX_Free(m_pMaskedLine); FX_Free(m_pLineBuf); - m_pCachedBitmap.Reset(); // TODO(tsepez): determine if required early here. FX_Free(m_pCompData); if (m_pColorSpace && m_pDocument) { auto* pPageData = m_pDocument->GetPageData(); |