Opened 4 months ago

Last modified 2 weeks ago

#30916 merge_ready defect

assert in dimap_add_entry()

Reported by: dgoulet Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.1.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 (10)

comment:1 Changed 4 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 5 weeks ago by nickm

Keywords: 042-should added

comment:3 Changed 4 weeks ago by nickm

Owner: set to nickm
Status: newaccepted

comment:4 Changed 4 weeks 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 4 weeks ago by nickm

Status: acceptedneeds_review

comment:6 Changed 3 weeks ago by asn

Reviewer: teor

comment:7 Changed 3 weeks 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 3 weeks ago by nickm

Keywords: asn-merge added

comment:9 Changed 3 weeks 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 2 weeks ago by teor

Keywords: consider-backport-after-0423 added; consider-backport-after-0433 removed
Note: See TracTickets for help on using tickets.