commit 278c4ecdce7b28ad53fb6a326493b9588a48878e
author Lei Zhang <thestig@chromium.org> Mon Mar 23 21:44:19 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Mon Mar 23 21:44:19 2020 +0000
tree 780d5e525013e9a24b68dd119c7ad5a369bb0c72
parent 5958fafa4756c89cd8fb2b4d5de0b2442cdb7f7a

Make CPDF_DIB::ValidateDictParam() more consistent.

Currently, this method treats /Filter [/RunLengthDecode] and
/Filter /RunLengthDecode differently, which does not make sense.
Remove (loosen) the check for the name case to match the array case.

Change-Id: I3de744e710b4251f2ee744a88de0c63fe784f52e
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/67753
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>

core/fpdfapi/page/cpdf_dib.cpp

diff --git a/core/fpdfapi/page/cpdf_dib.cpp b/core/fpdfapi/page/cpdf_dib.cpp
index 1c51732..ccd7bd5 100644
--- a/core/fpdfapi/page/cpdf_dib.cpp
+++ b/core/fpdfapi/page/cpdf_dib.cpp
@@ -934,10 +934,6 @@
       if (filter == "CCITTFaxDecode" || filter == "JBIG2Decode") {
         m_bpc = 1;
         m_nComponents = 1;
-      } else if (filter == "RunLengthDecode") {
-        if (m_bpc != 1) {
-          m_bpc = 8;
-        }
       } else if (filter == "DCTDecode") {
         m_bpc = 8;
       }