[tor-bugs] #6877 [Tor]: Finally replace all char[] buffers with uint8_t[] buffers
Tor Bug Tracker & Wiki
blackhole at torproject.org
Tue Oct 22 22:54:31 UTC 2013
#6877: Finally replace all char[] buffers with uint8_t[] buffers
------------------------+--------------------------------
Reporter: nickm | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: Tor: 0.2.5.x-final
Component: Tor | Version:
Resolution: | Keywords: tor-client
Actual Points: | Parent ID:
Points: |
------------------------+--------------------------------
Comment (by unixninja92):
Replying to [comment:9 nickm]:
> c48c2fb5979ad7189fb9f37a25e250e9379ad9eb:
> * Looks okay.
> * For crypto_digest_smartlist, since it's defined to take a list of
NUL-terminated strings, I think it makes sense to have them remain char*.
I made *append a char for crypto_dyget_smartlist again in
299de8cafb60bab080268dfef8a3f371e34439de. I left *digest_out as a uinit_8
because all the other digests in crypto are now uinit_8 and it would
require a lot of casting to change it.
>
> 24f08854b9e3efa94158ccb9b1f161f1a27b5e4a:
> * Wow, that's a big one...
> * In dirvote.c, since fingerprint is a hex string, it probably makes
sense to have it stay char.
This should be fixed in 6211cd4e1d8c093e64ee6cf0195694c08abb57ed.
> * Otherwise looks okay.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/6877#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list