Document limitation of log_from_handle with partial reads

Factor out and re-write code for splitting lines from a handle

Now handles non-printable characters and will not output a spurious
new-line if given a partial line.

Refactor out command line formatting

Now correctly handles whitespace, quotes and backslashes. Passes all unit tests.

Add a sanity check

Replace two magic tristates with #define'd names

- process_handle_t.status
- return value of tor_get_exit_code()

Make signature of tor_spawn_background more conventional

Conventionally in Tor, structs are returned as pointers, so change
tor_spawn_background() to return the process handle in a pointer rather
than as return value.

Appease "make check-spaces"

Improve comments and fix one bug

We don't need to find our own path, just tell Windows to search

Find test-child.exe by looking in same directory as test.exe

Fix compilation errors under *nix

Fix test cases to handle MSYS style paths (/c/foo rather than c:/foo)

Also fix test case to expect 1 on successfully spawning a subprocess

Tidy up subprocess code

- Better error handling
- Write description of functions
- Don't assume non-negative process return values

Fix some compiler warnings

Revert change to libminiupnpc search path

This was supposed to fix an oddity with library searching in autoconf
but it didn't actually fix anything.

Merge branch 'bug1983-port-tor-fw-helper-to-windows' into bug2046

Conflicts:
configure.in
src/tools/tor-fw-helper/Makefile.am
src/tools/tor-fw-helper/tor-fw-helper-upnp.c
src/tools/tor-fw-helper/tor-fw-helper.c

Correct reference to libiphlpapi from libiphlapi

Appease "make check spaces"

Only link ws2_32 and iphlapi on windows.

This is a tweak for the tor-fw-helper port to windows.

Link and build tor-fw-helper on Windows

- Update configure script to test for libminiupnpc along with the
  libws2_32 and libiphlpapi libraries required by libminiupnpc
- When building tor-fw-helper, link in libiphlpapi
- Link in libminiupnpc statically becasue I could not get the DLL
  to link properly
- Call WSAStartup before doing network operations
- Fix up a compiler warning about uninitialized backend_state

N.B. The changes to configure.in and Makefile.am will break on non-
Windows platforms.

Remove warning about a loop parsing evbuffer socks

This behavior is normal when we want more data than the evbuffer
actually has for us.  We'll ask for (say) 7 bytes, get only 5
(because that's all there is), try to parse the 5 bytes, and get
told "no, I want 7".  One option would be to bail out early whenever
want_length is > buflen, but sometimes we use an over-large
want_length.  So instead, let's just remove the warning here: it's
not a bug after all.

Appease "make check-spaces"

Automatically use filtering bufferevents with IOCP.

Merge remote branch 'origin/master' into bug2046

Complete logging of output from port forwarding helper

Call evthread_use_windows_threads when running with IOCP on windows

Use evbuffer_copyout() in inspect_evbuffer().

Bufferevents now requires Libevent 2.0.13-stable.

(Earlier Libevent versions have bufferevent bugs that affect us, and
are missing some APIs that it would be handy to use.)

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

Conflicts:
src/or/config.c

Don't write ControlPorts to disk till after we setuid and make datadir

Fix for bug 3747; bugfix on 0.2.2.26-beta.

Merge branch 'bug3743'

Remove a needless config_find_option

Actually pick a random port when "auto" is specified

ddc65e2b3303559ab7b842a176ee6c2eda9e4027 had broken this

Don't compare LINELIST_S and OBSOLETE config options.

Merge remote-tracking branch 'asn/bug3728'

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

Remove extra quotation marks around the result of esc_for_log

Oops; _actually_ enable microdesc-by-default for clients

It turns out that it wasn't enough to set the configuration to
"auto", since the correct behavior for "auto" had been disabled in
microdesc.c.   :p

(Hasn't been in a release yet, so doesn't need a changes entry.)

Free global_rate_limit in connection_free_all().

Move connection_free_all() to the bottom of connection.c to enjoy maximum visibility.

Merge remote-tracking branch 'sebastian/bug3700'

Don't warn on http connection to my orport

Also remove a few other related warnings that could occur during the ssl
handshake. We do this because the relay operator can't do anything about
them, and they aren't their fault.

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

Merge branch 'bug3643'

Ignore deprecation warnings on OS X

Starting with Lion, Apple decided to deprecate the system openssl. We
can start requiring users to install their own openssl once OS X doesn't
ship with it anymore.

Whitespace tweaks

Fix a const warning in bug1692 code

Merge branch 'bug1692-squashed'

Fix condition reported by nickm.

Return if CONF_CHANGED isn't interesting.

Escape configuration values before sending them via CONF_CHANGED.

Use smartlist_asprintf_add() to improve readability.

Add smartlist_[v]asprintf_add

I should have added this before implementing #2411.

Make FooPort 0 correctly disable the FooPort.

Since the prop171 stuff, it had instead made Tor bind port 0, and
re-bind it differently all the time.

Resolves bug3704; not in any released version.

Sticking a size_t into long generates a warn on win

Get rid of an unused parameter warning on win

Sockets are unsigned on windows

this gets rid of a warning about signed/unsigned comparison

Minor code readability fix.

Add changes file.

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

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

Minor comment fix.

Update to the August 2011 GeoIP database.

Refactor to do CONF_CHANGED event formatting inside control.c

Merge remote-tracking branch 'public/bug3550'

Remove connection_edge_streams_are_compatible

It's dead code (not used anywhere by the current proposal 171 algorithm).

Fix handling of ISO_STREAM

Now we track *which* stream with ISO_STREAM set is associated to a
particular circuit, so that we won't think that stream is incompatible
with its circuit and launch another one a second later, and we use that
same field to mark circuits which have had an ISO_STREAM stream attached
to them, so that we won't ever put a second stream on that circuit.

Fixes bug 3695.

Emits CONF_CHANGED events whenever Tor's configuration values change.

Fix log message typo

Merge remote-tracking branch 'public/bug3683'

Clear socks auth fields before free

Handle storing much longer socks4 authentication data.

Clean up bridge-stats code.

Only write a bridge-stats string if bridge stats have been
initialized.  This behavior is similar to dirreq-stats, entry-stats,
etc.

Also add a few unit tests for the bridge-stats code.

Separate generation of an entry-stats string from writing it to disk.

This commit is similar to the previous two commits for dirreq-stats,
but for entry-stats.

Separate generation of a dirreq-stats string from writing it to disk.

This patch separates the generation of a dirreq-stats string from
actually writing it to disk.  The new geoip_format_dirreq_stats()
generates a dirreq-stats string that geoip_dirreq_stats_write() writes
to disk.  All the state changing (e.g., resetting the dirreq-stats
history and initializing the next measurement interval) takes place in
geoip_dirreq_stats_write().  That allows us to finally test the
dirreq-stats code better.

Clean up geoip_dirreq_stats_write().

We later want to split this function into one function that generates
the dirreq-stats string and one that writes it to disk.

Add unit tests for buffer-stats.

Now that formatting the buffer-stats string is separate from writing
it to disk, we can also decouple the logic to extract stats from
circuits and finally write some unit tests for the history code.

Separate generation of a buffer-stats string from writing it to disk.

The new rep_hist_format_buffer_stats() generates a buffer-stats string
that rep_hist_buffer_stats_write() writes to disk.  All the state
changing (e.g., resetting the buffer-stats history and initializing
the next measurement interval) takes place in
rep_hist_buffer_stats_write().  That allows us to finally test the
buffer-stats code better.

Clean up rep_hist_buffer_stats_write().

We later want to split this function into one function that generates
the buffer-stats string and one that writes it to disk.

Create cell-stats history even if we didn't see a single circuit.

So far, if we didn't see a single circuit, we refrained from
generating a cell-stats string and logged a warning.  Nobody will
notice the warning, and people will wonder why there's no cell-stats
string in the extra-info descriptor.  The better behavior is to
generate a cell-stats string with all zeros.

Replace files in stats/ rather than appending to them.

Right now, we append statistics to files in the stats/ directory for
half of the statistics, whereas we overwrite these files for the other
half. In particular, we append buffer, dirreq, and entry stats and
overwrite exit, connection, and bridge stats.

Appending to files was useful when we didn't include stats in extra-info
descriptors, because otherwise we'd have to copy them away to prevent
Tor from overwriting them.

But now that we include statistics in extra-info descriptors, it makes
no sense to keep the old statistics forever. We should change the
behavior to overwriting instead of appending for all statistics.

Implements #2930.

Make --quiet and --hush apply to default logs, not only initial logs

Fixes bug 3550; bugfix on 0.2.0.10-alpha (where --quiet was introduced).

Treat socks_request->{username,password} as non-NUL-terminated

They *are* non-NUL-terminated, after all (and they have to be, since
the SOCKS5 spec allows them to contain embedded NULs.  But the code
to implement proposal 171 was copying them with tor_strdup and
comparing them with strcmp_opt.

Fix for bug on 3683; bug not present in any yet-released version.

Implement protocol-type isolation correctly.

Previously we'd just looked at the connection type, but that's
always CONN_TYPE_AP.  Instead, we should be looking at the type of
the listener that created the connection.

Spotted by rransom; fixes bug 3636.

put the 0.2.1.30 stanza in the master changelog too

reach back in time and correct two changelog entries

Initial patch to build Tor with msvc and nmake

We'll still need to tweak it so that it looks for includes and
libraries somewhere more sensible than "where we happened to find
them on Erinn's system"; so that tests and tools get built too;
so that it's a bit documented; and so that we actually try running
the output.

Work done with Erinn Clark.

Re-order tor_fw_helper_LDADD flags so that they work

Patch from "blueness".

Test case for reading the partial output of a background process

Fix compilation on non-Windows platforms

If hProcess is NULL, read_all_handle returns if it would block

Use PeekNamedPipe to avoid blocking ReadFile when there is nothing to read

Add code to read all from a handle, but this block forever

See http://stackoverflow.com/questions/3722409/windows-child-process-with-redirected-input-and-output
for a potential solution

Add Windows version of tor_spawn_background and ancillary functions

Generalize process spawning so its test compiles (but fails) in Windows

- pid, stdout/stderr_pipe now encapsulated in process_handle
- read_all replaced by tor_read_all_from_process_stdin/stderr
- waitpid replaced by tor_get_exit_code

Untested on *nix

Improve log messages for optimistic data retry

Under Windows, call WSAStartup before using the network

XXX Link in libiphlpapi (should be disabled on non Win32 platforms)

XXX Use static linking on Windows because miniupnpc doesn't work as DLL

Add src/test/test{-child}.exe to gitignore. bug3626