diff options
author | Bo Xu <bo_xu@foxitsoftware.com> | 2014-08-21 15:19:00 -0700 |
---|---|---|
committer | Bo Xu <bo_xu@foxitsoftware.com> | 2014-09-24 16:55:01 -0700 |
commit | 93978601e9c00f27b34ddbb6e1ecc06df700b49a (patch) | |
tree | dd4cfa746fa011f435d0119b11d4b73434b35b52 | |
parent | 76ba4aab69627f4f6242832ea3296e1efc2247c2 (diff) | |
download | pdfium-93978601e9c00f27b34ddbb6e1ecc06df700b49a.tar.xz |
For pattern colorspace, its base colorspace should not be a pattern colorspace
BUG=405588
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/496883002
-rw-r--r-- | core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp b/core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp index da48093135..c6a4f49e47 100644 --- a/core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp +++ b/core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp @@ -843,6 +843,9 @@ FX_BOOL CPDF_PatternCS::v_Load(CPDF_Document* pDoc, CPDF_Array* pArray) CPDF_DocPageData* pDocPageData = pDoc->GetPageData(); m_pBaseCS = pDocPageData->GetColorSpace(pBaseCS, NULL); if (m_pBaseCS) { + if (m_pBaseCS->GetFamily() == PDFCS_PATTERN) { + return FALSE; + } m_nComponents = m_pBaseCS->CountComponents() + 1; if (m_pBaseCS->CountComponents() > MAX_PATTERN_COLORCOMPS) { return FALSE; @@ -855,9 +858,11 @@ FX_BOOL CPDF_PatternCS::v_Load(CPDF_Document* pDoc, CPDF_Array* pArray) FX_BOOL CPDF_PatternCS::GetRGB(FX_FLOAT* pBuf, FX_FLOAT& R, FX_FLOAT& G, FX_FLOAT& B) const { if (m_pBaseCS) { + ASSERT(m_pBaseCS->GetFamily() != PDFCS_PATTERN); PatternValue* pvalue = (PatternValue*)pBuf; - m_pBaseCS->GetRGB(pvalue->m_Comps, R, G, B); - return TRUE; + if (m_pBaseCS->GetRGB(pvalue->m_Comps, R, G, B)) { + return TRUE; + } } R = G = B = 0.75f; return FALSE; |