Arlo Breault authored 11 years ago and Nick Mathewson committed 11 years ago

For consistency with the rest of buffer lengths in Tor.

See #8960

(This was the only wholly untested function in containers.c)

This catches tor_accept as well.

Peter Retzlaff authored 11 years ago and Nick Mathewson committed 11 years ago

- Preserve old eventdns code.
- Add function to close sockets cross-platform, without accounting.
- Add changes/ file.

(Bugfix on tests for #9288 fix; bug not in any released Tor)

George Kadianakis authored 11 years ago and Nick Mathewson committed 11 years ago

Both 'managed_proxy_list' and 'unconfigured_proxies_n' are global
src/or/transports.c variables that are not initialized properly when
unit tests are run.

Now a user who changes only NumEntryGuards will get the behavior she
expects. Fixes bug 9354; bugfix on 0.2.4.8-alpha.

Patch from CharlieB for ticket #9214

Fixes bug 9337; bugfix on 0.2.4.7-alpha.

Nick Mathewson authored 11 years ago and Andrea Shepard committed 11 years ago

When we moved channel_matches_target_addr_for_extend() into a separate
function, its sense was inverted from what one might expect, and we
didn't have a ! in one place where we should have.

Found by skruffy.

When we moved channel_matches_target_addr_for_extend() into a separate
function, its sense was inverted from what one might expect, and we
didn't have a ! in one place where we should have.

Found by skruffy.

This exercises the code that crashed and caused #9296.

Likely fix for the crash bug of #9296, which was introduced through a
combination of #7912 and #8586.  Bugfix not in any released Tor.

I added this so I could write a unit test for ServerTransportOptions,
but it incidentally exercises the succeed-on-defaults case of
options_validate too.

This removes some INLINE markers from functions that probably didn't
need them.