Counter cleanup

Hal Murray hmurray at megapathdsl.net
Sat Dec 16 11:45:06 UTC 2017


I just pushed code that fixes the counter overflow problem, Issue #424.

I made a pass through the mode6 code and fixed various other counters too.  
If you can read it, it should be big enough.  The hit count in the MRU slots 
didn't get changed.  It will take a long sustained attack to overflow.

I also added the type uptime_t and fixed all the places I found that use it.

I could easily have missed something.  I doubt if I broke anything, but there 
is the possibility that I fat fingered an edit.

I removed a couple of casts that seemed like clutter to me.  They may have 
been there to avoid warning messages - maybe about a hidden sign/unsigned 
change when a signed int is passed to a procedure that takes an unsigned.  I 
couldn't trigger any warnings.  If you get them, please tell me what's going 
on.  I expect the right cleanup is something other then adding a cast.

---------

We still need to make a pass through the code to verify that all the received 
packet processing counters get bumped and/or that all the bail paths bump 
something.



-- 
These are my opinions.  I hate spam.





More information about the devel mailing list