Comment the heck out of the stream/circuit attaching process.

Document main.c a lot better

Resolve memory leaks in test_dir_networkstatus_compute_bw_weights_v10

Convert remaining files to smartlist_add_strdup

The coccinelle script produced errors with these test files so
convert the remaining cases of smartlist_add to
smartlist_add_strdup by hand.

Automated change to use smartlist_add_strdup

Use the following coccinelle script to change uses of
smartlist_add(sl, tor_strdup(str)) to
smartlist_add_strdup(sl, string) (coccinelle script from nickm
via bug 20048):

@@
expression a;
expression b;
@@
- smartlist_add
+ smartlist_add_strdup
   (a,
- tor_strdup(
   b
- )
  )

Add implementation of smartlist_add_strdup

Add smartlist_add_strdup(sl, string) - replaces the use of
smartlist_add(sl, tor_strdup(string)). Fixes bug 20048.

Merge branch 'ticket14881-v3'

Small refactoring: explain G,M,E,D,T initialization

Add historic bwweight tests, comments, line len fixes

Add consensus weight calculation tests

Fix default bw weights with new consensus method

See #14881

Merge remote-tracking branch 'pastly/ticket20459'

Merge branch 'maint-0.2.9'

Avoid tor_fragile_assert() failure with DNSPort on RESOLVED_TYPE_ERROR

The tor_fragile_assert() bug has existed here since c8a5e2d588e0d91
in tor-0.2.1.7-alpha forever, but tor_fragile_assert() was mostly a
no-op until 0.2.9.1-alpha.

Fixes bug 19869.

Continue the module documentation effort with circuitbuild and circuituse

More module documentation (circpathbias, circuitlist)

Regenerate trunnel code with trunnel 1.5

More module-level documentation.

Whitespace fixes

Allow asking a bridge's own descriptor over one-hop connection

When we refactored purpose_needs_anonymity(), we made it so _all_
bridge requests required anonymity.  But that missed the case
that we are allowed to ask a bridge for its own descriptor.

With this patch, we consider the resource, and allow "authority.z"
("your own descriptor, compressed") for a bridge's server descriptor
to be non-anonymous.

Fix for bug 20410; bug not in any released Tor.

Fix ewma_cmp_cmux never considering policies different

Merge remote-tracking branch 'pastly/ticket20273'

Merge remote-tracking branch 'dgoulet/bug20435_030_01'

Module-document dnsserv.c, policies.c, routerkeys.c

html error in doxygen comment in rendservice.c

doxygen says these options are obsolete

Module documentation for config.c and confparse.c

Module documentation (and an important caveat) for dirvote.c

man: Fix default value of AuthDirGuardBWGuarantee to 2MB

Closes #20435

Signed-off-by: David Goulet <dgoulet@torproject.org>

Turn scheduler.c documentation into doxygen.

changes file for 20389

Say 'Invalid argument' instead of unclear 'Unrecognized'

BUG in purpose_needs_anonymity if switch not matched.

I believe that this should never trigger, but if it does, it
suggests that there was a gap between is_sensitive_dir_purpose and
purpose_needs_anonymity that we need to fill.  Related to 20077.

Merge remote-tracking branch 'chelseakomlo/master'

Compilation fix. (function pointer implicit cast in tests.)

Merge remote-tracking branch 'andrea/ticket19858_v2'

Conflict in entrynodes.c: any_bridge_supports_microdescriptors was
removed in master, and modified in 19858_v2

Unify code in channel_write_*cell()

Patch from pingl; patch for 13827.

Merge remote-tracking branch 'arma/bug6769'

Bump version on master to 0.3.0.0-alpha-dev.

Refactor purpose_needs_anonymity to use switch statement

Refactor to use purpose_needs_anonymity and remove is_sensitive_dir_purpose

changes file for module docs

Module-level docs for ext_orport and router.c

Turn replaycache.c module doc into doxygen

Document connection_or.c and connection.c at module level

Module documentation for circuitmux_ewma.c

Turn circuitmux.c comments into module docs.

Module docs for channel.c and channeltls.c

Merge branch 'maint-0.2.8'

"ours" commit to avoid bumping version.

Bump to 0.2.8.9-dev

Bump master to 0.2.9.4-alpha-dev

release-notes for 0.2.8.9

Merge branch 'maint-0.2.8'

("ours" merge to avoid taking version bump)

bump to 0.2.8.9

Fold 20384 into changelog

Update the buffer sentinel fix to work with our #20081 fix.

Merge branch 'maint-0.2.8'

Merge branch 'buf_sentinel_026_v2' into maint-0.2.8

Add a one-word sentinel value of 0x0 at the end of each buf_t chunk

This helps protect against bugs where any part of a buf_t's memory
is passed to a function that expects a NUL-terminated input.

It also closes TROVE-2016-10-001 (aka bug 20384).

changelog typo fixes from Rubiate

Bump version to 0.2.9.4-alpha.

edits to changelog from kate and david

Wrap the changelog, add a blurb

Start on an 0.2.9.4-alpha changelog

Fix a syntax problem

changes file for module docs

Merge branch 'module_docs_1'

Write a bunch of module documentation.

This commit adds or improves the module-level documenation for:

  buffers.c circuitstats.c command.c connection_edge.c control.c
  cpuworker.c crypto_curve25519.c crypto_curve25519.h
  crypto_ed25519.c crypto_format.c dircollate.c dirserv.c dns.c
  dns_structs.h fp_pair.c geoip.c hibernate.c keypin.c ntmain.c
  onion.c onion_fast.c onion_ntor.c onion_tap.c periodic.c
  protover.c protover.h reasons.c rephist.c replaycache.c
  routerlist.c routerparse.c routerset.c statefile.c status.c
  tor_main.c workqueue.c

In particular, I've tried to explain (for each documented module)
what each module does, what's in it, what the big idea is, why it
belongs in Tor, and who calls it.  In a few cases, I've added TODO
notes about refactoring opportunities.

I've also renamed an argument, and fixed a few DOCDOC comments.

Add a logic-consistency test when parsing unix socket configs

Coverity couldn't tell that 'addrport' was always non-NULL at this
point in the function, so I'm adding an explicit check. This closes
CID 1374060.

Whitespace cleaning

adding instructions how to include new test files

Merge branch 'bug20176_v2'

Merge remote-tracking branch 'public/spaces_in_unix_addrs'

Make the FreeBSD ersatz_socketpair test even more skippable.

(This is safe, since only windows actually -uses- erstaz_socketpair.)

Merge branch 'bug18357_v2'

Extract ExitPolicy-and-IPv6Exit check into a new function

(I've done this instead of changing the semantics of
router_compare_to_my_exit_policy, because dns.c uses
router_compare_to_my_exit_policy too, in a slightly weird way.)

Refactor to always allocate chosen_entry_guards in new guard_selection_new() function

Use tor_memeq() instead of tor_memcmp() per code review

Adjust comment per code review

ersatz socketpair tests: work around freebsd jails.

Tweak patch for 18529.

- function doesn't need to be inline.

- rename function

- Make documentation more pedantically correct

- Remove needless "? 1 : 0."

Remove duplicate code that checks for default authorities

Patch from ericho.

Fixes 18529. Simple refactoring.

Fix a bug in displaying IPv6 addrs in test_op_ipv6_ with --verbose

The test code, if it failed, or if it was run in verbose mode, would
use the wrong variable for its loop.  Patch from rubiate uploaded to
19999.

Switch from "AF_UNIX is always equal" to "always unequal" to avoid wacky bugs. See discussion on 20261

Merge remote-tracking branch 'yawning-schwanenlied/bug20261'

Fix a new compilation warning with broken-mulodi i386 clang builds. :(

Fix spurious compiler warning in do_getpass().

Some compilers apparently noticed that p2len was allowed to be equal
to msg, and so maybe we would be doing memset(prompt2, ' ', 0), and
decided that we probably meant to do memset(prompt2, 0, 0x20);
instead.

Stupid compilers, doing optimization before this kind of warning!

My fix is to just fill the entire prompt2 buffer with spaces,
because it's harmless.

Bugfix on e59f0d4cb964387c5, not in any released Tor.

torrc parsing b0rks on carriage-return

(Specifically, carriage return after a quoted value in a config
line. Fixes bug 19167; bugfix on 0.2.0.16-alpha when we introduced
support for quoted values. Unit tests, changes file, and this
parenthetical by nickm.)

Merge remote-tracking branch 'asn/bug19223'

Bug 20261: Treat AF_UNIX addresses as equal when comparing them.

This is a kludge to deal with the fact that `tor_addr_t` doesn't contain
`sun_path`.  This currently ONLY happens when circuit isolation is being
checked, for an isolation mode that is force disabled anyway, so the
kludge is "ugly but adequate", but realistically, making `tor_addr_t`
and the AF_UNIX SocksPort code do the right thing is probably the better
option.

Fix non-triggerable heap corruption at do_getpass().

Merge branch 'no_openssl_100'

Stop implying that we support openssl 1.0.0; we don't.

Closes ticket 20303.

The LIBRESSL_VERSION_NUMBER check is needed because if our openssl
is really libressl, it will have an openssl version number we can't
really believe.

comment tweak. Fixes 20271. patch from pastly.

Merge branch 'maint-0.2.8'

Merge branch 'maint-0.2.7' into maint-0.2.8

Merge branch 'maint-0.2.6' into maint-0.2.7

Merge branch 'maint-0.2.5' into maint-0.2.6

Merge branch 'maint-0.2.4' into maint-0.2.5

Only use -levent when checking functions if we will use it to link.

Fixes 19904; bugfix on b62abf9f21499ab; patch from Rubiate.