From 8ada2ef8000abce4eb98506cc6195c78067f1000 Mon Sep 17 00:00:00 2001
From: Lei Zhang <thestig@chromium.org>
Date: Wed, 23 Aug 2017 16:26:38 -0700
Subject: Fix some nits in XFACodecFuzzer.

Change-Id: Ia9738e14d90a813b6ec7abd72843248fd3a3bff0
Reviewed-on: https://pdfium-review.googlesource.com/11910
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
---
 testing/libfuzzer/xfa_codec_fuzzer.h | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'testing')

diff --git a/testing/libfuzzer/xfa_codec_fuzzer.h b/testing/libfuzzer/xfa_codec_fuzzer.h
index 88227c91a9..0d6b43e7b8 100644
--- a/testing/libfuzzer/xfa_codec_fuzzer.h
+++ b/testing/libfuzzer/xfa_codec_fuzzer.h
@@ -28,7 +28,7 @@ class XFACodecFuzzer {
 
     std::unique_ptr<CCodec_ProgressiveDecoder> decoder =
         mgr->CreateProgressiveDecoder();
-    CFX_RetainPtr<Reader> source(new Reader(data, size));
+    auto source = pdfium::MakeRetain<Reader>(data, size);
     FXCODEC_STATUS status = decoder->LoadImageInfo(source, type, nullptr, true);
     if (status != FXCODEC_STATUS_FRAME_READY)
       return 0;
@@ -38,8 +38,9 @@ class XFACodecFuzzer {
 
     int32_t frames;
     if (decoder->GetFrames(frames) != FXCODEC_STATUS_DECODE_READY ||
-        frames == 0)
+        frames == 0) {
       return 0;
+    }
 
     status = decoder->StartDecode(bitmap, 0, 0, bitmap->GetWidth(),
                                   bitmap->GetHeight());
@@ -71,7 +72,7 @@ class XFACodecFuzzer {
 
    private:
     const uint8_t* const m_data;
-    size_t m_size;
+    const size_t m_size;
   };
 };
 
-- 
cgit v1.2.3