[tor-bugs] #8235 [Tor]: pathbias_count_build_success()

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Apr 2 08:07:13 UTC 2013


#8235: pathbias_count_build_success()
----------------------------+-----------------------------------------------
    Reporter:  cypherpunks  |       Owner:  mikeperry                  
        Type:  defect       |      Status:  needs_revision             
    Priority:  major        |   Milestone:  Tor: 0.2.4.x-final         
   Component:  Tor          |     Version:  Tor: 0.2.4.10-alpha        
  Resolution:               |    Keywords:  tor-client, MikePerry201304
      Parent:               |      Points:                             
Actualpoints:               |  
----------------------------+-----------------------------------------------

Comment(by mikeperry):

 Replying to [comment:14 andrea]:
 > Begin code review:
 >
 >  * 2e1dd2d9d0ab11160ab6c45e6923551b50717027 looks fine to me
 >  * In c29da58ecb58f6513a4a3051144b285bb3653f3e, you're emitting log
 message if the counts become inconsistent after scaling but were
 consistent before (counts_are_sane && ...); if !counts_are_sane, are we
 certain to have already warned when that condition first occurred, or can
 this cause it to silently not log?  Change this or say you're sure this is
 the case, I think.

 I don't believe the checks in this particular commit will silence any log
 messages. The changes to entrynodes.c in 2e1dd2d9 might silence some, but
 they are meant to cover the cases where previous tor versions had
 overcounting bugs that this one does not.

 Should I perhaps promote 2e1dd2d9 to notice level? It might be nice to
 know if that keeps happening... I can add a version check in there for
 0.2.4.10-alpha I guess (assuming that is easy).

 Otherwise, I think these log messages will at least get us closer to
 solving this bug. Of course, I don't claim that any of these commits solve
 it for sure as of yet. (That's why the changes file is so gimpy and
 doesn't mention any solved bugs or versions).

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


More information about the tor-bugs mailing list