commit b1b33d0ea4c1159f5d45273b00be812a09bb83fa
author Tom Sepez <tsepez@chromium.org> Wed Mar 27 23:13:31 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Wed Mar 27 23:13:31 2019 +0000
tree ef6c2f86a4aed59292bdf092cf6b20bed88ace30
parent a4382cfdce2a583dd006c9d9390a0994ff56d154

pStream can not be null in CPDF_FlateEncoder.

CPDF_FlateEncoder is only constructed in one place with
an argument of |this|.

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

core/fpdfapi/parser/cpdf_flateencoder.cpp

diff --git a/core/fpdfapi/parser/cpdf_flateencoder.cpp b/core/fpdfapi/parser/cpdf_flateencoder.cpp
index c5ad9e8..cf56540 100644
--- a/core/fpdfapi/parser/cpdf_flateencoder.cpp
+++ b/core/fpdfapi/parser/cpdf_flateencoder.cpp
@@ -22,7 +22,7 @@
     : m_pAcc(pdfium::MakeRetain<CPDF_StreamAcc>(pStream)), m_dwSize(0) {
   m_pAcc->LoadAllDataRaw();
 
-  bool bHasFilter = pStream && pStream->HasFilter();
+  bool bHasFilter = pStream->HasFilter();
   if (bHasFilter && !bFlateEncode) {
     auto pDestAcc = pdfium::MakeRetain<CPDF_StreamAcc>(pStream);
     pDestAcc->LoadAllDataFiltered();