Opened 2 years ago

Closed 2 years ago

Last modified 9 months ago

#16360 closed defect (fixed)

tor_main continues after failed crypto_early_init in tor_init

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.2.5.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: 026-backport, 2016-bug-retrospective
Cc: nickm Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

A typo in the return value from tor_init means that if crypto_early_init fails, tor_main can continue running, rather than returning with an error value.

This bug was introduced in commit d3fb846d8c98 possibly related to feature #4900, first released in 0.2.5.2-alpha.

Given it's a very simple fix, where tor continues to run half-initialised, we might want to backport it.

I'll post a branch very soon.

Child Tickets

Change History (4)

comment:1 Changed 2 years ago by teor

Branch: bug16360-failed-crypto-early-init
Repository: https://github.com/teor2345/tor.git
Content:

  • Make return value on failed crypto_early_init be -1, not 1
  • Changes file

comment:2 Changed 2 years ago by nickm

Resolution: fixed
Status: newclosed

Cherry-picked to 0.2.5 and merged forwards. Thanks!

comment:3 Changed 18 months ago by nickm

Keywords: 2016-bug-retrospective added

Mark more tickets for bug retrospective based on hand-review of changelogs from 0.2.5 onwards.

comment:4 Changed 9 months ago by nickm

Milestone: Tor: 0.2.7.x-finalTor: 0.2.5.x-final
Severity: Normal

(Verified that this bug doesn't exist in 0.2.4.)

Note: See TracTickets for help on using tickets.