[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #16400 [Tor]: Bug: Assertion cp failed in microdescs_parse_from_string at ../src/or/routerparse.c:4168



#16400: Bug: Assertion cp failed in microdescs_parse_from_string at
../src/or/routerparse.c:4168
-------------------------+-------------------------------------------------
     Reporter:  torkeln  |      Owner:
         Type:  defect   |     Status:  needs_review
     Priority:  major    |  Milestone:  Tor: 0.2.7.x-final
    Component:  Tor      |    Version:  Tor: 0.2.6.9
   Resolution:           |   Keywords:  routerparse.c, assertion,
Actual Points:           |  026-backport
       Points:           |  Parent ID:
-------------------------+-------------------------------------------------

Comment (by asn):

 Replying to [comment:18 nickm]:
 > I think I actually _do_ want to include the digest in the invalid list,
 just so that if it *does* somehow match one in the consensus, we don't try
 to re-fetch it.  See fix with test in my branch `bug16400_026` -- how does
 that look?

 Patch at `bug16400_026` looks OK.

 `cypherpunks_backup` approach also looks reasonable. I think the
 `tor_assert(cp);` should be removed as well.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/16400#comment:22>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs