Looking through the code here:
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?
-- Aaron