[tor-bugs] #18642 [Core Tor/Tor]: Teach the OOM handler about the DNS cache

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Jul 17 13:02:05 UTC 2018


#18642: Teach the OOM handler about the DNS cache
-------------------------------------------------+-------------------------
 Reporter:  nickm                                |          Owner:  (none)
     Type:  enhancement                          |         Status:
                                                 |  needs_revision
 Priority:  Medium                               |      Milestone:  Tor:
                                                 |  0.3.5.x-final
Component:  Core Tor/Tor                         |        Version:
 Severity:  Normal                               |     Resolution:
 Keywords:  tor-dos, oom, tor-relay, dns, 035    |  Actual Points:
  -triaged-in-20180711                           |
Parent ID:                                       |         Points:  1
 Reviewer:  dgoulet                              |        Sponsor:
                                                 |  SponsorV-can
-------------------------------------------------+-------------------------
Changes (by dgoulet):

 * status:  needs_review => needs_revision


Comment:

 True, `ExitRelay` won't work. I'm also not very enthusiastic on using
 `policy_is_reject_star()` which possibly goes over the entire policy just
 to learn if we are an Exit relay with a non reject policy.

 The Roles we've added recently for the callbacks in tor should be what we
 look for imo but don't have one for the Exit just yet (#25899).

 So for now, lets always run it, checking the HT size is very cheap and the
 OOM is only triggered on memory pressure so shouldn't be that often in
 most cases.

 Sorry neel, the PR looks good, just remove the `ExitRelay` condition :P.
 Thanks!

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/18642#comment:28>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list