Skip to content

Commit

Permalink
Merge pull request #315 from amirlivneh/fuzz-shutdown-streams
Browse files Browse the repository at this point in the history
fuzz_http3serverreq: Fuzz stream shutdown
  • Loading branch information
tatsuhiro-t authored Jan 14, 2025
2 parents 1c3fe9a + b7e5ced commit 458a8c2
Showing 1 changed file with 25 additions and 0 deletions.
25 changes: 25 additions & 0 deletions fuzz/fuzz_http3serverreq.cc
Original file line number Diff line number Diff line change
Expand Up @@ -226,6 +226,27 @@ int send_data(nghttp3_conn *conn) {
}
}; // namespace

namespace {
int shutdown_streams(nghttp3_conn *conn,
FuzzedDataProvider &fuzzed_data_provider) {
for (; fuzzed_data_provider.ConsumeBool();) {
auto stream_id = fuzzed_data_provider.ConsumeIntegralInRange<int64_t>(
0, NGHTTP3_MAX_VARINT);

if (fuzzed_data_provider.ConsumeBool()) {
auto rv = nghttp3_conn_shutdown_stream_read(conn, stream_id);
if (rv != 0) {
return rv;
}
} else {
nghttp3_conn_shutdown_stream_write(conn, stream_id);
}
}

return 0;
}
}; // namespace

namespace {
int set_stream_priorities(nghttp3_conn *conn,
FuzzedDataProvider &fuzzed_data_provider) {
Expand Down Expand Up @@ -343,6 +364,10 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
}
}

if (shutdown_streams(conn, fuzzed_data_provider) != 0) {
goto fin;
}

if (!shutdown_started && fuzzed_data_provider.ConsumeBool()) {
shutdown_started = true;

Expand Down

0 comments on commit 458a8c2

Please sign in to comment.