Fix blank page issue caused by too strict correction on bpc

core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp

Index: core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp
diff --git a/core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp b/core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp
index 0362ff2e90cc48eea7fbc99f4eb80dc1b32f83e2..4c79d3637b0adbd772a3b1241dc14db76cf85d84 100644
--- a/core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp
+++ b/core/src/fpdfapi/fpdf_render/fpdf_render_loadimage.cpp
@@ -957,8 +957,10 @@ void CPDF_DIBSource::ValidateDictParam() {
         m_bpc = 1;
         m_nComponents = 1;
       }
-      if (filter == FX_BSTRC("RunLengthDecode") ||
-          filter == FX_BSTRC("DCTDecode")) {
+      if (filter == FX_BSTRC("RunLengthDecode") && m_bpc != 1) {

[Tom Sepez, 2015/09/08 15:47:19]

nit: I might write
   if (filter == FX_BSTRC("RunLengthDecode")) {
      if (m_bpc != 1) {
         m_bpc = 8;
      }
   } else if (filter == FX_BSTRC("DCTDecode) { ...

to avoid a second string comparison when when know the first one matched.

[jun_fang, 2015/09/09 09:57:12]

Acknowledged.

+        m_bpc = 8;
+      }
+      if (filter == FX_BSTRC("DCTDecode")) {
         m_bpc = 8;
       }
     } else if (pFilter->GetType() == PDFOBJ_ARRAY) {