[tor-bugs] #19142 [Core Tor/Tor]: Remove the environ configure check

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue May 24 09:53:44 UTC 2016

#19142: Remove the environ configure check
 Reporter:  cypherpunks   |          Owner:
     Type:  defect        |         Status:  new
 Priority:  Medium        |      Milestone:
Component:  Core Tor/Tor  |        Version:
 Severity:  Trivial       |     Resolution:
 Keywords:                |  Actual Points:
Parent ID:                |         Points:
 Reviewer:                |        Sponsor:

Comment (by cypherpunks):

 It turns out the `environ` variable cannot be redeclared because of
 `-Wredundant-decls`. Instead i looked into why it was added and if it can
 be removed.

 It was initially added in commit
 efb7b9dec135594718b765234bc1f745855f60d2] because (according to the
 comment) FreeBSD needed it. The
 man page] (all the way back to FreeBSD 1.0) tells us otherwise. If someone
 could give the reasoning why FreeBSD requires declaring `environ` that
 would be helpful because i don't have a FreeBSD machine and cannot test it

 In commit
 bc66878bdea0250991fc99b2d023146f67a6f4bb] the environ configure check was
 added because it gave redundant declaration warnings on Linux. These were
 the exact same warnings i got when testing redeclaring `environ`

Ticket URL: <https://trac.torproject.org/projects/tor/ticket/19142#comment:1>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online

More information about the tor-bugs mailing list