[tor-bugs] #17668 [Tor]: moria1, with updated v3 cert: Bug: Generated a networkstatus consensus we couldn't parse.

[Tor Bug Tracker & Wiki]

#17668: moria1, with updated v3 cert: Bug: Generated a networkstatus consensus we
couldn't parse.
-------------------------------------------------+-------------------------
 Reporter:  arma                                 |          Owner:  nickm
     Type:  defect                               |         Status:
 Priority:  High                                 |  needs_revision
Component:  Tor                                  |      Milestone:
 Severity:  Blocker                              |        Version:
 Keywords:  201512-deferred, TorCoreTeam201602,  |     Resolution:
  must-fix-before-028-rc                         |  Actual Points:
Parent ID:                                       |         Points:
 Reviewer:                                       |        Sponsor:
-------------------------------------------------+-------------------------

Comment (by nickm):

 Replying to [comment:36 Sebastian]:
 > typos:
 >
 > +    /* If not enough authorties listed this exact <ed,rsa> pair,
 >
 > +   * by_rsa_sha1 above. We __an__ include <NULL,RSA-SHA1> entries for
 votes that

 These two are fixed in fixup commits.

 >
 > dir-spec: Looks fine, but I think we should include a note about needing
 to look at the consensus to verify that an ed key listed in a descriptor
 is actually valid, at least until the ed key becomes the canonical key.

 Calling this `S1`.

 > Comment from above that didn't get a reply, but I think it should:
 >
 > "Should we add asserts to the functions that say "this may only be
 called after dircollator_collate"? seems like we have the is_collated
 field for that purpose."

 Calling this `S2`.

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