[tor-bugs] #29537 [Core Tor/Tor]: verify intptr_t round-trip through void *
Tor Bug Tracker & Wiki
blackhole at torproject.org
Wed Mar 6 22:01:13 UTC 2019
#29537: verify intptr_t round-trip through void *
----------------------------------------+----------------------------------
Reporter: catalyst | Owner: (none)
Type: defect | Status: needs_revision
Priority: Medium | Milestone: Tor:
| 0.4.1.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: portability technical-debt | Actual Points:
Parent ID: | Points:
Reviewer: catalyst | Sponsor:
----------------------------------------+----------------------------------
Changes (by catalyst):
* status: needs_review => needs_revision
Comment:
Replying to [comment:7 rl1987]:
> The following patch checks that we can put all valid values of `int` and
`unsigned int` into `void *` and take them back. I suppose this is the
ranges of number we care about in practice, as we may want to put signed
and unsigned integer values into smartlist.
>
> https://github.com/torproject/tor/pull/724
Thanks! That seems to exhaustively check the entire `int` and `unsigned
int` ranges. It might be better to check some limited ranges, as nickm
suggested above. That way we can catch some likely failures without taking
too much time.
The check doesn't have to be exhaustive because this is mostly an interim
check until we stop doing this questionable thing in the future.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/29537#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list