Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#3321 closed defect (fixed)

Edge case lets us still use an unconfigured bridge

Reported by: arma Owned by:
Priority: High Milestone: Tor: 0.2.2.x-final
Component: Core Tor/Tor Version:
Severity: Keywords: tor-bridge
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Our fix for #2511 made us throw away descriptors for unconfigured bridges when we read them from the disk cache. That's a good start.

But if we setconf X as our only bridge, then later setconf Y as our only bridge, we won't throw away X's descriptor so we'll still be willing to use it. Reported by anonym on #2355.

I just confirmed this bug.

Child Tickets

Change History (5)

comment:1 Changed 8 years ago by arma

See my bug3321 branch. I think this should resolve it.

comment:2 Changed 8 years ago by arma

Status: newneeds_review

comment:3 Changed 8 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Looks good, merging. The merge into 0.2.3 (6a320b99058ff19a2b) was a little tricky, so somebody should take a look at it.

comment:4 Changed 7 years ago by nickm

Keywords: tor-bridge added

comment:5 Changed 7 years ago by nickm

Component: Tor BridgeTor
Note: See TracTickets for help on using tickets.