diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2015-10-26 16:14:23 -0400 |
|---|---|---|
| committer | Dan Sinclair <dsinclair@chromium.org> | 2015-10-26 16:14:23 -0400 |
| commit | 85361b227ad6786d2aeef8409b79a8d077a26ee9 (patch) | |
| tree | fe1ecb088ee13e6a8104183d4777d8fd3e216771 /core/include | |
| parent | 6ee3484bcb413f4cce303e237fe15e2bdaf51ade (diff) | |
| download | pdfium-85361b227ad6786d2aeef8409b79a8d077a26ee9.tar.xz | |
Make m_pShadingObj a CPDF_Stream instead of CPDF_Object.
This object is required to be a stream and was being converted as such. With
the new type checking this caused us to pass a nullptr where previously we'd
have, incorrectly, cast a CPDF_Dictionary to a CPDF_Stream.
This CL changes the m_pShadingObj to always be a CPDF_Stream. Then, we never
go down the bad code path because we check if m_pShadingObj is nullptr earlier
and bail out.
BUG=chromium:547706
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1426713002 .
Diffstat (limited to 'core/include')
| -rw-r--r-- | core/include/fpdfapi/fpdf_resource.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/include/fpdfapi/fpdf_resource.h b/core/include/fpdfapi/fpdf_resource.h index 9fe6e2b95f..0b85333cfc 100644 --- a/core/include/fpdfapi/fpdf_resource.h +++ b/core/include/fpdfapi/fpdf_resource.h @@ -688,7 +688,7 @@ class CPDF_ShadingPattern : public CPDF_Pattern { ~CPDF_ShadingPattern() override; - CPDF_Object* m_pShadingObj; + CPDF_Stream* m_pShadingObj; FX_BOOL m_bShadingObj; |