Opened 8 months ago

Closed 4 months ago

Last modified 4 months ago

#30916 closed defect (fixed)

assert in dimap_add_entry()

Reported by: dgoulet Owned by: nickm
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Normal Keywords: crash, 042-should consider-backport-after-0423 035-backport 040-backport 041-backport BugSmashFund
Cc: Actual Points: .1
Parent ID: Points:
Reviewer: teor Sponsor:


From tor-relays@:

The stack trace is:

Jun 18 13:33:31.000 [notice] Bootstrapped 0%: Starting
Jun 18 13:33:32.000 [notice] Starting with guard context "default"

============================================================ T= 1560854012
INTERNAL ERROR: Raw assertion failed at ../src/lib/ctime/di_ops.c:179: ! 
Jun 18 13:33:33.000 [notice] Tor opening log file.

It appears that tor tried to add the same value in the di_digest256_map_t twice.

Logs indicate

Child Tickets

Change History (13)

comment:1 Changed 8 months ago by nickm

I think this is probably caused by having an ntor key stored as the "old ntor key" before it is removed as the "current ntor key". For a workaround, removing secret_onion_key_ntor.old should be sufficient.

comment:2 Changed 6 months ago by nickm

Keywords: 042-should added

comment:3 Changed 5 months ago by nickm

Owner: set to nickm
Status: newaccepted

comment:4 Changed 5 months ago by nickm

Actual Points: .1
Keywords: 035-backport 040-backport 041-backport BugSmashFund added

See branch bug30915_035 with PR at .

It merges cleanly to 0.4.0, but hits a conflict for 0.4.1. See branch bug30915_041 with PR at .

I'll put this in needs_review once CI is passing.

comment:5 Changed 5 months ago by nickm

Status: acceptedneeds_review

comment:6 Changed 5 months ago by asn

Reviewer: teor

comment:7 Changed 5 months ago by teor

Keywords: consider-backport-after-0433 added
Status: needs_reviewmerge_ready
Version: Tor:

Looks good to me, pretty simple change.

comment:8 Changed 5 months ago by nickm

Keywords: asn-merge added

comment:9 Changed 5 months ago by asn

Keywords: asn-merge removed
Milestone: Tor: 0.4.2.x-finalTor: 0.4.1.x-final

Merged. Leaving open for backports.

comment:10 Changed 5 months ago by teor

Keywords: consider-backport-after-0423 added; consider-backport-after-0433 removed

comment:11 Changed 4 months ago by nickm

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

Backported to 0.4.1. Marking for possible further backport.

comment:12 Changed 4 months ago by teor

Milestone: Tor: 0.4.0.x-finalTor: 0.2.9.x-final
Resolution: fixed
Status: merge_readyclosed

Merged to 0.3.5 and later.
Merged #31107, #31466, #30916, #31408, #31837, and #31897 together.

comment:13 Changed 4 months ago by teor

Milestone: Tor: 0.2.9.x-finalTor: 0.3.5.x-final
Note: See TracTickets for help on using tickets.