Opened 23 months ago

Closed 22 months ago

Last modified 22 months ago

#27205 closed defect (implemented)

add unit test for double-voting

Reported by: cypherpunks3 Owned by:
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Normal Keywords: protover
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


protover.c has a comment about very important code that guards against double-voting:

  smartlist_uniq_strings(expanded); // This makes voting work. do not remove

But no test fails if this is removed.

Child Tickets

Change History (3)

comment:1 Changed 23 months ago by cypherpunks3

See branch doublevote1 at ​​​​​

comment:2 Changed 23 months ago by nickm

Milestone: Tor: 0.3.5.x-final
Status: newneeds_review

comment:3 Changed 22 months ago by nickm

Resolution: implemented
Status: needs_reviewclosed

lgtm; merged to master.

Note: See TracTickets for help on using tickets.