[tor-bugs] #12160 [Tor]: ORPort self-testing fails behind tcp proxy when using version 0.2.4.22

[Tor Bug Tracker & Wiki]

#12160: ORPort self-testing fails behind tcp proxy when using version 0.2.4.22
------------------------+---------------------------
     Reporter:  kargig  |      Owner:
         Type:  defect  |     Status:  new
     Priority:  normal  |  Milestone:
    Component:  Tor     |    Version:  Tor: 0.2.4.22
   Resolution:          |   Keywords:
Actual Points:          |  Parent ID:
       Points:          |
------------------------+---------------------------

Comment (by kargig):

 I've tried to bisect the problem but it proved to be rather hard because
 many versions would not compile. I started from 0.2.4.9 because I had
 already tested it and it wasn't working.

 Here's the bisect log:
 {{{
 git bisect start
 # bad: [0301a1df6c96888680ff8f310af818f239e93f13] Bump to 0.2.4.9-alpha-
 dev
 git bisect bad 0301a1df6c96888680ff8f310af818f239e93f13
 # good: [a2f57b97998b325c059d2fac06ca37d4b4dc52a3] bump to 0.2.4.3-alpha-
 dev
 git bisect good a2f57b97998b325c059d2fac06ca37d4b4dc52a3
 # bad: [08436b27ffba4094760fd1fe5321bbd255043b53] Merge remote-tracking
 branch 'origin/maint-0.2.3'
 git bisect bad 08436b27ffba4094760fd1fe5321bbd255043b53
 # bad: [3a33b1fe3bd7ba4cb1fff73f97ee722a2b127db5] Merge branch
 'move_contrib_source' of git://git.torproject.org/nickm/tor
 git bisect bad 3a33b1fe3bd7ba4cb1fff73f97ee722a2b127db5
 # skip: [b208539b8047a12fb2f1f794c9932fddd577dfdb] Use circuitmux_t in
 channels and when relaying cells
 git bisect skip b208539b8047a12fb2f1f794c9932fddd577dfdb
 # good: [feabf4148fc00a8535714ff72d9caa8303a73eaf] Drop support for
 openssl 0.9.7
 git bisect good feabf4148fc00a8535714ff72d9caa8303a73eaf
 # skip: [e4a11b890e7c5fe45dc1f5f271fbd8130ccc9c55] Implement
 circuitmux_alloc()/circuitmux_free() and chanid/circid->muxinfo hash table
 git bisect skip e4a11b890e7c5fe45dc1f5f271fbd8130ccc9c55
 # good: [dc014c97472e3adf2306938841c13de0040a2ff0] Merge branch
 'maint-0.2.3'
 git bisect good dc014c97472e3adf2306938841c13de0040a2ff0
 # skip: [3c41d7f414511aeb6e9e0fd6bfb9be1af539840a] Implement
 circuitmux_attach_circuit() in circuitmux.c
 git bisect skip 3c41d7f414511aeb6e9e0fd6bfb9be1af539840a
 # bad: [965c9de498ab7f6c7ce3dce133bb34456f3d668e] Abolish superfluous
 channel_find_by_remote_nickname()
 git bisect bad 965c9de498ab7f6c7ce3dce133bb34456f3d668e
 # good: [751b3aabb5ab88fca16834e559a8d9835831b05f] Merge remote-tracking
 branch 'public/openssl_1_is_best'
 git bisect good 751b3aabb5ab88fca16834e559a8d9835831b05f
 # skip: [28f108bcceab59fcf9f27e33065f64bfdb0f159a] Use dirreq_id from
 channel_t when appropriate
 git bisect skip 28f108bcceab59fcf9f27e33065f64bfdb0f159a
 # bad: [cb62a0b69a7d67b427224ca4c3075b49853a3a1f] Use
 channel_is_bad_for_new_circs(), connection_or_get_num_circs() in main.c
 git bisect bad cb62a0b69a7d67b427224ca4c3075b49853a3a1f
 # skip: [35924435d22c2469ecbe06156c8069a928859d63] Make reachabiity test
 in dirserv.c use channel_t
 git bisect skip 35924435d22c2469ecbe06156c8069a928859d63
 # skip: [8b14db9628f0e8982e894034e86c8efdd78cff32] Switch onion.c over to
 channel_t
 git bisect skip 8b14db9628f0e8982e894034e86c8efdd78cff32
 # skip: [e136f7ccb4e671e33b6c92a48df819082291f5c1] Convert relay.c/relay.h
 to channel_t
 git bisect skip e136f7ccb4e671e33b6c92a48df819082291f5c1
 # skip: [15303c32ec9d84aff8de5ed9df28e779c36c6e5c] Initial
 channeltls.c/channeltls.h for bug 6465
 git bisect skip 15303c32ec9d84aff8de5ed9df28e779c36c6e5c
 # skip: [4768c0efe3e9471cc367c3740d1a4ba0ab79626c] Support channel_t in
 connection_edge.c
 git bisect skip 4768c0efe3e9471cc367c3740d1a4ba0ab79626c
 # skip: [32337502f11e6c84e4db8591f5f81c4fc6d1da58] Use channel_t rather
 than or_connection_t for circuits
 git bisect skip 32337502f11e6c84e4db8591f5f81c4fc6d1da58
 # skip: [6cce6241dd4405f6cf21057f9913e07633cf18bb] Query circuit count
 from associated channel of or_conn in control.c
 git bisect skip 6cce6241dd4405f6cf21057f9913e07633cf18bb
 # skip: [f0f87cb68a22feaf552a18b521d3313d843f8793] Convert rendmid.c to
 channel_t
 git bisect skip f0f87cb68a22feaf552a18b521d3313d843f8793
 # skip: [519c971f6a3b89f1e81cda3c0290d4d943ec0d78] Use channel_t in cmd.c
 git bisect skip 519c971f6a3b89f1e81cda3c0290d4d943ec0d78
 # skip: [7f952da55334d3a3693d1c6e8531fd96730265db] Fix make check-spaces
 in circuitbuild.c and router.h
 git bisect skip 7f952da55334d3a3693d1c6e8531fd96730265db
 # skip: [77dac97354974e8a819d8e35ad4e7a76199999b4] Use channel_t in
 cpuworker.c
 git bisect skip 77dac97354974e8a819d8e35ad4e7a76199999b4
 # skip: [838743654c1bed2bfe22789ff53a1993c005f176] Add channel.c/channel.h
 for bug 6465
 git bisect skip 838743654c1bed2bfe22789ff53a1993c005f176
 # skip: [9ad7ba9f2267a9ee34fafda9356f1fa86633f00f] Use
 connection_or_get_num_circuits() in control.c
 git bisect skip 9ad7ba9f2267a9ee34fafda9356f1fa86633f00f

 There are only 'skip'ped commits left to test.
 The first bad commit could be any of:
 35924435d22c2469ecbe06156c8069a928859d63
 e136f7ccb4e671e33b6c92a48df819082291f5c1
 4768c0efe3e9471cc367c3740d1a4ba0ab79626c
 6cce6241dd4405f6cf21057f9913e07633cf18bb
 519c971f6a3b89f1e81cda3c0290d4d943ec0d78
 77dac97354974e8a819d8e35ad4e7a76199999b4
 32337502f11e6c84e4db8591f5f81c4fc6d1da58
 8b14db9628f0e8982e894034e86c8efdd78cff32
 15303c32ec9d84aff8de5ed9df28e779c36c6e5c
 28f108bcceab59fcf9f27e33065f64bfdb0f159a
 7f952da55334d3a3693d1c6e8531fd96730265db
 f0f87cb68a22feaf552a18b521d3313d843f8793
 838743654c1bed2bfe22789ff53a1993c005f176
 9ad7ba9f2267a9ee34fafda9356f1fa86633f00f
 cb62a0b69a7d67b427224ca4c3075b49853a3a1f
 We cannot bisect more!

 }}}

 I've also tested release-0.2.3 branch and
 e318ab14b10f353da1ebcece0d6490191517e21a works fine.

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