Don't just tell the controller "foo" on id mismatch

Fixes bug 4169; bugfix on 0.2.1.1-alpha.

Init conn->addr to "unspec" on cpuworker connections

Fixes bug 4532 reported by "troll_un"

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

Conflicts:
src/or/connection_or.c

The conflict in src/or/connection_or.c is resolved by taking the
version in master, since e27a26d5 already fixed bug 4531 on master.
This merge just adds the changes file from 0.2.2.

Don't call tor_tls_set_logged_address till after checking conn->tls

Fixes bug 4531; partial backport of e27a26d5.

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

tor_accept_socket() should take tor_addr_t for listener arg

Fixes bug 4535; bugfix on 0.2.2.28-beta; found by "troll_un"

Give DirAllowPrivateAddress an explicit default

By convention, we say whether each bool's default is 0 or 1

Fixes 4536; found by "troll_un"

Resolve bug 3448: remove mention of tor-ops (which is not in use)

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

Fix bug 4530; check return val of tor_addr_lookup correctly

Fix on 0.2.1.5-alpha; reported by troll_un

Merge remote-tracking branch 'sebastian/coverity_strlen_fp'

Work around a false positive in Coverity.

Fixes cid 501 and 502.

Appease check-spaces

Don't segfault when checking whether a not-yet-used intro point should expire

Found by katmagic.  Bugfix on the #3460 branch, not yet in any release.

Fix warnings.

Remove environ declaration.
Use ORPort->value.  And it's a string.
Make tmp a char *.

Add an initializer to appease older gcc

Only define set_buffer_lengths_to_zero if bufferevents are enabled

Otherwise, on Windows, gcc will warn about the function being unused

Merge remote-tracking branch 'sjmurdoch/bug1983-port-tor-fw-helper-to-windows'

Make file descriptor type an unsigned integer

This avoids a warning from gcc (comparison between signed and unsigned
integer expressions [-Werror=sign-compare]), under Windows

Merge branch 'feature2553-v4-rebased'

Add some more documentation

Add support for tor2web mode via configure

Turn off LearnCircuitBuildTimeout when tor2web mode is on

Add ifdefs to disable assertion in connection_ap_handshake_send_begin

Add ifdefs to disable #3332 assertions

Use single-hop intro and rend circuits when in tor2web mode

Perform single-hop HS desc fetches when in tor2web mode

Don't allow tor2web-mode Tors to connect to non-HS addresses

The client's anonymity when accessing a non-HS address in tor2web-mode
would be easily nuked by inserting an inline image with a .onion URL, so
don't even pretend to access non-HS addresses through Tor.

Warn loudly on startup and SIGHUP if Tor is built for a non-anonymous mode

Add a compile-time #define to control whether Tor runs in 'tor2web mode'

The Tor2webMode torrc option is still required to run a Tor client in
'tor2web mode', but now it can't be turned on at runtime in a normal build
of Tor.  (And a tor2web build of Tor can't be used as a normal Tor client,
so we don't have to worry as much about someone distributing packages with
this particular pistol accessible to normal users.)

Support NAT-PMP on Windows

- Link in libws32 and libiphlpapi, needed for libnatpmp (both in
  ./configure and when compiling tor-fw-helper-natpmp.c)
- Define STATICLIB under Windows, to allow tor-fw-helper-natpmp.c to link
- Don't include arpa/inet.h which isn't present in Mingw32 and doesn't
  appear to be needed on either Windows or MacOS X

Merge branch 'bug933_nm_rebased_v2'

Conflicts:
src/test/test.c

Tweak addressmap_rewrite a little more

This resolves a loop warning on "MapAddress *.example.com
example.com", makes the rewrite log messages correct, and fixes the
behavior of "MapAddress *.a *.b" when just given "a" as an input.

Fix an issue in my mapaddress domains code spotted by arma

MapAddress *.torproject.org torproject.org would have been interpreted
as a map from a domain to itself, and would have cleared the mapping.
Now we require not only a match of domains, but of wildcards.

Add some post-comma spaces to please arma

Incidentally, we've got 30969 lines in master with a comma
in them, of which 1995 have a comma followed by a non-newline,
non-space character.  So about 93% of our commas are right,
but we have a substantial number of "crowded" lines.

Revise MapAddress manpage

make the asciidoc work; make the example more generic; tighten the
prose a little; be more specific that *.example.com matches
example.com; simplify an example.

Forbid remapping of *

It might be nice to support this someday, but for now it would fail
with an infinite remap cycle.  (If I say "remap * *.foo.exit",
then example.com ->
     example.com.foo.exit ->
     example.com.foo.exit.foo.exit ->
     example.com.foo.exit.foo.exit.foo.exit -> ...)

Refactor addressmap_match_superdomains and representation of wildcards

In this new representation for wildcarded addresses, there are no
longer any 'magic addresses': rather, "a.b c.d", "*.a.b c.d" and
"*.a.b *.c.d" are all represented by a mapping from "a.b" to "c.d". we
now distinguish them by setting bits in the addressmap_entry_t
structure, where src_wildcard is set if the source address had a
wildcard, and dst_wildcard is set if the target address had a
wildcard.

This lets the case where "*.a.b *.c.d" or "*.a.b c.d" remap the
address "a.b" get handled trivially, and lets us simplify and improve
the addressmap_match_superdomains implementation: we can now have it
run in O(parts of address) rather than O(entries in addressmap).

No new "LEGACY" tests allowed.

Fix compilation: get_options() now returns const

Address nickm's comments at https://trac.torproject.org/projects/tor/ticket/933#comment:8

    1. Only allow '*.' in MapAddress expressions. Ignore '*ample.com' and '.example.com'.
       This has resulted in a slight refactoring of config_register_addressmaps.
    2. Add some more detail to the man page entry for AddressMap.
    3. Fix initialization of a pointer to NULL rather than 0.
    4. Update the unit tests to cater for the changes in 1 and test more explicitly for
       recursive mapping.

Address nickm's comments at https://trac.torproject.org/projects/tor/ticket/933#comment:4

1. Implement the following mapping rules:

   MapAddress a.b.c d.e.f # This is what we have now
   MapAddress .a.b.c d.e.f # Replaces any address ending with .a.b.c with d.e.f
   MapAddress .a.b.c .d.e.f # Replaces the .a.b.c at the end of any addr with .d.e.f

   (Note that 'a.b.c .d.e.f' is invalid, and will be rejected.)

2. Add tests for the new rules.

3. Allow proper wildcard annotation, i.e. '*.d.e' '.d.e' will still work.

4. Update addressmap_entry_t with an is_wildcard member.

bug933 - Match against super-domains in MapAddress

Allow MapAddress to handle directives such as:

MapAddress .torproject.org .torserver.exit
MapAddress .org 1.1.1.1

Add tests for addressmap_rewrite.

Merge branch 'bug4538_v2'

Free the global DH parameters in crypto_global_cleanup().

Move crypto_global_cleanup() to the bottom of crypto.c.

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

Fix a memory leak in error path of my default-torrc code

Fixes Coverity CID # 500

Resolve a resource leak in test_util_split_lines

Fixes coverity CID # 488

appease "make check-spaces"

Merge branch 'ipv6_bridges_squashed'

Rename one more recalcitrant function.

Add changes file.

Warn user about client ignoring non-preferred IP address for a bridge.

Whitespace changes.

Make the router_get_*_orport interface consistent with node_*.

Make router_get_{prim,alt,pref}_addr_port take tor_addr_port_t *.

Rename to *_orport for consistency with node_*.

Merge node_get_{prim,pref,pref_ipv6}_addr with their _orport counterparts.

This keeps the IP address and TCP for a given OR port together,
reducing the risk of using an address for one address family with a
port of another.

Make node_get_addr() a wrapper function for compatibility.

Use correct address family where necessary for bridges on IPv6.

Use the preferred address and port when initiating a connection.

This is not as conservative as we could do it, f.ex. by looking at the
connection and only do this for connections to bridges.  A non-bridge
should never have anything else than its primary IPv4 address set
though, so I think this is safe.

Take IPv6 into account when rewriting routerinfo for a bridge and maintain ipv6_preferred.

Don't touch the string representation in routerinfo_t->address.

Also, set or clear the routerinfo_t->ipv6_preferred flag based on the
address family of the bridge.

Use preferred address when looking for bridges by routerinfo_t.

This should be safe to do for all uses of get_configured_bridge_by_routerinfo().

Clarify function documentation.

First chunk of support for bridges on IPv6

Comments below focus on changes, see diff for added code.

New type tor_addr_port_t holding an IP address and a TCP/UDP port.

New flag in routerinfo_t, ipv6_preferred.  This should go in the
node_t instead but not now.

Replace node_get_addr() with
- node_get_prim_addr() for primary address, i.e. IPv4 for now
- node_get_pref_addr() for preferred address, IPv4 or IPv6.

Rename node_get_addr_ipv4h() node_get_prim_addr_ipv4h() for
consistency.  The primary address will not allways be an IPv4 address.
Same for node_get_orport() -> node_get_prim_orport().

Rewrite node_is_a_configured_bridge() to take all OR ports into account.

Extend argument list to extend_info_from_node and
extend_info_from_router with a flag indicating if we want to use the
routers primary address or the preferred address.  Use the preferred
address in as few situtations as possible for allowing clients to
connect to bridges over IPv6.

Turn get_first_advertised_v4_port_by_type() into get_first_advertised_port_by_type_af().

Add some logging and comments.

Bridges can advertise a single IPv6 address with orport

This is deliberately more restrictive than we'd want to be.

Needs testing!

Initial support for simplest use of prop186 or-address lines

This lets a routerinfo_t have a single IPv6 or-address, and adds
support for formatting and parsing those lines.

Initial hacking for proposal 186.

This code handles the new ORPort options, and incidentally makes all
remaining port types use the new port configuration systems.

There are some rough edges!  It doesn't do well in the case where your
Address says one thing but you say to Advertise another ORPort.  It
doesn't handle AllAddrs.  It doesn't actually advertise anything besides
the first listed advertised IPv4 ORPort and DirPort.  It doesn't do
port forwarding to them either.

It's not tested either, it needs more documentation, and it probably
forgets to put the milk back in the refrigerator.

Add an assert before dereferencing entry_conn->socks_request

This may turn a segfault which katmagic saw into an assertion failure.

Merge remote-tracking branch 'rransom-tor/bug3460-v4'

Conflicts:
src/or/rendservice.c

Make unit tests run again. Fix bug 4606.

Merge remote-tracking branch 'rransom-tor/bug4605'

Add 'config-defaults-file' to getinfo_items table

Bugfix on commit 230422b955e1708f27f42cdd25e8b21a33fdd3dd, not yet in any
release; fixes bug #4605.

Set torrc_fname in load_torrc_from_disk

Bugfix on commit 230422b955e1708f27f42cdd25e8b21a33fdd3dd, not yet in any
release. Fixes bug #4604; reported by koolfy.

Merge branch 'bug4587_v2'

Set renegotiation callbacks immediately on tls inititation

This way, we can't miss a renegotiation attempt in a v2 handshake,
or miss excess renegotiation attempts.  Partial fix for bug 4587.

Correct manpage entry for DynamicDHGroups

Merge remote-tracking branch 'asn-mytor/bug4548_take2'

Merge branch 'disable_network'

Fix some DOCDOCs

Make sure we never launch an evdns resolve when DisableNetwork is 1

Merge branch 'multilevel_cfg'

Make pending libevent actions cancelable

This avoids a dangling pointer issue in the 3412 code, and should
fix bug 4599.

log a notice when disablenetwork is set

Manpage for DisableNetwork

New 'DisableNetwork' option to prevent Tor from using the network

Some controllers want this so they can mess with Tor's configuration
for a while via the control port before actually letting Tor out of
the house.

We do this with a new DisableNetwork option, that prevents Tor from
making any outbound connections or binding any non-control
listeners.  Additionally, it shuts down the same functionality as
shuts down when we are hibernating, plus the code that launches
directory downloads.

To make sure I didn't miss anything, I added a clause straight to
connection_connect, so that we won't even try to open an outbound
socket when the network is disabled.  In my testing, I made this an
assert, but since I probably missed something, I've turned it into a
BUG warning for testing.

Support for a defaults torrc file.

This will mainly help distributors by giving a way to set system or package
defaults that a user can override, and that a later package can replace.

No promises about the particular future location or semantics for this:
we will probably want to tweak it some before 0.2.3.x-rc

The file is searched for in CONFDIR/torrc-defaults , which can be
overridden with the "--defaults-torrc" option on the command line.

Add the ability to append and clear linelist options from cmdline

This will be important for getting stuff to work right across zones.

Make linelists always overridden by the command line

This starts an effort to refactor torrc handling code to make it easier
to live with.  It makes it possible to override exit policies from the
command line, and possible to override (rather than append to) socksport
lists from the command line.

It'll be necessary to make a "base" torrc implementation work at all.

Correct documentation comments for fields formerly named accepted_intros

Rename accepted_intros fields

Merge branch 'safer_params_squashed'

Implement consensus method 12 (proposal 178)

indent; add comment

Merge remote-tracking branch 'asn/bug4584'

Don't schedule excess_renegotiations_callback unless it's set

Partial fix for bug 4587; reported by "frosty_un".

Add a changes file.

Use random bytes as our certificate serial numbers.

Instead of using time(NULL) in our certificate serial numbers, use
eight random bytes as suggested in proposal 179.

Write dynamic DH parameters to a file.

Instead of only writing the dynamic DH prime modulus to a file, write
the whole DH parameters set for forward compatibility. At the moment
we only accept '2' as the group generator.

The DH parameters gets stored in base64-ed DER format to the
'dynamic_dh_params' file.