summaryrefslogtreecommitdiff
path: root/core/fpdfapi
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2017-03-17 11:16:18 -0700
committerChromium commit bot <commit-bot@chromium.org>2017-03-17 22:11:51 +0000
commitac6e2a059dbd74f6f9f1c216600496cfa5676387 (patch)
treee844921177b0f7ba09c2a3c34ead15b0c24b4000 /core/fpdfapi
parent7a1220dc1c0051f2a6bf50f3f38419ae51ecb9a1 (diff)
downloadpdfium-ac6e2a059dbd74f6f9f1c216600496cfa5676387.tar.xz
Bring CPDF_ICCBasedCS closer to PDF spec.
The spec says the N dictionary field is required and must be set to a valid value. Adjust the code based on this assertion. BUG=pdfium:675,chromium:691967,chromium:702238 Change-Id: Iaa76fa0e16ce4aaa9822ad471668cbf8af5fb7cb Reviewed-on: https://pdfium-review.googlesource.com/3112 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Nicolás Peña <npm@chromium.org>
Diffstat (limited to 'core/fpdfapi')
-rw-r--r--core/fpdfapi/page/cpdf_colorspace.cpp118
-rw-r--r--core/fpdfapi/page/fpdf_page_colors.cpp23
-rw-r--r--core/fpdfapi/page/pageint.h11
3 files changed, 83 insertions, 69 deletions
diff --git a/core/fpdfapi/page/cpdf_colorspace.cpp b/core/fpdfapi/page/cpdf_colorspace.cpp
index 7a85284ca7..1aaa1440a4 100644
--- a/core/fpdfapi/page/cpdf_colorspace.cpp
+++ b/core/fpdfapi/page/cpdf_colorspace.cpp
@@ -153,13 +153,14 @@ class CPDF_ICCBasedCS : public CPDF_ColorSpace {
int image_height,
bool bTransMask) const override;
- bool IsSRGB() const { return m_pProfile->m_bsRGB; }
+ bool IsSRGB() const { return m_pProfile->IsSRGB(); }
private:
// If no valid ICC profile or using sRGB, try looking for an alternate.
bool FindAlternateProfile(CPDF_Document* pDoc, CPDF_Dictionary* pDict);
void UseStockAlternateProfile();
+ bool IsValidComponents(int32_t nComps) const;
void PopulateRanges(CPDF_Dictionary* pDict);
CFX_MaybeOwned<CPDF_ColorSpace> m_pAlterCS;
@@ -784,43 +785,64 @@ bool CPDF_ICCBasedCS::v_Load(CPDF_Document* pDoc, CPDF_Array* pArray) {
if (!pStream)
return false;
+ // The PDF 1.7 spec says the number of components must be valid. While some
+ // PDF viewers tolerate invalid values, Acrobat does not, so be consistent
+ // with Acrobat and reject bad values.
+ CPDF_Dictionary* pDict = pStream->GetDict();
+ int32_t nDictComponents = pDict ? pDict->GetIntegerFor("N") : 0;
+ if (!IsValidComponents(nDictComponents))
+ return false;
+
+ m_nComponents = nDictComponents;
m_pProfile = pDoc->LoadIccProfile(pStream);
if (!m_pProfile)
return false;
- // Try using the |nComponents| from ICC profile
- m_nComponents = m_pProfile->GetComponents();
- CPDF_Dictionary* pDict = pStream->GetDict();
- if (!m_pProfile->m_pTransform && !FindAlternateProfile(pDoc, pDict))
+ // The PDF 1.7 spec also says the number of components in the ICC profile
+ // must match the N value. However, that assumes the viewer actually
+ // understands the ICC profile.
+ // If the valid ICC profile has a mismatch, fail.
+ if (m_pProfile->IsValid() && m_pProfile->GetComponents() != m_nComponents)
return false;
+ // If PDFium does not understand the ICC profile format at all, or if it's
+ // SRGB, a profile PDFium recognizes but does not support well, then try the
+ // alternate profile.
+ if (!m_pProfile->IsSupported() && !FindAlternateProfile(pDoc, pDict)) {
+ // If there is no alternate profile, use a stock profile as mentioned in
+ // the PDF 1.7 spec in table 4.16 in the "Alternate" key description.
+ UseStockAlternateProfile();
+ }
+
PopulateRanges(pDict);
return true;
}
bool CPDF_ICCBasedCS::GetRGB(float* pBuf, float* R, float* G, float* B) const {
- if (m_pProfile && m_pProfile->m_bsRGB) {
+ ASSERT(m_pProfile);
+ if (IsSRGB()) {
*R = pBuf[0];
*G = pBuf[1];
*B = pBuf[2];
return true;
}
- CCodec_IccModule* pIccModule = CPDF_ModuleMgr::Get()->GetIccModule();
- if (!m_pProfile->m_pTransform || !pIccModule) {
- if (m_pAlterCS)
- return m_pAlterCS->GetRGB(pBuf, R, G, B);
-
- *R = 0.0f;
- *G = 0.0f;
- *B = 0.0f;
+ if (m_pProfile->transform()) {
+ float rgb[3];
+ CCodec_IccModule* pIccModule = CPDF_ModuleMgr::Get()->GetIccModule();
+ pIccModule->SetComponents(m_nComponents);
+ pIccModule->Translate(m_pProfile->transform(), pBuf, rgb);
+ *R = rgb[0];
+ *G = rgb[1];
+ *B = rgb[2];
return true;
}
- float rgb[3];
- pIccModule->SetComponents(m_nComponents);
- pIccModule->Translate(m_pProfile->m_pTransform, pBuf, rgb);
- *R = rgb[0];
- *G = rgb[1];
- *B = rgb[2];
+
+ if (m_pAlterCS)
+ return m_pAlterCS->GetRGB(pBuf, R, G, B);
+
+ *R = 0.0f;
+ *G = 0.0f;
+ *B = 0.0f;
return true;
}
@@ -851,16 +873,16 @@ void CPDF_ICCBasedCS::TranslateImageLine(uint8_t* pDestBuf,
int image_width,
int image_height,
bool bTransMask) const {
- if (m_pProfile->m_bsRGB) {
+ if (IsSRGB()) {
ReverseRGB(pDestBuf, pSrcBuf, pixels);
- } else if (m_pProfile->m_pTransform) {
+ } else if (m_pProfile->transform()) {
int nMaxColors = 1;
for (uint32_t i = 0; i < m_nComponents; i++) {
nMaxColors *= 52;
}
if (m_nComponents > 3 || image_width * image_height < nMaxColors * 3 / 2) {
CPDF_ModuleMgr::Get()->GetIccModule()->TranslateScanline(
- m_pProfile->m_pTransform, pDestBuf, pSrcBuf, pixels);
+ m_pProfile->transform(), pDestBuf, pSrcBuf, pixels);
} else {
if (!m_pCache) {
((CPDF_ICCBasedCS*)this)->m_pCache = FX_Alloc2D(uint8_t, nMaxColors, 3);
@@ -876,7 +898,7 @@ void CPDF_ICCBasedCS::TranslateImageLine(uint8_t* pDestBuf,
}
}
CPDF_ModuleMgr::Get()->GetIccModule()->TranslateScanline(
- m_pProfile->m_pTransform, m_pCache, temp_src, nMaxColors);
+ m_pProfile->transform(), m_pCache, temp_src, nMaxColors);
FX_Free(temp_src);
}
for (int i = 0; i < pixels; i++) {
@@ -899,45 +921,19 @@ void CPDF_ICCBasedCS::TranslateImageLine(uint8_t* pDestBuf,
bool CPDF_ICCBasedCS::FindAlternateProfile(CPDF_Document* pDoc,
CPDF_Dictionary* pDict) {
- CPDF_Object* pAlterCSObj =
- pDict ? pDict->GetDirectObjectFor("Alternate") : nullptr;
- if (!pAlterCSObj) {
- UseStockAlternateProfile();
- return true;
- }
+ CPDF_Object* pAlterCSObj = pDict->GetDirectObjectFor("Alternate");
+ if (!pAlterCSObj)
+ return false;
auto pAlterCS = CPDF_ColorSpace::Load(pDoc, pAlterCSObj);
- if (!pAlterCS) {
- UseStockAlternateProfile();
- return true;
- }
-
- if (m_nComponents) {
- ASSERT(IsSRGB()); // Using sRGB case.
- if (pAlterCS->CountComponents() == m_nComponents)
- m_pAlterCS = std::move(pAlterCS);
- else
- UseStockAlternateProfile();
- return true;
- }
-
- // NO valid ICC profile
- if (pAlterCS->CountComponents() > 0) {
- // Use Alternative colorspace
- m_nComponents = pAlterCS->CountComponents();
- m_pAlterCS = std::move(pAlterCS);
- return true;
- }
+ if (!pAlterCS)
+ return false;
- int32_t nDictComponents = pDict ? pDict->GetIntegerFor("N") : 0;
- if (nDictComponents == 1 || nDictComponents == 3 || nDictComponents == 4) {
- m_nComponents = nDictComponents;
- UseStockAlternateProfile();
- return true;
- }
+ if (pAlterCS->CountComponents() != m_nComponents)
+ return false;
- // No valid alternative colorspace
- return false;
+ m_pAlterCS = std::move(pAlterCS);
+ return true;
}
void CPDF_ICCBasedCS::UseStockAlternateProfile() {
@@ -950,6 +946,10 @@ void CPDF_ICCBasedCS::UseStockAlternateProfile() {
m_pAlterCS = GetStockCS(PDFCS_DEVICECMYK);
}
+bool CPDF_ICCBasedCS::IsValidComponents(int32_t nComps) const {
+ return nComps == 1 || nComps == 3 || nComps == 4;
+}
+
void CPDF_ICCBasedCS::PopulateRanges(CPDF_Dictionary* pDict) {
CPDF_Array* pRanges = pDict->GetArrayFor("Range");
m_pRanges = FX_Alloc2D(float, m_nComponents, 2);
diff --git a/core/fpdfapi/page/fpdf_page_colors.cpp b/core/fpdfapi/page/fpdf_page_colors.cpp
index 061aae807a..e69620ea9a 100644
--- a/core/fpdfapi/page/fpdf_page_colors.cpp
+++ b/core/fpdfapi/page/fpdf_page_colors.cpp
@@ -25,6 +25,11 @@ float NormalizeChannel(float fVal) {
return std::min(std::max(fVal, 0.0f), 1.0f);
}
+bool DetectSRGB(const uint8_t* pData, uint32_t dwSize) {
+ return dwSize == 3144 &&
+ FXSYS_memcmp(pData + 0x190, "sRGB IEC61966-2.1", 17) == 0;
+}
+
} // namespace
uint32_t ComponentsForFamily(int family) {
@@ -214,16 +219,20 @@ void CPDF_DeviceCS::TranslateImageLine(uint8_t* pDestBuf,
}
CPDF_IccProfile::CPDF_IccProfile(const uint8_t* pData, uint32_t dwSize)
- : m_bsRGB(false), m_pTransform(nullptr), m_nSrcComponents(0) {
- if (dwSize == 3144 &&
- FXSYS_memcmp(pData + 0x190, "sRGB IEC61966-2.1", 17) == 0) {
- m_bsRGB = true;
+ : m_bsRGB(DetectSRGB(pData, dwSize)) {
+ if (m_bsRGB) {
m_nSrcComponents = 3;
- } else if (CPDF_ModuleMgr::Get()->GetIccModule()) {
- m_pTransform = CPDF_ModuleMgr::Get()->GetIccModule()->CreateTransform_sRGB(
- pData, dwSize, m_nSrcComponents);
+ return;
+ }
+ auto* pIccModule = CPDF_ModuleMgr::Get()->GetIccModule();
+ if (pIccModule) {
+ uint32_t nSrcComps = 0;
+ m_pTransform = pIccModule->CreateTransform_sRGB(pData, dwSize, nSrcComps);
+ if (m_pTransform)
+ m_nSrcComponents = nSrcComps;
}
}
+
CPDF_IccProfile::~CPDF_IccProfile() {
if (m_pTransform) {
CPDF_ModuleMgr::Get()->GetIccModule()->DestroyTransform(m_pTransform);
diff --git a/core/fpdfapi/page/pageint.h b/core/fpdfapi/page/pageint.h
index 0737c2ed6e..9637155343 100644
--- a/core/fpdfapi/page/pageint.h
+++ b/core/fpdfapi/page/pageint.h
@@ -138,12 +138,17 @@ class CPDF_IccProfile {
public:
CPDF_IccProfile(const uint8_t* pData, uint32_t dwSize);
~CPDF_IccProfile();
+
+ bool IsValid() const { return IsSRGB() || IsSupported(); }
+ bool IsSRGB() const { return m_bsRGB; }
+ bool IsSupported() const { return !!m_pTransform; }
+ void* transform() { return m_pTransform; }
uint32_t GetComponents() const { return m_nSrcComponents; }
- bool m_bsRGB;
- void* m_pTransform;
private:
- uint32_t m_nSrcComponents;
+ const bool m_bsRGB;
+ void* m_pTransform = nullptr;
+ uint32_t m_nSrcComponents = 0;
};
class CPDF_DeviceCS : public CPDF_ColorSpace {