GitLab

Alexander Færøy authored Mar 08, 2017 and Alexander Færøy committed Mar 08, 2017

This patch adds the `int *eof` parameter to tor_read_all_handle() on
Windows to unify the API between Unix and Windows.

Additionally this patch cleans up locations in the utility module as
well as the tests where we have different code paths that depends upon
whether we are running on Windows or not.

See: https://bugs.torproject.org/21678

Alexander Færøy authored Mar 08, 2017 and Alexander Færøy committed Mar 08, 2017

This patch adds a test case for the get_string_from_pipe() function
found in the utility module.

See: See: https://bugs.torproject.org/21654

Alexander Færøy authored Mar 08, 2017 and Alexander Færøy committed Mar 08, 2017

This patch removes the buffered I/O stream usage in process_handle_t and
its related utility functions. This simplifies the code and avoids racy
code where we used buffered I/O on non-blocking file descriptors.

See: https://bugs.torproject.org/21654

Alexander Færøy authored Mar 08, 2017 and Alexander Færøy committed Mar 08, 2017

This patch modifies `tor_read_all_handle()` to use read(2) instead of
fgets(3) when reading the stdout from the child process. This should
eliminate the race condition that can be triggered in the 'slow/util/*'
tests on slower machines running OpenBSD, FreeBSD and HardenedBSD.

See: https://bugs.torproject.org/21654

Make hidden services with 8 to 10 introduction points check for failed
circuits immediately after startup. Previously, they would wait for 5
minutes before performing their first checks.

Fixes bug 21594; bugfix on commit 190aac0e in Tor 0.2.3.9-alpha.
Reported by alecmuffett.

Closes ticket 21598.

Use an existing flag to check if an introduction point is established.

Cleanup after 21596.

Fixes bug 21599; bugfix on 0.2.7.2-alpha.

These are from today's stable releases in 0.2.4 through 0.2.8

Previously, they would stop checking when they exceeded their intro point
creation limit.

Fixes bug 21596; bugfix on commit d67bf8b2 in Tor 0.2.7.2-alpha.
Reported by alecmuffett.

Previously, they would stop checking when they exceeded their intro point
creation limit.

Fixes bug 21596; bugfix on commit d67bf8b2 in Tor 0.2.7.2-alpha.
Reported by alecmuffett.

Fixes bug 21576; bugfix on Tor 0.2.9.3-alpha.
Reported by alecmuffett.

"ours" to avoid version bump

This patch resets `buf` in test_util_fgets_eagain() after each succesful
ivocation to avoid stray artifacts left in the buffer by erroneous
tor_fgets() calls.

This patch changes our use of fgets() to tor_fgets() for more consistent
error handling across different versions of the C library.

This patch adds the `tor_fgets()` function to our compatibility layer.

`tor_fgets()` adds an additional check for whether the error-bit have
been enabled for the given file stream, if that is the case and `errno`
is set to `EAGAIN` we make sure that we always return NULL.

Unfortunately `fgets(3)` behaves differently on different versions of
the C library.

See: https://bugs.torproject.org/21416
See: https://bugs.torproject.org/20988

In that chutney test, the bridge client is configured to connect to
the same bridge at 127.0.0.1:5003 _and_ at [::1]:5003, with no
change in transports.

That meant, I think, that the descriptor is only assigned to the
first bridge when it arrives, and never the second.

- Make sure we check at least two guards for descriptor before making
  circuits. We typically use the first primary guard for circuits, but
  it can also happen that we use the second primary guard (e.g. if we
  pick our first primary guard as an exit), so we should make sure we
  have descriptors for both of them.

- Remove BUG() from the guard_has_descriptor() check since we now know
  that this can happen in rare but legitimate situations as well, and we
  should just move to the next guard in that case.

"Ours" merge to avoid version bump

Previously, this only worked on systems which had bash installed, due to
some bash-specific code in the script.

Fixes bug 19699; follow-up to 21581.