[tor-bugs] #34251 [Core Tor/Tor]: Fix edge case handling in Rust protover is supported
Tor Bug Tracker & Wiki
blackhole at torproject.org
Mon May 18 11:48:08 UTC 2020
#34251: Fix edge case handling in Rust protover is supported
-------------------------+-------------------------------------------------
Reporter: teor | Owner: (none)
Type: defect | Status: new
Priority: Medium | Milestone: Tor: 0.4.4.x-final
Component: Core | Version: Tor: 0.3.3.5-rc
Tor/Tor | Keywords: 044-should, technical-debt, rust,
Severity: Normal | protover, 042-backport, 043-backport
Actual Points: 0.1 | Parent ID:
Points: 0.5 | Reviewer:
Sponsor: |
-------------------------+-------------------------------------------------
Tor's Rust FFI for protocol_list_supports_protocol_or_later() returns true
for the empty protocol list.
In C, the function returns false, but this behaviour is undocumented.
This bug doesn't affect protocol_list_supports_protocol() in Rust, because
the Rust error checks are done in a different order.
I'll add a quick fix to #33222, but someone else will need to do the
backport. We might want to do the Rust error checks in the same order,
too.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/34251>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list