ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

session: fix pacer bucket update cast

Browse Source 
Make sure comparison is done between two i64 values.

Type: fix

Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: Ief5706f2bd9415587994a5b665d5e380b8e14f68
This commit is contained in:
Florin Coras
2022-01-09 17:20:28 -08:00
committed by Florin Coras
parent 26ea3ede43
commit 93dd58c8fb
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/vnet/session/transport.c
View File

@@ -660,12 +660,12 @@ static inline u32
spacer_max_burst (spacer_t * pacer, clib_us_time_t time_now) spacer_max_burst (spacer_t * pacer, clib_us_time_t time_now)
{ {
u64 n_periods = (time_now - pacer->last_update); u64 n_periods = (time_now - pacer->last_update);
u64 inc; i64 inc;
if ((inc = (f32) n_periods * pacer->tokens_per_period) > 10) if ((inc = (f32) n_periods * pacer->tokens_per_period) > 10)
{ {
pacer->last_update = time_now; pacer->last_update = time_now;
pacer->bucket = clib_min (pacer->bucket + inc, pacer->max_burst); pacer->bucket = clib_min (pacer->bucket + inc, (i64) pacer->max_burst);
} }
return pacer->bucket > 0 ? pacer->max_burst : 0; return pacer->bucket > 0 ? pacer->max_burst : 0;