Opened 3 years ago

Closed 14 months ago

#24182 closed defect (wontfix)

Pad INTRODUCE cells to hide the number of link specifiers

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: prop224, tor-hs, defence-in-depth, 034-triage-20180328, 034-removed-20180328
Cc: Actual Points:
Parent ID: #23493 Points: 0.5
Reviewer: Sponsor:


Before #23577, there are two sizes of introduce cell data:

  • legacy id and IPv4 (24 bytes)
  • legacy id and IPv4 and ed25519 id (56 bytes)

After #23577: there are another two sizes:

  • legacy id and IPv4 and IPv6 (40 bytes)
  • legacy id and IPv4 and ed25519 id and IPv6 (72 bytes)

And if we ever remove IPv4 addresses, or add another link specifier, we create even more sizes.

This is ok at the moment, because we pad all cells to HS_CELL_INTRODUCE1_MIN_SIZE (246 bytes), and no cells are larger than this size. But we should make sure we are padding larger cells to the maximum cell length, to avoid splitting the rendezvous point anonymity set with future link specifiers.

Child Tickets

Change History (7)

comment:1 Changed 3 years ago by dgoulet

Owner: set to dgoulet
Status: newassigned

comment:2 Changed 3 years ago by dgoulet

Milestone: Tor: 0.3.3.x-finalTor: 0.3.4.x-final

Move 033 ticket I own to 034

comment:3 Changed 3 years ago by nickm

Keywords: 034-triage-20180328 added

comment:4 Changed 3 years ago by nickm

Keywords: 034-removed-20180328 added

Per our triage process, these tickets are pending removal from 0.3.4.

comment:5 Changed 3 years ago by nickm

Milestone: Tor: 0.3.4.x-finalTor: unspecified

These tickets, tagged with 034-removed-*, are no longer in-scope for 0.3.4. We can reconsider any of them, if time permits.

comment:6 Changed 17 months ago by gaba

Owner: dgoulet deleted

Releasing some old tickets.

comment:7 Changed 14 months ago by teor

Resolution: wontfix
Status: assignedclosed

I think we decided not to make this change.

Note: See TracTickets for help on using tickets.