From afd0d1f488ea55da545b3310fd8f22e45522a695 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Tue, 4 Apr 2017 14:37:18 -0700 Subject: RefCount CPDF_StreamAcc all the time. Pass stream argument to constructor; it feels like a stream accessor should always be made from a stream rather than passing one in after the fact. Change-Id: Iaa46cb37677b81f0170f5d39bab76ad38ea4af44 Reviewed-on: https://pdfium-review.googlesource.com/3620 Commit-Queue: Tom Sepez Reviewed-by: Lei Zhang --- testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc') diff --git a/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc b/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc index d23e3e07fa..9378141ffa 100644 --- a/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc +++ b/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc @@ -33,14 +33,15 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { auto stream = pdfium::MakeUnique(); stream->AsStream()->SetData(data, size); - CPDF_StreamAcc src_stream; - src_stream.LoadAllData(stream->AsStream(), true); + + auto src_stream = pdfium::MakeRetain(stream->AsStream()); + src_stream->LoadAllData(true); CCodec_Jbig2Module module; CCodec_Jbig2Context jbig2_context; std::unique_ptr document_context; FXCODEC_STATUS status = module.StartDecode( - &jbig2_context, &document_context, width, height, &src_stream, nullptr, + &jbig2_context, &document_context, width, height, src_stream, nullptr, bitmap->GetBuffer(), bitmap->GetPitch(), nullptr); while (status == FXCODEC_STATUS_DECODE_TOBECONTINUE) -- cgit v1.2.3