Opened 14 months ago

Closed 11 months ago

Last modified 8 months ago

#26943 closed defect (fixed)

Privcount blinding and encryption: Safety fixes

Reported by: teor Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: privcount, 035-roadmap-master, 035-triaged-in-20180711, rust
Cc: teor, nickm, chelseakomlo Actual Points: .1
Parent ID: #25669 Points:
Reviewer: teor Sponsor: SponsorV

Child Tickets

Change History (9)

comment:1 Changed 14 months ago by teor

Owner: changed from teor to nickm

comment:2 Changed 13 months ago by nickm

Status: assignedneeds_review

I've done this as the middle part of my branch ticket26939_26943_26940; PR at . All of the unwrap() and assert! calls are now confined to the tests.

This makes a lot more things return a Result than did before, but thanks to the ? operator, the code is fairly clean.

Also, it is really cool that Result<Vec<X>,Y> implements FromIterator<Result<X,Y>>.

comment:3 Changed 13 months ago by asn

Reviewer: teor

comment:4 Changed 13 months ago by nickm

Sponsor: SponsorV

comment:5 Changed 12 months ago by nickm

Milestone: Tor: 0.3.5.x-finalTor: 0.3.6.x-final

Deferring privcount tickets in 0.3.5 to 0.3.6

comment:6 Changed 12 months ago by teor

Status: needs_reviewmerge_ready

Commits 3-5 look good.

The CI is here:

comment:7 Changed 11 months ago by nickm

Resolution: fixed
Status: merge_readyclosed

Merged the branch !

comment:8 Changed 11 months ago by nickm

Milestone: Tor: 0.3.6.x-finalTor: 0.4.0.x-final

Tor 0.3.6.x has been renamed to 0.4.0.x.

comment:9 Changed 8 months ago by nickm

Actual Points: .1
Note: See TracTickets for help on using tickets.