From f0799fe84df4d86f25d342b03525f3016d674f86 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Tue, 28 Mar 2017 09:31:32 -0700 Subject: Revert "Revert "Refcount all CFX_DIBSources (and subclasses) all the time."" This reverts commit 31b08d4cdaa17d7a03f35e087096a77036af98ec. Re-landing the patch after fixing skia build issue. Change-Id: Ie7039890088b803a6ec5ce365d70f57277459b48 Reviewed-on: https://pdfium-review.googlesource.com/3245 Commit-Queue: Tom Sepez Commit-Queue: dsinclair Reviewed-by: dsinclair --- testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc | 2 +- testing/libfuzzer/pdf_jpx_fuzzer.cc | 3 +-- testing/libfuzzer/xfa_codec_fuzzer.h | 4 ++-- 3 files changed, 4 insertions(+), 5 deletions(-) (limited to 'testing/libfuzzer') diff --git a/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc b/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc index 1db39751db..162a0bf772 100644 --- a/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc +++ b/testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc @@ -25,7 +25,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { size -= kParameterSize; data += kParameterSize; - std::unique_ptr bitmap(new CFX_DIBitmap); + auto bitmap = pdfium::MakeRetain(); if (!bitmap->Create(width, height, FXDIB_1bppRgb)) return 0; diff --git a/testing/libfuzzer/pdf_jpx_fuzzer.cc b/testing/libfuzzer/pdf_jpx_fuzzer.cc index bea7ba2dbe..92ba210be3 100644 --- a/testing/libfuzzer/pdf_jpx_fuzzer.cc +++ b/testing/libfuzzer/pdf_jpx_fuzzer.cc @@ -37,8 +37,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { width = (width * components + 2) / 3; format = FXDIB_Rgb; } - - std::unique_ptr bitmap(new CFX_DIBitmap); + auto bitmap = pdfium::MakeRetain(); if (!bitmap->Create(width, height, format)) return 0; diff --git a/testing/libfuzzer/xfa_codec_fuzzer.h b/testing/libfuzzer/xfa_codec_fuzzer.h index 9a8b23e368..85cedec9a1 100644 --- a/testing/libfuzzer/xfa_codec_fuzzer.h +++ b/testing/libfuzzer/xfa_codec_fuzzer.h @@ -32,7 +32,7 @@ class XFACodecFuzzer { if (status != FXCODEC_STATUS_FRAME_READY) return 0; - std::unique_ptr bitmap(new CFX_DIBitmap); + auto bitmap = pdfium::MakeRetain(); bitmap->Create(decoder->GetWidth(), decoder->GetHeight(), FXDIB_Argb); int32_t frames; @@ -40,7 +40,7 @@ class XFACodecFuzzer { frames == 0) return 0; - status = decoder->StartDecode(bitmap.get(), 0, 0, bitmap->GetWidth(), + status = decoder->StartDecode(bitmap, 0, 0, bitmap->GetWidth(), bitmap->GetHeight()); while (status == FXCODEC_STATUS_DECODE_TOBECONTINUE) status = decoder->ContinueDecode(); -- cgit v1.2.3