Opened 10 years ago

Closed 10 years ago

Last modified 8 years ago

#2403 closed defect (fixed)

Tor can choose a relay as both entry and exit node for a circuit

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


2011-01-17 02:28 <piebeer> Every time when i'm going to remove "consider_exit_family = 0; goto retry;" for choose_random_entry(), I thoung that it's something wrong with me, not with a func. Till today. I found reason why that with "smartlist_add(exit_family, chosen_exit)" is a fail.
2011-01-17 02:28 <piebeer> "EntryNodes"+"StrictEntryNodes" is non recomended, so we can ignore such case. Bridge that has allowed exit policy is a bug, so we can ignore that too.
2011-01-17 02:29 <piebeer> But. We have a 1776 that allowed to use any guard as bridge. for some time exit relay can be guard.
2011-01-17 03:25 <piebeer> Hi. What can I do if circuit have the first node the same as the last node?
2011-01-17 03:48 <piebeer> I'm sure: 1. used entrynodes+strict*nodes (why need it allowed if it danger) 2. bridge addr_of_fast_relay (it's exit too)
2011-01-17 04:17 <piebeer> "then be flexible about families" and chosen exit. Better to have every build try to success and if choose_random_entry() returned chosen_exit so what, it's a price for success.

Child Tickets

Change History (6)

comment:1 Changed 10 years ago by nickm

Milestone: Tor: 0.2.2.x-final

comment:2 Changed 10 years ago by arma

Status: newneeds_review

The bug2403 branch in my arma should resolve this one.

comment:3 Changed 10 years ago by arma

This patch made me open #2616, which I think can wait and be solved later (i.e. in master).

comment:4 Changed 10 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Merged; thanks!

comment:5 Changed 8 years ago by nickm

Keywords: tor-client added

comment:6 Changed 8 years ago by nickm

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