[tor-bugs] #1042 [Tor Relay]: rate-limit MaxOnionsPending warns
Tor Bug Tracker & Wiki
torproject-admin at torproject.org
Sun Aug 29 04:35:50 UTC 2010
#1042: rate-limit MaxOnionsPending warns
--------------------------------+-------------------------------------------
Reporter: arma | Type: defect
Status: needs_review | Priority: minor
Milestone: Tor: 0.2.2.x-final | Component: Tor Relay
Version: 0.2.1.19 | Resolution: None
Keywords: easy | Parent:
--------------------------------+-------------------------------------------
Comment(by Sebastian):
I think you mean <= instead of >= in this line, otherwise this never
triggers:
{{{
+ if (lim->rate + lim->last_allowed >= now) {
}}}
Also I'd make rate_limit_is_ready() static. It can't be used to peek how
many log messages where generated so far and for logging purposes
rate_limit_log() should be entirely sufficient.
For the interval that is used to ratelimit onionskin handoff messages, we
should probably use WARN_TOO_MANY_CONNS_INTERVAL, or is there a reason why
you want it to log more often?
Other than that, looks good to me.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/1042#comment:16>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list