From 07401bae6d9f95911b144e6fabb42f19e40def49 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Mon, 11 Dec 2017 22:12:08 +0000 Subject: Remove default arguments to CPDF_StreamAcc::LoadAllData(). Add LoadAllDataFiltered() and LoadAllDataRaw() and update callers. Change-Id: I9b80ee34a358db204968acdc8b1adc9db0b6b83f Reviewed-on: https://pdfium-review.googlesource.com/20810 Commit-Queue: Lei Zhang Reviewed-by: Henrique Nakashima --- core/fpdfapi/edit/cpdf_flateencoder.cpp | 4 ++-- core/fpdfapi/font/cpdf_cidfont.cpp | 4 ++-- core/fpdfapi/font/cpdf_tounicodemap.cpp | 2 +- core/fpdfapi/page/cpdf_colorspace.cpp | 2 +- core/fpdfapi/page/cpdf_contentparser.cpp | 6 +++--- core/fpdfapi/page/cpdf_docpagedata.cpp | 4 ++-- core/fpdfapi/page/cpdf_meshstream.cpp | 2 +- core/fpdfapi/page/cpdf_psfunc.cpp | 2 +- core/fpdfapi/page/cpdf_sampledfunc.cpp | 2 +- core/fpdfapi/parser/cpdf_crypto_handler.cpp | 2 +- core/fpdfapi/parser/cpdf_hint_tables.cpp | 2 +- core/fpdfapi/parser/cpdf_parser.cpp | 4 ++-- core/fpdfapi/parser/cpdf_stream.cpp | 6 +++--- core/fpdfapi/parser/cpdf_stream_acc.cpp | 8 ++++++++ core/fpdfapi/parser/cpdf_stream_acc.h | 6 +++--- core/fpdfapi/parser/fpdf_parser_utility.cpp | 2 +- core/fpdfapi/render/cpdf_dibsource.cpp | 2 +- core/fpdfdoc/cpdf_metadata.cpp | 2 +- core/fxcodec/jbig2/JBig2_BitStream_unittest.cpp | 12 ++++++------ core/fxcrt/cfx_seekablemultistream.cpp | 2 +- fpdfsdk/fpdf_flatten.cpp | 4 ++-- fpdfsdk/fpdfsave.cpp | 2 +- fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp | 2 +- testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc | 2 +- xfa/fxfa/cxfa_ffdoc.cpp | 2 +- 25 files changed, 48 insertions(+), 40 deletions(-) diff --git a/core/fpdfapi/edit/cpdf_flateencoder.cpp b/core/fpdfapi/edit/cpdf_flateencoder.cpp index f8e2b5ad55..555da84244 100644 --- a/core/fpdfapi/edit/cpdf_flateencoder.cpp +++ b/core/fpdfapi/edit/cpdf_flateencoder.cpp @@ -14,12 +14,12 @@ CPDF_FlateEncoder::CPDF_FlateEncoder(CPDF_Stream* pStream, bool bFlateEncode) : m_dwSize(0), m_pAcc(pdfium::MakeRetain(pStream)) { - m_pAcc->LoadAllData(true); + m_pAcc->LoadAllDataRaw(); bool bHasFilter = pStream && pStream->HasFilter(); if (bHasFilter && !bFlateEncode) { auto pDestAcc = pdfium::MakeRetain(pStream); - pDestAcc->LoadAllData(); + pDestAcc->LoadAllDataFiltered(); m_dwSize = pDestAcc->GetSize(); m_pData = pDestAcc->DetachData(); diff --git a/core/fpdfapi/font/cpdf_cidfont.cpp b/core/fpdfapi/font/cpdf_cidfont.cpp index 082b5039e3..21f8addf8f 100644 --- a/core/fpdfapi/font/cpdf_cidfont.cpp +++ b/core/fpdfapi/font/cpdf_cidfont.cpp @@ -374,7 +374,7 @@ bool CPDF_CIDFont::Load() { return false; } else if (CPDF_Stream* pStream = pEncoding->AsStream()) { auto pAcc = pdfium::MakeRetain(pStream); - pAcc->LoadAllData(false); + pAcc->LoadAllDataFiltered(); m_pCMap = pdfium::MakeRetain(); m_pCMap->LoadEmbedded(pAcc->GetData(), pAcc->GetSize()); } else { @@ -412,7 +412,7 @@ bool CPDF_CIDFont::Load() { if (pmap) { if (CPDF_Stream* pStream = pmap->AsStream()) { m_pStreamAcc = pdfium::MakeRetain(pStream); - m_pStreamAcc->LoadAllData(false); + m_pStreamAcc->LoadAllDataFiltered(); } else if (pmap->GetString() == "Identity") { #if _FX_PLATFORM_ == _FX_PLATFORM_APPLE_ if (m_pFontFile) diff --git a/core/fpdfapi/font/cpdf_tounicodemap.cpp b/core/fpdfapi/font/cpdf_tounicodemap.cpp index 0ddaa34e13..0b746d7bfc 100644 --- a/core/fpdfapi/font/cpdf_tounicodemap.cpp +++ b/core/fpdfapi/font/cpdf_tounicodemap.cpp @@ -125,7 +125,7 @@ uint32_t CPDF_ToUnicodeMap::GetUnicode() { void CPDF_ToUnicodeMap::Load(CPDF_Stream* pStream) { CIDSet cid_set = CIDSET_UNKNOWN; auto pAcc = pdfium::MakeRetain(pStream); - pAcc->LoadAllData(false); + pAcc->LoadAllDataFiltered(); CPDF_SimpleParser parser(pAcc->GetData(), pAcc->GetSize()); while (1) { ByteStringView word = parser.GetWord(); diff --git a/core/fpdfapi/page/cpdf_colorspace.cpp b/core/fpdfapi/page/cpdf_colorspace.cpp index f9214c75d6..c09562ce52 100644 --- a/core/fpdfapi/page/cpdf_colorspace.cpp +++ b/core/fpdfapi/page/cpdf_colorspace.cpp @@ -1031,7 +1031,7 @@ bool CPDF_IndexedCS::v_Load(CPDF_Document* pDoc, m_Table = pString->GetString(); } else if (CPDF_Stream* pStream = pTableObj->AsStream()) { auto pAcc = pdfium::MakeRetain(pStream); - pAcc->LoadAllData(false); + pAcc->LoadAllDataFiltered(); m_Table = ByteStringView(pAcc->GetData(), pAcc->GetSize()); } return true; diff --git a/core/fpdfapi/page/cpdf_contentparser.cpp b/core/fpdfapi/page/cpdf_contentparser.cpp index cc93b4df9e..93151af305 100644 --- a/core/fpdfapi/page/cpdf_contentparser.cpp +++ b/core/fpdfapi/page/cpdf_contentparser.cpp @@ -37,7 +37,7 @@ CPDF_ContentParser::CPDF_ContentParser(CPDF_Page* pPage) CPDF_Stream* pStream = pContent->AsStream(); if (pStream) { m_pSingleStream = pdfium::MakeRetain(pStream); - m_pSingleStream->LoadAllData(false); + m_pSingleStream->LoadAllDataFiltered(); return; } CPDF_Array* pArray = pContent->AsArray(); @@ -102,7 +102,7 @@ CPDF_ContentParser::CPDF_ContentParser(CPDF_Form* pForm, } m_pSingleStream = pdfium::MakeRetain(pForm->m_pFormStream.Get()); - m_pSingleStream->LoadAllData(false); + m_pSingleStream->LoadAllDataFiltered(); m_pData.Reset(m_pSingleStream->GetData()); m_Size = m_pSingleStream->GetSize(); } @@ -149,7 +149,7 @@ bool CPDF_ContentParser::Continue(IFX_PauseIndicator* pPause) { pContent ? pContent->GetDirectObjectAt(m_CurrentOffset) : nullptr); m_StreamArray[m_CurrentOffset] = pdfium::MakeRetain(pStreamObj); - m_StreamArray[m_CurrentOffset]->LoadAllData(false); + m_StreamArray[m_CurrentOffset]->LoadAllDataFiltered(); m_CurrentOffset++; } } diff --git a/core/fpdfapi/page/cpdf_docpagedata.cpp b/core/fpdfapi/page/cpdf_docpagedata.cpp index 31d6bae63f..dc2554f92d 100644 --- a/core/fpdfapi/page/cpdf_docpagedata.cpp +++ b/core/fpdfapi/page/cpdf_docpagedata.cpp @@ -417,7 +417,7 @@ RetainPtr CPDF_DocPageData::GetIccProfile( return it->second; auto pAccessor = pdfium::MakeRetain(pProfileStream); - pAccessor->LoadAllData(false); + pAccessor->LoadAllDataFiltered(); uint8_t digest[20]; CRYPT_SHA1Generate(pAccessor->GetData(), pAccessor->GetSize(), digest); @@ -457,7 +457,7 @@ RetainPtr CPDF_DocPageData::GetFontFileStreamAcc( org_size = std::max(org_size, 0); auto pFontAcc = pdfium::MakeRetain(pFontStream); - pFontAcc->LoadAllData(false, org_size); + pFontAcc->LoadAllData(false, org_size, false); m_FontFileMap[pFontStream] = pFontAcc; return pFontAcc; } diff --git a/core/fpdfapi/page/cpdf_meshstream.cpp b/core/fpdfapi/page/cpdf_meshstream.cpp index 1850efde9c..083acce68a 100644 --- a/core/fpdfapi/page/cpdf_meshstream.cpp +++ b/core/fpdfapi/page/cpdf_meshstream.cpp @@ -116,7 +116,7 @@ CPDF_MeshStream::CPDF_MeshStream( CPDF_MeshStream::~CPDF_MeshStream() {} bool CPDF_MeshStream::Load() { - m_pStream->LoadAllData(); + m_pStream->LoadAllDataFiltered(); m_BitStream = pdfium::MakeUnique(m_pStream->GetData(), m_pStream->GetSize()); CPDF_Dictionary* pDict = m_pShadingStream->GetDict(); diff --git a/core/fpdfapi/page/cpdf_psfunc.cpp b/core/fpdfapi/page/cpdf_psfunc.cpp index debbbd6e6d..9f101eb91b 100644 --- a/core/fpdfapi/page/cpdf_psfunc.cpp +++ b/core/fpdfapi/page/cpdf_psfunc.cpp @@ -15,7 +15,7 @@ CPDF_PSFunc::~CPDF_PSFunc() {} bool CPDF_PSFunc::v_Init(CPDF_Object* pObj) { auto pAcc = pdfium::MakeRetain(pObj->AsStream()); - pAcc->LoadAllData(false); + pAcc->LoadAllDataFiltered(); return m_PS.Parse(reinterpret_cast(pAcc->GetData()), pAcc->GetSize()); } diff --git a/core/fpdfapi/page/cpdf_sampledfunc.cpp b/core/fpdfapi/page/cpdf_sampledfunc.cpp index 91c6b5a70a..98e1fd03e0 100644 --- a/core/fpdfapi/page/cpdf_sampledfunc.cpp +++ b/core/fpdfapi/page/cpdf_sampledfunc.cpp @@ -51,7 +51,7 @@ bool CPDF_SampledFunc::v_Init(CPDF_Object* pObj) { m_SampleMax = 0xffffffff >> (32 - m_nBitsPerSample); m_pSampleStream = pdfium::MakeRetain(pStream); - m_pSampleStream->LoadAllData(false); + m_pSampleStream->LoadAllDataFiltered(); FX_SAFE_UINT32 nTotalSampleBits = 1; m_EncodeInfo.resize(m_nInputs); for (uint32_t i = 0; i < m_nInputs; i++) { diff --git a/core/fpdfapi/parser/cpdf_crypto_handler.cpp b/core/fpdfapi/parser/cpdf_crypto_handler.cpp index e8f0265ae4..d963df4887 100644 --- a/core/fpdfapi/parser/cpdf_crypto_handler.cpp +++ b/core/fpdfapi/parser/cpdf_crypto_handler.cpp @@ -327,7 +327,7 @@ std::unique_ptr CPDF_CryptoHandler::DecryptObjectTree( // TODO(art-snake): Move decryption into the CPDF_Stream class. CPDF_Stream* stream = child->AsStream(); auto stream_access = pdfium::MakeRetain(stream); - stream_access->LoadAllData(true); + stream_access->LoadAllDataRaw(); if (IsCipherAES() && stream_access->GetSize() < 16) { stream->SetData(nullptr, 0); diff --git a/core/fpdfapi/parser/cpdf_hint_tables.cpp b/core/fpdfapi/parser/cpdf_hint_tables.cpp index 2b58e9d8f3..c9123d4b97 100644 --- a/core/fpdfapi/parser/cpdf_hint_tables.cpp +++ b/core/fpdfapi/parser/cpdf_hint_tables.cpp @@ -471,7 +471,7 @@ bool CPDF_HintTables::LoadHintStream(CPDF_Stream* pHintStream) { return false; auto pAcc = pdfium::MakeRetain(pHintStream); - pAcc->LoadAllData(); + pAcc->LoadAllDataFiltered(); uint32_t size = pAcc->GetSize(); // The header section of page offset hint table is 36 bytes. diff --git a/core/fpdfapi/parser/cpdf_parser.cpp b/core/fpdfapi/parser/cpdf_parser.cpp index 4e77773a3b..0d1b02a5ce 100644 --- a/core/fpdfapi/parser/cpdf_parser.cpp +++ b/core/fpdfapi/parser/cpdf_parser.cpp @@ -1096,7 +1096,7 @@ bool CPDF_Parser::LoadCrossRefV5(FX_FILESIZE* pos, bool bMainXRef) { uint32_t totalWidth = dwAccWidth.ValueOrDie(); auto pAcc = pdfium::MakeRetain(pStream); - pAcc->LoadAllData(); + pAcc->LoadAllDataFiltered(); const uint8_t* pData = pAcc->GetData(); uint32_t dwTotalSize = pAcc->GetSize(); @@ -1250,7 +1250,7 @@ RetainPtr CPDF_Parser::GetObjectStream(uint32_t objnum) { return nullptr; auto pStreamAcc = pdfium::MakeRetain(pStream); - pStreamAcc->LoadAllData(); + pStreamAcc->LoadAllDataFiltered(); m_ObjectStreamMap[objnum] = pStreamAcc; return pStreamAcc; } diff --git a/core/fpdfapi/parser/cpdf_stream.cpp b/core/fpdfapi/parser/cpdf_stream.cpp index bf2feaed22..074e747a78 100644 --- a/core/fpdfapi/parser/cpdf_stream.cpp +++ b/core/fpdfapi/parser/cpdf_stream.cpp @@ -80,7 +80,7 @@ std::unique_ptr CPDF_Stream::CloneNonCyclic( std::set* pVisited) const { pVisited->insert(this); auto pAcc = pdfium::MakeRetain(this); - pAcc->LoadAllData(true); + pAcc->LoadAllDataRaw(); uint32_t streamSize = pAcc->GetSize(); CPDF_Dictionary* pDict = GetDict(); @@ -147,7 +147,7 @@ bool CPDF_Stream::HasFilter() const { WideString CPDF_Stream::GetUnicodeText() const { auto pAcc = pdfium::MakeRetain(this); - pAcc->LoadAllData(false); + pAcc->LoadAllDataFiltered(); return PDF_DecodeText(pAcc->GetData(), pAcc->GetSize()); } @@ -156,7 +156,7 @@ bool CPDF_Stream::WriteTo(IFX_ArchiveStream* archive) const { return false; auto pAcc = pdfium::MakeRetain(this); - pAcc->LoadAllData(true); + pAcc->LoadAllDataRaw(); return archive->WriteBlock(pAcc->GetData(), pAcc->GetSize()) && archive->WriteString("\r\nendstream"); } diff --git a/core/fpdfapi/parser/cpdf_stream_acc.cpp b/core/fpdfapi/parser/cpdf_stream_acc.cpp index 5792a80d14..d115b48226 100644 --- a/core/fpdfapi/parser/cpdf_stream_acc.cpp +++ b/core/fpdfapi/parser/cpdf_stream_acc.cpp @@ -56,6 +56,14 @@ void CPDF_StreamAcc::LoadAllData(bool bRawAccess, m_bNewBuf = m_pData != m_pStream->GetRawData(); } +void CPDF_StreamAcc::LoadAllDataFiltered() { + LoadAllData(false, 0, false); +} + +void CPDF_StreamAcc::LoadAllDataRaw() { + LoadAllData(true, 0, false); +} + CPDF_Dictionary* CPDF_StreamAcc::GetDict() const { return m_pStream ? m_pStream->GetDict() : nullptr; } diff --git a/core/fpdfapi/parser/cpdf_stream_acc.h b/core/fpdfapi/parser/cpdf_stream_acc.h index 52ac7e2e05..d54e000097 100644 --- a/core/fpdfapi/parser/cpdf_stream_acc.h +++ b/core/fpdfapi/parser/cpdf_stream_acc.h @@ -23,9 +23,9 @@ class CPDF_StreamAcc : public Retainable { CPDF_StreamAcc(const CPDF_StreamAcc&) = delete; CPDF_StreamAcc& operator=(const CPDF_StreamAcc&) = delete; - void LoadAllData(bool bRawAccess = false, - uint32_t estimated_size = 0, - bool bImageAcc = false); + void LoadAllData(bool bRawAccess, uint32_t estimated_size, bool bImageAcc); + void LoadAllDataFiltered(); + void LoadAllDataRaw(); const CPDF_Stream* GetStream() const { return m_pStream.Get(); } CPDF_Dictionary* GetDict() const; diff --git a/core/fpdfapi/parser/fpdf_parser_utility.cpp b/core/fpdfapi/parser/fpdf_parser_utility.cpp index 2185e88ff6..45284a4ceb 100644 --- a/core/fpdfapi/parser/fpdf_parser_utility.cpp +++ b/core/fpdfapi/parser/fpdf_parser_utility.cpp @@ -209,7 +209,7 @@ std::ostream& operator<<(std::ostream& buf, const CPDF_Object* pObj) { const CPDF_Stream* p = pObj->AsStream(); buf << p->GetDict() << "stream\r\n"; auto pAcc = pdfium::MakeRetain(p); - pAcc->LoadAllData(true); + pAcc->LoadAllDataRaw(); buf.write(reinterpret_cast(pAcc->GetData()), pAcc->GetSize()); buf << "\r\nendstream"; diff --git a/core/fpdfapi/render/cpdf_dibsource.cpp b/core/fpdfapi/render/cpdf_dibsource.cpp index 3cd18909eb..d8ddc767ba 100644 --- a/core/fpdfapi/render/cpdf_dibsource.cpp +++ b/core/fpdfapi/render/cpdf_dibsource.cpp @@ -328,7 +328,7 @@ int CPDF_DIBSource::ContinueLoadDIBSource(IFX_PauseIndicator* pPause) { m_pStreamAcc->GetImageParam()->GetStreamFor("JBIG2Globals"); if (pGlobals) { m_pGlobalStream = pdfium::MakeRetain(pGlobals); - m_pGlobalStream->LoadAllData(false); + m_pGlobalStream->LoadAllDataFiltered(); } } iDecodeStatus = pJbig2Module->StartDecode( diff --git a/core/fpdfdoc/cpdf_metadata.cpp b/core/fpdfdoc/cpdf_metadata.cpp index b31348aa0b..efe3c29dbf 100644 --- a/core/fpdfdoc/cpdf_metadata.cpp +++ b/core/fpdfdoc/cpdf_metadata.cpp @@ -21,7 +21,7 @@ CPDF_Metadata::CPDF_Metadata(const CPDF_Document* pDoc) { return; auto pAcc = pdfium::MakeRetain(pStream); - pAcc->LoadAllData(false); + pAcc->LoadAllDataFiltered(); m_pXmlElement = CXML_Element::Parse(pAcc->GetData(), pAcc->GetSize()); } diff --git a/core/fxcodec/jbig2/JBig2_BitStream_unittest.cpp b/core/fxcodec/jbig2/JBig2_BitStream_unittest.cpp index 7b91eff982..b66d8b0651 100644 --- a/core/fxcodec/jbig2/JBig2_BitStream_unittest.cpp +++ b/core/fxcodec/jbig2/JBig2_BitStream_unittest.cpp @@ -17,7 +17,7 @@ TEST(JBig2_BitStream, ReadNBits) { auto in_stream = pdfium::MakeUnique(std::move(data), 1, nullptr); auto acc = pdfium::MakeRetain(in_stream.get()); - acc->LoadAllData(); + acc->LoadAllDataFiltered(); CJBig2_BitStream stream(acc); @@ -42,7 +42,7 @@ TEST(JBig2_BitStream, ReadNBitsLargerThenData) { auto in_stream = pdfium::MakeUnique(std::move(data), 1, nullptr); auto acc = pdfium::MakeRetain(in_stream.get()); - acc->LoadAllData(); + acc->LoadAllDataFiltered(); CJBig2_BitStream stream(acc); @@ -54,7 +54,7 @@ TEST(JBig2_BitStream, ReadNBitsLargerThenData) { TEST(JBig2_BitStream, ReadNBitsNullStream) { auto in_stream = pdfium::MakeUnique(nullptr, 0, nullptr); auto acc = pdfium::MakeRetain(in_stream.get()); - acc->LoadAllData(); + acc->LoadAllDataFiltered(); CJBig2_BitStream stream(acc); @@ -70,7 +70,7 @@ TEST(JBig2_BitStream, ReadNBitsEmptyStream) { auto in_stream = pdfium::MakeUnique(std::move(data), 0, nullptr); auto acc = pdfium::MakeRetain(in_stream.get()); - acc->LoadAllData(); + acc->LoadAllDataFiltered(); CJBig2_BitStream stream(acc); @@ -87,7 +87,7 @@ TEST(JBig2_BitStream, ReadNBitsOutOfBounds) { auto in_stream = pdfium::MakeUnique(std::move(data), 1, nullptr); auto acc = pdfium::MakeRetain(in_stream.get()); - acc->LoadAllData(); + acc->LoadAllDataFiltered(); CJBig2_BitStream stream(acc); @@ -108,7 +108,7 @@ TEST(JBig2_BitStream, ReadNBitsWhereNIs36) { auto in_stream = pdfium::MakeUnique(std::move(data), 5, nullptr); auto acc = pdfium::MakeRetain(in_stream.get()); - acc->LoadAllData(); + acc->LoadAllDataFiltered(); CJBig2_BitStream stream(acc); diff --git a/core/fxcrt/cfx_seekablemultistream.cpp b/core/fxcrt/cfx_seekablemultistream.cpp index 12682d7678..92286b1939 100644 --- a/core/fxcrt/cfx_seekablemultistream.cpp +++ b/core/fxcrt/cfx_seekablemultistream.cpp @@ -16,7 +16,7 @@ CFX_SeekableMultiStream::CFX_SeekableMultiStream( const std::vector& streams) { for (CPDF_Stream* pStream : streams) { m_Data.push_back(pdfium::MakeRetain(pStream)); - m_Data.back()->LoadAllData(); + m_Data.back()->LoadAllDataFiltered(); } } diff --git a/fpdfsdk/fpdf_flatten.cpp b/fpdfsdk/fpdf_flatten.cpp index 2850c37bfb..4d06693006 100644 --- a/fpdfsdk/fpdf_flatten.cpp +++ b/fpdfsdk/fpdf_flatten.cpp @@ -198,7 +198,7 @@ void SetPageContents(const ByteString& key, if (!pContentsArray) { pContentsArray = pDocument->NewIndirect(); auto pAcc = pdfium::MakeRetain(pContentsStream); - pAcc->LoadAllData(); + pAcc->LoadAllDataFiltered(); ByteString sStream = "q\n"; ByteString sBody = ByteString(pAcc->GetData(), pAcc->GetSize()); sStream = sStream + sBody + "\nQ"; @@ -394,7 +394,7 @@ FPDF_EXPORT int FPDF_CALLCONV FPDFPage_Flatten(FPDF_PAGE page, int nFlag) { pObj->GetObjNum()); auto pAcc = pdfium::MakeRetain(pNewXObject); - pAcc->LoadAllData(); + pAcc->LoadAllDataFiltered(); ByteString sStream(pAcc->GetData(), pAcc->GetSize()); CFX_Matrix matrix = pAPDic->GetMatrixFor("Matrix"); CFX_Matrix m = GetMatrix(rcAnnot, rcStream, matrix); diff --git a/fpdfsdk/fpdfsave.cpp b/fpdfsdk/fpdfsave.cpp index a21310dfa1..ddbc994577 100644 --- a/fpdfsdk/fpdfsave.cpp +++ b/fpdfsdk/fpdfsave.cpp @@ -99,7 +99,7 @@ bool SaveXFADocumentData(CPDFXFA_Context* pContext, if (iTemplate > -1) { CPDF_Stream* pTemplateStream = pArray->GetStreamAt(iTemplate); auto pAcc = pdfium::MakeRetain(pTemplateStream); - pAcc->LoadAllData(); + pAcc->LoadAllDataFiltered(); RetainPtr pTemplate = pdfium::MakeRetain( const_cast(pAcc->GetData()), pAcc->GetSize(), false); diff --git a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp index 70b0f1b514..fdf0d31ea2 100644 --- a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp +++ b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp @@ -496,7 +496,7 @@ void CPDFXFA_DocEnvironment::ExportData(CXFA_FFDoc* hDoc, content.GetLength()); } auto pAcc = pdfium::MakeRetain(pStream); - pAcc->LoadAllData(); + pAcc->LoadAllDataFiltered(); fileWrite->WriteBlock(pAcc->GetData(), fileWrite->GetSize(), pAcc->GetSize()); } diff --git a/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc b/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc index ac3c5c3260..7b8e2aac33 100644 --- a/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc +++ b/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc @@ -44,7 +44,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { stream->AsStream()->SetData(data, size); auto src_stream = pdfium::MakeRetain(stream->AsStream()); - src_stream->LoadAllData(true); + src_stream->LoadAllDataRaw(); CCodec_Jbig2Module module; CCodec_Jbig2Context jbig2_context; diff --git a/xfa/fxfa/cxfa_ffdoc.cpp b/xfa/fxfa/cxfa_ffdoc.cpp index 8ca1e95d87..3ff947ca4f 100644 --- a/xfa/fxfa/cxfa_ffdoc.cpp +++ b/xfa/fxfa/cxfa_ffdoc.cpp @@ -375,7 +375,7 @@ RetainPtr CXFA_FFDoc::GetPDFNamedImage( return nullptr; auto pAcc = pdfium::MakeRetain(pStream); - pAcc->LoadAllData(); + pAcc->LoadAllDataFiltered(); RetainPtr pImageFileRead = pdfium::MakeRetain( -- cgit v1.2.3