summaryrefslogtreecommitdiff
path: root/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc
diff options
context:
space:
mode:
authorRyan Harrison <rharrison@chromium.org>2018-01-26 17:11:51 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-01-26 17:11:51 +0000
commit4b9d69806c30775f55563e72b0842daa06483f99 (patch)
treede54ec0133b6ad5ee625060adbfb02c45a0b9c83 /testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc
parent49363202ce06ca9ff418b4df384cffadf924303c (diff)
downloadpdfium-4b9d69806c30775f55563e72b0842daa06483f99.tar.xz
Clean up construction of CFX_Barcode
Because certain enum values can cause the initialization of the class to fail there is a seperate init method from the constructor. This CL is converting the code to use a standard factory pattern for this, instead of the existing implementation. Change-Id: Ia2293ce94ad0db5862db9796aeb8a224fd2b45f9 Reviewed-on: https://pdfium-review.googlesource.com/24230 Reviewed-by: Henrique Nakashima <hnakashima@chromium.org> Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc')
-rw-r--r--testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc14
1 files changed, 7 insertions, 7 deletions
diff --git a/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc b/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc
index 3334e86bfc..aaeebf3ace 100644
--- a/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc
+++ b/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc
@@ -17,20 +17,20 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) {
data += sizeof(wchar_t);
size -= sizeof(wchar_t);
- CFX_Barcode barcode;
- if (!barcode.Create(type))
+ auto barcode = CFX_Barcode::Create(type);
+ if (!barcode)
return 0;
// TODO(tsepez): Setup more options from |data|.
- barcode.SetModuleHeight(300);
- barcode.SetModuleWidth(420);
- barcode.SetHeight(298);
- barcode.SetWidth(418);
+ barcode->SetModuleHeight(300);
+ barcode->SetModuleWidth(420);
+ barcode->SetHeight(298);
+ barcode->SetWidth(418);
WideStringView content(reinterpret_cast<const wchar_t*>(data),
size / sizeof(wchar_t));
- if (!barcode.Encode(content))
+ if (!barcode->Encode(content))
return 0;
// TODO(tsepez): Output to device.