Commit c76a0243 authored by Max Moroz's avatar Max Moroz Committed by Commit Bot

[net] Fix hpack_decoder_fuzzer not to pass nullptr to http2::DecodeBuffer constructor.

Bug: 907693
Change-Id: I3514c9aa5aacf68e7b9964b637ac79c5fbe29b50
Reviewed-on: https://chromium-review.googlesource.com/c/1351286Reviewed-by: default avatarRyan Hamilton <rch@chromium.org>
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611004}
parent f96bcdff
...@@ -26,6 +26,11 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { ...@@ -26,6 +26,11 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) {
size_t chunk_size = fuzzed_data_provider.ConsumeUint32InRange(1, 32); size_t chunk_size = fuzzed_data_provider.ConsumeUint32InRange(1, 32);
std::vector<char> chunk = std::vector<char> chunk =
fuzzed_data_provider.ConsumeBytes<char>(chunk_size); fuzzed_data_provider.ConsumeBytes<char>(chunk_size);
// http2::DecodeBuffer constructor does not accept nullptr buffer.
if (chunk.data() == nullptr)
continue;
http2::DecodeBuffer fragment(chunk.data(), chunk.size()); http2::DecodeBuffer fragment(chunk.data(), chunk.size());
decoder.DecodeFragment(&fragment); decoder.DecodeFragment(&fragment);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment