Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
There was some shared state when the HCM was creating singletons. This clears the map.

Risk Level: Low
Testing: Checked bug fix in PR that reproduces envoyproxy#12530

Signed-off-by: Asra Ali <[email protected]>
Signed-off-by: Clara Andrew-Wani <[email protected]>
  • Loading branch information
asraa authored and clarakosi committed Sep 3, 2020
1 parent fdf6ff1 commit 32eebc6
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ void UberFilterFuzzer::reset() {
Event::MockDispatcher& mock_dispatcher =
dynamic_cast<Event::MockDispatcher&>(read_filter_callbacks_->connection_.dispatcher_);
mock_dispatcher.clearDeferredDeleteList();
factory_context_.admin_.config_tracker_.config_tracker_callbacks_.clear();
read_filter_.reset();
}

Expand Down Expand Up @@ -94,7 +95,6 @@ void UberFilterFuzzer::fuzz(
checkInvalidInputForFuzzer(filter_name, message.get());
ENVOY_LOG_MISC(info, "Config content after decoded: {}", message->DebugString());
cb_ = factory.createFilterFactoryFromProto(*message, factory_context_);

} catch (const EnvoyException& e) {
ENVOY_LOG_MISC(debug, "Controlled exception in filter setup {}", e.what());
return;
Expand Down

0 comments on commit 32eebc6

Please sign in to comment.