Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
changes/bug5090

bug5090 was already merged and corrected in the master branch's changelog.

bump to 0.2.3.13-alpha

cleanup and blurb for 0.2.3.13-alpha

fold in latest changes entries

explain that bug 5090 allows a post-auth heap overflow

resolves bug 5402.

Never disable debugger attachment for the unit tests

Shorten links to law.cornell.edu for exit-note file

The links we have currently redirect to those new files now.

merge in the changes files so far

Stop discarding command-line arguments when TestingTorNetwork is set

Discovered by Kevin Bauer. Fixes bug 5373; bugfix on 0.2.3.9-alpha,
where task 4552 added support for two layers of torrc files.

retroactively say which ticket the defaults torrc patch was

Fix spelling of Esteban Manchado Velázquez.

Close fds on pipe() error in tor_spawn_background().

Use a given name in the bug5090 message, at its holder's request.

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
src/or/circuitbuild.c

Never choose a bridge as an exit. Bug 5342.

Merge remote-tracking branch 'origin/maint-0.2.2'

Revise "sufficient exit nodes" check to work with restrictive ExitNodes

If you set ExitNodes so that only 1 exit node is accepted, the
previous patch would have made you unable to build circuits.

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
src/or/routerlist.c

Merge branch 'bug5343' into maint-0.2.2

Merge remote-tracking branch 'origin/maint-0.2.2'

Oops; credit bug5090 patch to flupzor. estebanm only found the bug.

Correctly handle broken escape sequences in torrc values

Previously, malformatted torrc values could crash us.

Patch by Esteban Manchado.  Fixes bug 5090; fix on 0.2.0.16-alpha.

changes file for extra-tests branch

Disable some tor_sscanf tests that gcc says are illegal formats

Fix new strcmp_opt/len tests on OSs where strcmp() can return values >1 or <-1

Add unit tests for path_is_relative

Some more corner cases for tor_sscanf

expand_filename tests for trailing slash in $HOME

Add some tests for expand_filename

Add two small tests for tor_sscanf

Add test for broken escapes in parse_config_...

Add some unit tests for parse_iso_time

Add tests for string_is_C_identifier

Improve coverage in string-related unit tests

Improve coverage in time-related unit tests

Remove unreliable assertion

* This assertion fails when executing the whole suite, but not when executing
  this test by itself
* Ideally I'd prefer starting with a guaranteed empty directory, but it's not
  very important in this case as non-existence of other paths is being checked
  explicitly

Add extra tests for wrap_string

Add extra tests for get_parent_directory

Tweak test case text so it's clearer

Improve eat_whitespace* unit tests

Put expected first in n_bits_set_u8 unit tests

Improve tor_split_lines unit tests

* Add some more test cases
* Switch to test_assert et al

Use test_streq in join_win_cmdline unit tests

Use test_eq et al in spawn_background unit tests

Use test_streq in format_helper_exit_status tests

Improve get_parent_directory unit tests

* Add more test cases to the get_parent_directory tests
* Switch the parameter order so that the expected value is the first one

Improve listdir unit tests

Improve a bit asprintf unit tests

Rewrite the find_str_at_start_of_line unit tests

Improve/extend strtok_r unit tests

* Add several failing tests (embedded in an "#if 0" block) for behaviour that
  doesn't match strtok_r
* Add another, passing, more interesting test
* Use test_eq_ptr(NULL, ...) instead of test_assert(NULL == ...)

Improve tor_sscanf tests

* Add many new test cases, tweak/improve existing ones, reorganize them a bit
* Switch the parameters in all test_eq calls so the expected value is the first
* Change all the "r = tor_sscanf(...);\ntest_eq(1, r)" to the more compact
  "test_eq(1, tor_sscanf(...))". It may be a tiny bit harder to find the
  tor_sscanf calls (it's the long lines anyway), but it saves a lot of lines,
  which should help readability.

Small fix in tor_sscanf documentation

Improve test case for read_escaped_data unit test

Trivial readability changes in mmap unit tests

Clean up and improve gzip unit tests

* Switch some test_eq parameters so the expected is always the first parameter
* Drop some manual checks of compressed format magic numbers (they're pointless
  and they make the unit tests less readable and more fragile, considering
  we're already indirectly checking those magic numbers via the
  detect_compression_method function)
* Add a couple of extra assertions

Add/improve the rest of the strmisc tests

Improve tor_snprintf tests

Add/improve tests for str{,case}cmp{start,end}

Add more/better tests in test_util_strmisc

Trivial cosmetic fix in test comments

Organize test_util_time in blocks

* Separate the tests for different functions with a comment
* Move some test init code further down, closer to where it was actually being
  used

Add test for comments in the config parser

Remove duplicated macro definition

Add test for broken hex parsing

* The test currently fails, but it's commented out (with an "#if 0")
* As a broken octal actually gives a parse error, it seems fair that this
  fails, too

Fix typo in config parser documentation

Improve config parser escaped content tests

Add tests for escaped content in config

Add quoted content tests for the config parser

Fix documentation URLs in config file comments

Require a threshold of exit nodes before building circuits

This mitigates an attack proposed by wanoskarnet, in which all of a
client's bridges collude to restrict the exit nodes that the client
knows about.  Fixes bug 5343.

Merge remote-tracking branch 'public/bug4361'

Conflicts:
src/or/command.c

Merge remote-tracking branch 'origin/maint-0.2.2'

Fix compile warnings in openbsd malloc

Merge remote-tracking branch 'origin/maint-0.2.2'

Merge remote-tracking branch 'karsten/geoip-march2012' into maint-0.2.2

Update to the March 2012 GeoIP database.

Remove misleading function comment (bug 5324)

In the distant past, connection_handle_read() could be called when there
are pending bytes in the TLS object during the main loop. The design
since then has been to always read all pending bytes immediately, so
read events only trigger when the socket actually has bytes to read.

Resolves bug 5324.

Merge remote-tracking branch 'public/bug4760'

Don't leak the env_vars_sorted smartlist in process_environment_make

Found by Coverity.

No changes/ file, because this bug has not been in a release yet.

Merge remote-tracking branch 'sebastian/bug5231'

Merge branch 'maint-0.2.2'

new ip address for maatuska

whitespace fix

Don't cannibalize already cannibalized circuits

This ensures we don't build circuits that have 5 hops or more. Patch
contributed by wanoskarnet, thanks!

When not fetching v2 dir info, don't require it for cleaning descriptors

Bugfix on 0.2.2.26-beta, which introduced the idea of caches not
cacheing v2 info.  Fixes bug 4838.

Merge remote-tracking branch 'sebastian/bug5161'

Merge remote-tracking branch 'pmezard/armv5'

Fix off-by-one error in test_util_make_environment

Fix crypto_hmac_sha256 documentation comment

Don't redeclare environ if std headers already did

This would cause a redundant redeclaration warning on some versions of
Linux otherwise.

document a cool attack that we evaluated

Use get_environment(), not environ.

Merge branch 'bug5105-v2-squashed'

Conflicts:
src/or/transports.c

Add unit tests for easily tested, non-trivial utility functions

Heap-allocate strings returned by get_current_process_environment_variables

Pass process_environment_t * to tor_spawn_background

Now tor_spawn_background's prototype is OS-independent.

Remove (void)envp from tor_spawn_background

The envp argument is used on Windows.

Rewrite managed proxy environment setup code

Now, the environment setup is entirely OS-independent, as well as less
hacky and brittle.

Change type of unixoid_environment_block to match tor_spawn_background's arg

Add set_environment_variable_in_smartlist