struct fr_connection 'number' label

Aaron Hurt ahurt at ena.com
Wed Jun 26 18:19:17 CEST 2013


Thank you all, I see the commit.  Much appreciated.

-- Aaron

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 mailing list