diff options
author | Jan Tattermusch <jtattermusch@users.noreply.github.com> | 2016-05-10 15:49:05 -0700 |
---|---|---|
committer | Jan Tattermusch <jtattermusch@users.noreply.github.com> | 2016-05-10 15:49:05 -0700 |
commit | 8b382748fd9ce6f5d898e1acf15c97851c4fbd8b (patch) | |
tree | 11fbcb4873304129f8f72715d28015ddac45360a /test | |
parent | e28415bdec34a9f4b19981d9d5d5d3df92eb8e5c (diff) | |
parent | c5943093b517c8672299307ac826e166509d1a6f (diff) |
Merge pull request #6514 from yang-g/va_start
Change argument type to avoid undefined behavior
Diffstat (limited to 'test')
-rw-r--r-- | test/core/client_config/lb_policies_test.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/core/client_config/lb_policies_test.c b/test/core/client_config/lb_policies_test.c index e766672cf5..2ec46124a8 100644 --- a/test/core/client_config/lb_policies_test.c +++ b/test/core/client_config/lb_policies_test.c @@ -438,9 +438,9 @@ static grpc_call **perform_multirequest(servers_fixture *f, return calls; } -static void assert_channel_connectivity( - grpc_channel *ch, size_t num_accepted_conn_states, - grpc_connectivity_state accepted_conn_state, ...) { +static void assert_channel_connectivity(grpc_channel *ch, + size_t num_accepted_conn_states, + int accepted_conn_state, ...) { size_t i; grpc_channel_stack *client_stack; grpc_channel_element *client_channel_filter; @@ -456,7 +456,7 @@ static void assert_channel_connectivity( grpc_exec_ctx_finish(&exec_ctx); va_start(ap, accepted_conn_state); for (i = 0; i < num_accepted_conn_states; i++) { - if (actual_conn_state == accepted_conn_state) { + if ((int)actual_conn_state == accepted_conn_state) { break; } accepted_conn_state = va_arg(ap, grpc_connectivity_state); |