struct fr_connection 'number' label
ahurt at ena.com
Wed Jun 26 18:19:17 CEST 2013
Thank you all, I see the commit. Much appreciated.
On Jun 26, 2013, at 10:24 AM, Alan DeKok <aland at deployingradius.com> wrote:
> 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.
> List info/subscribe/unsubscribe? See http://www.freeradius.org/list/devel.html
More information about the Freeradius-Devel