tree 2911f3990985ba87be3aecca374d578c5cedac19
parent d0aee85b29a17e6ef3fb1454a798e10ced0d0266
author Jacob Erlbeck <jerlbeck@sysmocom.de> 1434016998 +0200
committer Jacob Erlbeck <jerlbeck@sysmocom.de> 1434021235 +0200

bssgp: Calculate the avg_delay_ms in 32bit only (Coverity)

Currently the delay_sum is stored in 64 to avoid overflow errors.
But only the result of tv_sec * 1000 is casted to 64 bit, resulting
in an overflow if the accumulated queue delay reached 25 days (which
will not happen in practice, unless there are >200k LLC messages with
a max of 10s delay each in the queue). If that were the case, the
only impact would be a wrong number in a log message and in the BSSGP
FLOW CONTROL message.

This commit changes the calculations so that they are done in 32 bit
only, rather than to do the calculation in 64 bit properly.

Fixes: Coverity CID 1298705
Sponsored-by: On-Waves ehf