aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/core/ext/filters/client_channel/client_channel.cc
diff options
context:
space:
mode:
authorGravatar Mark D. Roth <roth@google.com>2018-08-01 07:11:55 -0700
committerGravatar Mark D. Roth <roth@google.com>2018-08-01 07:11:55 -0700
commit95f81046dc6d43ecdf63820c7e56b42ad0734830 (patch)
tree535771d4072ff0243dcaeb4e105b660916777f76 /src/core/ext/filters/client_channel/client_channel.cc
parent6975fe484cf17bf056b1f8e2daf23899a0378083 (diff)
Fix undefined behavior introduced in #16167.
Diffstat (limited to 'src/core/ext/filters/client_channel/client_channel.cc')
-rw-r--r--src/core/ext/filters/client_channel/client_channel.cc7
1 files changed, 6 insertions, 1 deletions
diff --git a/src/core/ext/filters/client_channel/client_channel.cc b/src/core/ext/filters/client_channel/client_channel.cc
index 45bca2738c..fead8feb17 100644
--- a/src/core/ext/filters/client_channel/client_channel.cc
+++ b/src/core/ext/filters/client_channel/client_channel.cc
@@ -578,7 +578,12 @@ static void start_transport_op_locked(void* arg, grpc_error* error_ignored) {
} else {
grpc_error* error = GRPC_ERROR_NONE;
grpc_core::LoadBalancingPolicy::PickState pick_state;
- memset(&pick_state, 0, sizeof(pick_state));
+ pick_state.initial_metadata = nullptr;
+ pick_state.initial_metadata_flags = 0;
+ pick_state.on_complete = nullptr;
+ memset(&pick_state.subchannel_call_context, 0,
+ sizeof(pick_state.subchannel_call_context));
+ pick_state.user_data = nullptr;
// Pick must return synchronously, because pick_state.on_complete is null.
GPR_ASSERT(chand->lb_policy->PickLocked(&pick_state, &error));
if (pick_state.connected_subchannel != nullptr) {