From 09658682c93805d5114a0d638388349c4276f9a3 Mon Sep 17 00:00:00 2001 From: Craig Tiller Date: Wed, 27 Oct 2021 16:08:05 -0700 Subject: [PATCH] Remove condition that I dont remember why it exists (#27836) --- test/core/end2end/fuzzers/api_fuzzer.cc | 3 - .../fuzzers/api_fuzzer_corpus/validator-fail | 67 +++++++++++++++++++ 2 files changed, 67 insertions(+), 3 deletions(-) create mode 100644 test/core/end2end/fuzzers/api_fuzzer_corpus/validator-fail diff --git a/test/core/end2end/fuzzers/api_fuzzer.cc b/test/core/end2end/fuzzers/api_fuzzer.cc index f62c1de239f..35d5a094252 100644 --- a/test/core/end2end/fuzzers/api_fuzzer.cc +++ b/test/core/end2end/fuzzers/api_fuzzer.cc @@ -444,9 +444,6 @@ class Call : public std::enable_shared_from_this { auto self = shared_from_this(); return MakeValidator([self, has_ops](bool) { --self->pending_ops_; - if ((has_ops & (1u << GRPC_OP_RECV_MESSAGE)) && self->call_closed_) { - GPR_ASSERT(self->recv_message_ == nullptr); - } if ((has_ops & (1u << GRPC_OP_RECV_MESSAGE) && self->recv_message_ != nullptr)) { grpc_byte_buffer_destroy(self->recv_message_); diff --git a/test/core/end2end/fuzzers/api_fuzzer_corpus/validator-fail b/test/core/end2end/fuzzers/api_fuzzer_corpus/validator-fail new file mode 100644 index 00000000000..c735e9a10c6 --- /dev/null +++ b/test/core/end2end/fuzzers/api_fuzzer_corpus/validator-fail @@ -0,0 +1,67 @@ +actions { + create_server { + } +} +actions { + create_channel { + target: "dns:server" + } +} +actions { + create_call { + method { + value: "/foo" + } + timeout: 1000000000 + } +} +actions { + request_call { + } +} +actions { + queue_batch { + operations { + send_initial_metadata { + } + } + operations { + send_message { + } + } + } +} +actions { + change_active_call { + } +} +actions { + advance_time: 10000000 +} +actions { + advance_time: 10000000 +} +actions { + poll_cq { + } +} +actions { + queue_batch { + operations { + receive_message { + } + } + } +} +actions { + queue_batch { + operations { + send_close_from_client { + } + } + operations { + receive_close_on_server { + } + } + } +}