From 994f20cfb76f4902491a94c4ef61f55705fc124d Mon Sep 17 00:00:00 2001 From: Ryan Harrison Date: Tue, 31 Oct 2017 17:57:48 +0000 Subject: Fix FPE in XFACodecFuzzer This is a div by 0 issue BUG=chromium:779890 Change-Id: I91ca2969ffe52fb486b1fab32abc8609cd6567a2 Reviewed-on: https://pdfium-review.googlesource.com/17250 Reviewed-by: Tom Sepez Commit-Queue: Ryan Harrison --- testing/libfuzzer/xfa_codec_fuzzer.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/testing/libfuzzer/xfa_codec_fuzzer.h b/testing/libfuzzer/xfa_codec_fuzzer.h index 9a1230dbf9..96074fe7cf 100644 --- a/testing/libfuzzer/xfa_codec_fuzzer.h +++ b/testing/libfuzzer/xfa_codec_fuzzer.h @@ -39,7 +39,8 @@ class XFACodecFuzzer { // Skipping very large images, since they will take a long time and may lead // to OOM. - if (decoder->GetWidth() > kXFACodecFuzzerPixelLimit / decoder->GetHeight()) + if (decoder->GetHeight() != 0 && + decoder->GetWidth() > kXFACodecFuzzerPixelLimit / decoder->GetHeight()) return 0; auto bitmap = pdfium::MakeRetain(); -- cgit v1.2.3