Chromium Code Reviews| Index: testing/libfuzzer/pdf_codec_fax_fuzzer.cc |
| diff --git a/testing/libfuzzer/pdf_codec_fax_fuzzer.cc b/testing/libfuzzer/pdf_codec_fax_fuzzer.cc |
| index 60e2c9ff19037ea903cfdb9202decc05ae079e98..2caf58ebca5046e764503d7c2c34a2a69d0094f5 100644 |
| --- a/testing/libfuzzer/pdf_codec_fax_fuzzer.cc |
| +++ b/testing/libfuzzer/pdf_codec_fax_fuzzer.cc |
| @@ -22,17 +22,16 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { |
| int K = GetInteger(data + 8); |
| int Columns = GetInteger(data + 12); |
| int Rows = GetInteger(data + 16); |
| - FX_BOOL EndOfLine = (data[20] & 0x01) == 0; |
| - FX_BOOL ByteAlign = (data[20] & 0x02) == 0; |
| - FX_BOOL BlackIs1 = (data[20] & 0x04) == 0; |
| + bool EndOfLine = (data[20] & 0x01) == 0; |
|
Tom Sepez
2016/09/23 21:55:48
nit: prefer !
Lei Zhang
2016/09/26 22:50:59
Didn't you review this the first time around? :-P
|
| + bool ByteAlign = (data[20] & 0x02) == 0; |
| + bool BlackIs1 = (data[20] & 0x04) == 0; |
| data += kParameterSize; |
| size -= kParameterSize; |
| CCodec_FaxModule fax_module; |
| - std::unique_ptr<CCodec_ScanlineDecoder> decoder; |
| - decoder.reset(fax_module.CreateDecoder(data, size, width, height, K, |
| - EndOfLine, ByteAlign, BlackIs1, |
| - Columns, Rows)); |
| + std::unique_ptr<CCodec_ScanlineDecoder> decoder( |
| + fax_module.CreateDecoder(data, size, width, height, K, EndOfLine, |
| + ByteAlign, BlackIs1, Columns, Rows)); |
| if (decoder) { |
| int line = 0; |