[tor-bugs] #27741 [Core Tor/Tor]: too many arguments in rust protover_compute_vote()

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Oct 30 02:04:30 UTC 2018


#27741: too many arguments in rust protover_compute_vote()
-------------------------------------------------+-------------------------
 Reporter:  cyberpunks                           |          Owner:  nickm
     Type:  defect                               |         Status:
                                                 |  needs_revision
 Priority:  Very High                            |      Milestone:  Tor:
                                                 |  0.3.5.x-final
Component:  Core Tor/Tor                         |        Version:  Tor:
                                                 |  0.3.3.6
 Severity:  Normal                               |     Resolution:
 Keywords:  035-must, protover, memory-safety,   |  Actual Points:
  033-backport, 034-backport                     |
Parent ID:  #27739                               |         Points:
 Reviewer:  teor                                 |        Sponsor:
-------------------------------------------------+-------------------------
Changes (by teor):

 * status:  needs_review => needs_revision


Comment:

 Replying to [comment:18 cyberpunks]:
 > Replying to [comment:16 teor]:
 > > We only applied rustfmt in 0.3.5,
 >
 > That's true. Though we might as well try to follow the rustfmt rules for
 the commits merged to maint-0.3.3 too, right? If we introduce new style
 differences between the branches that weren't there before, future fixes
 to adjacent code will have unnecessary merge conflicts.

 It's nickm's code, so I'm going to let him decide if he wants to copy the
 rustfmt changes from 0.3.5 to 0.3.3.

 > Also, this ticket is missing the `rust` tag.

 Fixed!

 nickm, if you're happy with the branches as-is, please merge:
 * https://github.com/torproject/tor/pull/346 to 0.3.3 and later
 * https://github.com/torproject/tor/pull/431 to 0.3.5 and later

 If you decide to copy the rustfmt changes from 0.3.5 to 0.3.3, I can do a
 quick review.

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


More information about the tor-bugs mailing list