Fix: check r < 0 before checking errno

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Fix possible infinite loop on pipe/sock_drain()

If the returned value of read/recv is 0 (meaning EOF), we'll end up in an
infinite loop (active wait) until something is written on the pipe which is
not really what we want here especially because those functions are called
from the main thread.

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Fix: test -ENOENT after config_parse_unix_port()

Check for -ENOENT instead of ENOENT after the HS port is parsed.

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Merge remote-tracking branch 'public/prop227_v2'

Conflicts:
src/test/test_dir.c

Correctly handle OutboundBindAddress again.

ca5ba2956bcd4b5ee1e526ccf5914f52fe6e6d51 broke this; bug not in any
released Tor.

Also fix a typo.

Fixes 14541 and 14527.  Reported by qbi.

Merge branch 'bug14451_026_v1'

changes file for 14451

Use HT_BUCKET_NUM_ in more places

(patch from sysrqb)

Merge remote-tracking branch 'public/ticket11737'

Document unix: addresses

Restrict unix: addresses to control and socks for now

Remove SocksSocket; it's now spelled differently thanks to 14451

Also, revise bug12585 changes file to mention new syntax

When there are no package lines, make consensus/packages say "".

Also, give a better error message when there is no consensus.

Correctly reject packages lines with empty entries

Define 'digesttype' correctly

Make hidden service use the config unix prefix

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Support unix: prefix in port configuration

It's now possible to use SocksPort or any other kind of port that can use a
Unix socket like so:

  SocksPort unix:/foo/bar/unix.sock

Fixes #14451

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Merge branch 'bug11485_026_v2_squashed'

fixup! Refactor the use of ifdef HAVE_SYS_UN_H

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Changes file for ticket #11485

Actually make connections to HSes on AF_UNIX sockets

Support connection_exit_connect() to AF_UNIX sockets

Handle config options for AF_UNIX hidden services rendservice.c

changes file for 8405

Bug #8405: Report SOCKS username/password in CIRC status events

Introduces two new circuit status name-value parameters: SOCKS_USERNAME
and SOCKS_PASSWORD. Values are enclosing in quotes and unusual characters
are escaped.

Example:

    650 CIRC 5 EXTENDED [...] SOCKS_USERNAME="my_username" SOCKS_PASSWORD="my_password"

Add GETINFO consensus/{valid-{after,until},fresh-until}

more typo fixes from mcs and gk

Tweak tor-resolve docs and logs

Resolves 14325

Bump a client authorization message from debug to info.

A user wants this for 14015, and it seems fairly reasonable.

New GETINFO consensus/packages to expose package information from consensus

Fixes on prop227 comments, based on comments by mcs on #10395

Merge branch 'doc13702_squashed'

Adding section on OpenBSD to doc/TUNING

Groundwork for AF_UNIX hidden services in rendservice.c

Bail early in cpuworker_onion_handshake_replyfn if the circuit is marked

changes file for bug14350

configure.ac: fix disabling systemd notification support

If --disable-systemd is given, $enable_systemd is set to "no", not "false".
As a result, if libsystemd is found, we still turn on systemd support even
if we explicitly disable it with --disable-system.

Try to fix some more memory leaks in the unit tests

Fix some memory leaks in new address tests

whitespace fixes

Split the slow unit tests into their own binary

This can run in parallel with the faster ones and the other tests.

Merge branch 'if_addr_refactoring_squashed'

Conflicts:
src/test/include.am
src/test/test.c

Refactor code that looks up addresses from interfaces

Now the code has separate implementation and examination functions,
uses smartlists sanely, and has relatively decent test coverage.

changes file for 9969

Tweak the 9969 fix a little

If we have busy nodes and excluded nodes, then don't retry with the
excluded ones enabled.  Instead, wait for the busy ones to be nonbusy.

Merge remote-tracking branch 'public/ticket9969'

Conflicts:
src/or/directory.c
src/or/routerlist.c
src/or/routerlist.h
src/test/include.am
src/test/test.c

fix warnings from lintChanges

Merge remote-tracking branch 'public/ticket13762'

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

Merge remote-tracking branch 'origin/maint-0.2.4' into maint-0.2.5

Merge remote-tracking branch 'karsten/geoip6-jan2015' into maint-0.2.4

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

fix some warnings in compat_threads.c

Move a redundant _GNU_SOURCE to where it is not redundant

Update geoip6 to the January 7 2015 database.

Update geoip to the January 7 2015 database.

use the correct free fn. spotted by dgoulet

Merge branch 'better_workqueue_v3_squashed'

Fix: remove whitespace and update a comment in cpuworker.c

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Fix: change copyright year in workqueue and thread tests

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Support monotonic time for pthread_cond_timedwait

This is to avoid that the pthread_cond_timedwait() is not affected by time
adjustment which could make the waiting period very long or very short which
is not what we want in any cases.

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Merge branch 'bug9819'

fix a comment

Fix up some workqueue/threading issues spotted by dgoulet.

handle EINTR in compat_*threads.c

Merge branch 'bug14084'

Expand manpage for HiddenServiceAllowUnknownPorts based on suggestions from qwerty1 and dgoulet

Merge remote-tracking branch 'dgoulet/bug14224_025_v1'

Merge branch 'ticket14254_squashed'

Fix SocksSocket 0.  That was easy!

Some days I just can't C.

Make check-spaces happier.

Remove a now-needless testing workaround.

Replace a 4 with a 6; fix a bug that nobody noticed :/

Fixes 14280 bugfix on 1053af0b9c4127873034a935ce33 in 0.2.4.7-alpha.

Merge branch 'bug7555_v2_squashed'

Conflicts:
src/or/connection_edge.c

Add a bunch of new comments to explain connection_ap_rewrite{,_and_attach}

Also, do a little light refactoring to move some variable declarations
around and make a few things const

Also fix an obnoxious bug on checking for the DONE stream end reason.
It's not a flag; it's a possible value or a variable that needs to be
masked.

Fix: close intro circuit if no more intro points are usable

Once a NACK is received on the intro circuit, tor tries an other usable one
by extending the current circuit to it. If no more intro points are usable,
now close the circuit. Also, it's reason is changed before closing it so we
don't report again an intro point failure and trigger an extra HS fetch.

Fixes #14224

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Fix: close intro circuit if no more intro points are usable

Once a NACK is received on the intro circuit, tor tries an other usable one
by extending the current circuit to it. If no more intro points are usable,
now close the circuit.

Fixes #14224

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Merge remote-tracking branch 'public/ticket13037'

Conflicts:
src/or/config.c

Remove the unused rend_cache_entry_t->received field.

(Patch from arma, commit message from nick.)

Closes #14222.

Merge remote-tracking branch 'public/bug14219_025'

Do not replace a HS descriptor with a different replica of itself

This fixes a bug where we'd fetch different replicas of the same
descriptor for a down hidden service over and over, until we got lucky
and fetched the same replica twice in a row.

Fixes bug 14219; bugfix on 0.2.0.10-alpha.

(Patch from Roger; commit message and changes file by Nick.)

Merge remote-tracking branch 'public/bug14261_025'

Conflicts:
src/or/directory.c

Increase limit for status vote download size by a factor of 5.

We've started to hit the limit here.  We introduced the limit in
0.1.2.5-alpha.  This fixes bug 14261, but we should have a smarter way
to not actually do the behavior this permits.  See #14267 for a ticket
about fixing that.

Fix memory leak in connection_ap_handshake_rewrite_and_attach()

Spotted by asn.  #14259.  Bugfix on 368eb6a97 in 0.2.0.1-alpha.

Merge remote-tracking branch 'public/bug12485'

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

Conflicts:
src/test/test_relaycell.c
src/test/test_util.c

fix another mingw64 unit test warnin

changes file for 9396005428c40cf7ef9f1e6ae1d725f3155391f5

changes suggested by weasel

Unify parse_unix_socket_config and parse_port_config

This incidentally makes unix SocksSocket support all the same options
as SocksPort.

This patch breaks 'SocksSocket 0'; next will restore it.

Resolves 14254.

No, client-side DNS cacheing should not be on by default.

Merge branch 'bug8546_squashed'

Conflicts:
src/or/connection.c
src/or/or.h
src/or/relay.c

changes file for bug 8546

Replace field-by-field copy with memcpy for entry_port_cfg

Move entry_port_cfg_t fields in entry_connection_t

Also rename some options for uniformity, and apply this script:

@@
entry_connection_t *conn;
@@
 conn->
+entry_cfg.
\(
 isolation_flags
\|
 session_group
\|
 socks_prefer_no_auth
\|
 ipv4_traffic
\|
 ipv6_traffic
\|
 prefer_ipv6
\|
 cache_ipv4_answers
\|
 cache_ipv6_answers
\|
 use_cached_ipv4_answers
\|
 use_cached_ipv6_answers
\|
 prefer_ipv6_virtaddr
\)

Combine entry_port_cfg_t fields in listener_connection_t

Also, revise the code using these options with this cocci script:

@@
listener_connection_t *conn;
@@
 conn->
+entry_cfg.
\(
 isolation_flags
\|
 session_group
\|
 socks_prefer_no_auth
\|
 ipv4_traffic
\|
 ipv6_traffic
\|
 prefer_ipv6
\|
 cache_ipv4_answers
\|
 cache_ipv6_answers
\|
 use_cached_ipv4_answers
\|
 use_cached_ipv6_answers
\|
 prefer_ipv6_virtaddr
\)

Split client-specific and server-specific parts of port_cfg_t

Also, apply this cocci script to transform accesses. (Plus manual
migration for accesses inside smartlist_foreach loops.)

@@

port_cfg_t *cfgx;
@@
 cfgx->
+server_cfg.
\(
 no_advertise
\|
 no_listen
\|
 all_addrs
\|
 bind_ipv4_only
\|
 bind_ipv6_only
\)

@@

port_cfg_t *cfgx;
@@
 cfgx->
+entry_cfg.
\(
 isolation_flags
\|
 session_group
\|
 socks_prefer_no_auth
\|
 ipv4_traffic
\|
 ipv6_traffic
\|
 prefer_ipv6
\|
 cache_ipv4_answers
\|
 cache_ipv6_answers
\|
 use_cached_ipv4_answers
\|
 use_cached_ipv6_answers
\|
 prefer_ipv6_virtaddr
\)

Fix a compilation warning in the unit tests

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