[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #18280 [Core Tor/Tor]: base32 encoding API doesn't work for a source length that is not a multiple of 5 or 8
#18280: base32 encoding API doesn't work for a source length that is not a multiple
of 5 or 8
----------------------------+------------------------------------
Reporter: dgoulet | Owner: dgoulet
Type: defect | Status: merge_ready
Priority: Medium | Milestone: Tor: 0.2.9.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: review-group-3 | Actual Points:
Parent ID: #18278 | Points: 1
Reviewer: nickm | Sponsor: SponsorR-can
----------------------------+------------------------------------
Changes (by dgoulet):
* status: needs_revision => merge_ready
Comment:
Replying to [comment:14 nickm]:
> So, if the tests really test correct behavior, then we can make it much
simpler. How about `bug18280_029_03_nm` ? (no guarantees.)
It does work yes! and much simpler.
Small detail, replace `buf` by `src` in this comment:
{{{
/* set v to the 16-bit value starting at buf[bits/8], 0-padded. */
}}}
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/18280#comment:16>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs