struct fr_connection 'number' label

Alan DeKok aland at deployingradius.com
Wed Jun 26 17:24:39 CEST 2013


Aaron Hurt wrote:
> Should fr_connection->number match fr_connection_pool_t->count  … it
> seems as though number is being assigned the incremented value of count
> and number is a signed int whereas count is an unsigned int?  What
> happens if count grows larger than the max signed int?  Should both of
> these be changed to a long or unsigned long to reduce the chance of
> overflow to a near impossibility?

  If it opens 100 new connections a second, it will take a month to roll
over 2^32.

  Annoying, but not critical.  Making it a uint64_t would fix that for
all reasonable definitions of "forever".

  Alan DeKok.


More information about the Freeradius-Devel mailing list