Fix whitespace

Merge branch 'bug6524_nm'

Changes file for nonrecursive make branch

Update .gitignore with new autotools droppings from nonrecursive make

Better fix to avoid loudness on mkdir -p

(commit message by nickm)

build: minimal adjustments to make out-of-tree build work

Merge remote-tracking branch 'linus/bug6364'

Merge remote-tracking branch 'linus/bug6362'

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

Merge remote-tracking branch 'public/bug6472' into maint-0.2.3

Make node_assert_ok less duplicatey

This comes at the cost of making its failure message a little less
friendly, but since when do assertion failures count as
user-friendly?

Clarify docs on get_configured_bridge_by_*_digest

We now need automake 1.9 or later

Automake 1.7 is too broken to even investigate why it broke at this point.

Move ipv6_preferred from routerinfo_t to node_t.

Move extend_info_from_router() from circuitbuild.c to router.c and
make it static.

Add get_configured_bridge_by_orports_digest() and have
get_configured_bridge_by_routerinfo() and
node_is_a_configured_bridge() use it. We now consider all OR ports of
a bridge when looking for it.

Move node_get_*_orport to nodelist.c.

Fix a cut'n'paste error in header of nodelist.h.

Add node_assert_ok().

Add router_get_all_orports(). It's duplicating code from
node_get_all_orports(). Worth fixing at the cost of complicating the
API slightly?

Add some attributes to make a2x quieter

Make asciidoc generation quieter

Make the _sha1.i file generation quieter

Add missing dependency so the html.in files get built

Replace man_MANS with nodist_man1_MANS

(commit message by nickm)

s/AM_CONFIG_HEADER/AC_CONFIG_HEADERS/

Apparently the former will stop working with newer auto*

(commit message by nickm)

Remove #6480 fix from 0.2.2.38 changelog - it was not included

(Forward-port from release-0.2.2 to master)

Fetch IPv6 address from NETINFO "other OR's address" field.

The my_apparent_addr is still unused, apart from now being logged in
the "Got good NETINFO cell" info message.

Merge branch 'bug6638'

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

Merge remote-tracking branch 'public/bug6404' into maint-0.2.3

Merge remote-tracking branch 'linus/bug6621'

Avoid spurious warnings in rend_service_introduce

There was some code in the "err:" block that would always log a
warning, reporting an "unknown error" if we hadn't set err_msg.  But
there were also plenty of "goto err" blocks that did their own
logging, and never set err_msg at all.  Now we should only log when
we have an error message to log.

This fixes bug 6638, from no released Tor version.

Send IPv6 address in NETINFO cells.

Closes #6364.

Make all relays, not only bridges, capable of advertising an IPv6 OR port.

Closes #6362.

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

whitespace fix

When iterating over connections pending DNS, skip marked ones

Failure to do this would lead to double-free cases and similar,
especially when the exit's DNS was broken. See bug 6472 for full
details; this is a fix for 6472.

Anonymous patch from "cypherpunks" on trac.

Merge remote-tracking branch 'public/bug5124'

Merge branch 'bug6468_squashed'

Remove needless flush-on-write code.

Long ago, before we had cell queues, it was necessary to maybe call
connection_handle_write() from connectino_write_to_buf_impl() on OR
connections, so that we wouldn't get into a loop of reading infinite
amounts of data and queueing it all on an outbuf before bothering to
write any data.

If that doesn't sounds like what our code does now, you're right:
right now, we won't stick more than OR_CONN_HIGHWATER bytes of cells
on an outbuf, and we won't suck more than CELL_QUEUE_HIGHWATER_SIZE
cells off any edge connection. So, there's no more call for that
code.

Removing this code will simplify our data flow, and that should be
something we can all get behind.

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

Consider IPv6 OR ports when deciding whether a routerinfo change is cosmetic.

Closes #6423.

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

Merge branch 'bug6379' into maint-0.2.3

Fix more warnings from openbsd_malloc

Apparently, (void)writev is not enough to suppress the "you are
ignoring the return value!" warnings on Linux.  Instead, remove the
whole warning/error logic when compiling openbsd_malloc for Tor: we
can't use it.

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

Merge remote-tracking branch 'public/bug6244_part_c' into maint-0.2.3

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

Merge remote-tracking branch 'public/bug6507' into maint-0.2.3

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

Whitespace and build fixes on 6475 patch

Merge remote-tracking branch 'mikeperry/bug6475' into maint-0.2.3

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

Merge remote-tracking branch 'public/bug6514' into maint-0.2.3

Make authorities not set Running unless all advertised OR ports are reachable.

Resolves #6621.

Address Nick's comments from code review.

Also promote log messages to notice and rate-limit them.

Bug 6475: Explicitly track our path bias state.

This is done to avoid spurious warns. Additional log lines are also
added to try to track down the codepaths where we are somehow overcounting
success counts.

Fix warnings and 64-bit problems in openbsd-malloc code

The warning fixes are:
  - Only define issetugid if it's missing.
  - Explicitly ignore the return value of writev.
  - Explicitly cast the retval of readlink() to int.

The 64-bit problems are related to just storing a size_t in an int. Not cool!  Use a size_t instead.

Fix for bug 6379. Bugfix on 0.2.0.20-rc, which introduced openbsd-malloc.

Fix wildcarded address mappings from the control port

Apparently, we weren't actually detecting wildcardedness when parsing
them: whoops!

bug 6244.  Bugfix on 0.2.3.9-alpha

Raise the part of torrc mapaddress handling that knows wildcards

This patch extracts the inner part of config_register_addressmaps --
the part that knows about detecting wildcard addresses addresses --
and makes it into a new function.  The new function is deliberately
not moved or reindented, so that the diff is smaller.

I need this to fix bug 6244.

Check for stream_id, not conn, on extend cells.

Extend cells aren't allowed to have a stream_id, but we were only
blocking them when they had a stream_id that corresponded to a
connection.  As far as I can tell, this change is harmless: it will
make some kinds of broken clients not work any more, but afaik nobody
actually make a client that was broken in that way.

Found while hunting for other places where we made the same mistake
as in 6271.

Bugfix on d7f50337c14c back from May 2003, which introduced
telescoping circuit construction into 0.0.2pre8.

Update description of what we did to upper limit on md size

Spotted by asn

Fix memory leak in dirvote_create_microdescriptor

Found by George, who gets a cookie.

Clarify the last changes message

oops: changes file for f45cde05f934ee95e0002ff7d72f7006ce156a4c

Remove tor_malloc_roundup().

This function never actually did us any good, and it added a little
complexity.  See the changes file for more info.

forward-port the 0.2.2.38 changelog

Reject attempts to say FooPort and FooPort 0 in the same cfg domain

Fix spaces from last patch

Don't infer we have a FooPort from the presence of a FooPort line

Thanks to the changes we started making with SocksPort and friends
in 0.2.3.3-alpha, any of our code that did "if (options->Sockport)"
became wrong, since "SocksPort 0" would make that test true whereas
using the default SocksPort value would make it false.  (We didn't
actually do "if (options->SockPort)" but we did have tests for
TransPort.  When we moved DirPort, ORPort, and ControlPort over to
the same system in 0.2.3.9-alpha, the problem got worse, since our
code is littered with checks for DirPort and ORPort as booleans.

This code renames the current linelist-based FooPort options to
FooPort_lines, and adds new FooPort_set options which get set at
parse-and-validate time on the or_options_t.  FooPort_set is true
iff we will actually try to open a listener of the given type. (I
renamed the FooPort options rather than leave them alone so that
every previous user of a FooPort would need to get inspected, and so
that any new code that forgetfully uses FooPort will need fail to
compile.)

Fix for bug 6507.

fix up calling of config.status to generate docs

Fix up make distcheck and greatly simplify docs dependencies (although it's still a bit odd)

fix circular dependency for generating code digests

fix TESTS to include full path to src/test/test

fix dependencies for some generated files

remove contrib/Makefile and contrib/suse/Makefile from configure.in

(otherwise would throw an error of "required file X" not found as part
of autogen.sh)

add subdir-objects to AUTOMAKE_OPTIONS, this builds object files in subdirs with non-recursive make

Move to non-recursive make

This gives us a few benefits:
1) make -j clean all
   this will start working, as it should. It currently doesn't.
2) increased parallel build
   recursive make will max out at number of files in a directory,
   non-recursive make doesn't have such a limitation
3) Removal of duplicate information in make files,
   less error prone

I've also slightly updated how we call AM_INIT_AUTOMAKE, as the way
that was used was not only deprecated but will be *removed* in the next
major automake release (1.13).... so probably best that we can continue
to bulid tor without requiring old automake.
(see http://www.gnu.org/software/automake/manual/html_node/Public-Macros.html )

For more reasons  why, see resources such as:
http://miller.emu.id.au/pmiller/books/rmch/

Temporarily make spurious sendmes warn louder at arma's suggestion.

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

Merge remote-tracking branch 'public/bug6252_again' into maint-0.2.3

forward-port the 0.2.3.20-rc changelog

Speak not the name of INT_MIN; it can upset older compilers

And more to the point, some GCCs will warn that you can't say it
before C90.

Bug not in any released version of Tor.

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

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

Conflicts:
src/or/routerlist.c

Try to clarify impact of bug 6537

I don't personally agree that this is likely to be easy to exploit,
and some initial experimention I've done suggests that cache-miss
times are just plain too fast to get useful info out of when they're
mixed up with the rest of Tor's timing noise.  Nevertheless, I'm
leaving Robert's initial changelog entry in the git history so that he
can be the voice of reason if I'm wrong. :)

Mitigate a side-channel leak of which relays Tor chooses for a circuit

Tor's and OpenSSL's current design guarantee that there are other leaks,
but this one is likely to be more easily exploitable, and is easy to fix.

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

Remove bogus comment claiming that an assertion is triggerable by consensus

Fix a bunch of "implicit 64->32" warnings from introduce refactoring

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

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

Clarify security impact of bug 6530

Avoid possible segfault when handling networkstatus vote with bad flavor

Fix for 6530; fix on 0.2.2.6-alpha.

Updated docs for new connections.

Merge remote-tracking branch 'sysrqb/bug6518'

Constify struct sockaddr *sa parameter for check

The values are only being checked, not modified.

Removed redundant check_sockaddr_family_match call

add $(AM_V_GEN) to documentation generation Makefile targets.

This makes the V=1 or V=0 automake silent build options display (or hide)
the full command line used.

  GEN foo.bar

will be seen rather than the full command.

As with all automake silent rules, "make V=1" will output the full command.

Enable the automake silent rules.

$ make V=1 # will temporarily disable them

otherwise you see:

   CC foo.c

rather than the giant long bulid line.

This makes it significantly easier to spot compiler warnings etc.

Additionally, make them conditional, so we won't error on automake <
1.11

(commits squashed by nickm.)

Defensive programming: clear rs_out between iterations.

I can't currently find a bug here, but there are a couple of
near-misses.  Addresses ticket 6514; reported pseudonymously on
IRC.

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

Merge branch 'bug6480_squashed' into maint-0.2.3

Avoid hard (impossible?)-to-trigger double-free in dns_resolve()

Fixes 6480; fix on 0.2.0.1-alpha; based on pseudonymous patch.

Merge branch 'bug6177_squashed'