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.h | |
| 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.h')
| -rw-r--r-- | core/fpdfapi/render/cpdf_dibsource.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/core/fpdfapi/render/cpdf_dibsource.h b/core/fpdfapi/render/cpdf_dibsource.h index a9f98e158d..452c347ac3 100644 --- a/core/fpdfapi/render/cpdf_dibsource.h +++ b/core/fpdfapi/render/cpdf_dibsource.h @@ -153,7 +153,10 @@ class CPDF_DIBSource : public CFX_DIBSource { RetainPtr<CPDF_DIBSource> m_pMask; RetainPtr<CPDF_StreamAcc> m_pGlobalStream; std::unique_ptr<CCodec_ScanlineDecoder> m_pDecoder; + + // Must come after |m_pCachedBitmap|. std::unique_ptr<CCodec_Jbig2Context> m_pJbig2Context; + UnownedPtr<CPDF_Stream> m_pMaskStream; LoadState m_Status = LoadState::kFail; }; |