diff options
author | 2017-09-11 15:26:57 -0700 | |
---|---|---|
committer | 2017-09-11 15:26:57 -0700 | |
commit | 20104ce596b8ccbe4cd167e245654f56e8d86edc (patch) | |
tree | 00a0ec5d247441fc91f5cbbe08f08316dccedf44 | |
parent | e29d497fa90506e9e09f2d66536052454acf3a6e (diff) |
Add a clamp to remove ubsan failure
-rw-r--r-- | src/core/ext/transport/chttp2/transport/flow_control.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/core/ext/transport/chttp2/transport/flow_control.c b/src/core/ext/transport/chttp2/transport/flow_control.c index cec99f6fb6..39aa521029 100644 --- a/src/core/ext/transport/chttp2/transport/flow_control.c +++ b/src/core/ext/transport/chttp2/transport/flow_control.c @@ -483,7 +483,8 @@ grpc_chttp2_flowctl_action grpc_chttp2_flowctl_get_bdp_action( if (grpc_bdp_estimator_get_bw(&tfc->bdp_estimator, &bw_dbl)) { // we target the max of BDP or bandwidth in microseconds. int32_t frame_size = (int32_t)GPR_CLAMP( - GPR_MAX((int32_t)bw_dbl / 1000, bdp), 16384, 16777215); + GPR_MAX((int32_t)GPR_CLAMP(bw_dbl, 0, INT_MAX) / 1000, bdp), 16384, + 16777215); grpc_chttp2_flowctl_urgency frame_size_urgency = delta_is_significant( tfc, frame_size, GRPC_CHTTP2_SETTINGS_MAX_FRAME_SIZE); if (frame_size_urgency != GRPC_CHTTP2_FLOWCTL_NO_ACTION_NEEDED) { |