Test argument concatenation in get_next_token()

First testcase for get_next_token()

Add new source file to test target

Merge branch 'maint-0.3.4'

test: Update approx_time before the test starts

This way we have the same time source when the IP is created and tested later.

Fixes #27810

Signed-off-by: David Goulet <dgoulet@torproject.org>

Merge remote-tracking branch 'public/bug27990'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4'

Fix another tor_assert(0) case

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.2.9' into maint-0.3.3

Fix make check-spaces.

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4'

Adjust tor_assert_unreached()

This version of the macro makes it obvious to the compiler that
"tor_assert_unreached()" will always abort().

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'bug27709_029' into maint-0.2.9

Explain a bit more about branch prediction in the unit-test case

Remove a double-newline and improve a comment

Merge branch 'bug27772_squashed'

Fix various GCC LTO warnings in the unit tests.

In tor_tls_get_my_certs(), set cert ptrs even on failure

Nothing should ever look at them on failure, but in some cases,
the unit tests don't check for failure, and then GCC-LTO freaks out.

Fixes part of 27772.

Initialize some locals in socks5 parsing code.

These confused GCC LTO, which thought they might be used
uninitialized.  I'm pretty sure that as long as 'res' indicates
success, they will always be set to something, but let's unconfuse
the compiler in any case.

Use the correct function signatures in test_relaycell.c

This is now officially an antipattern: please let's never copy a
function declaration in two places again.  That's what headers are
for.

Use assertions so GCC LTO doesn't worry about TLS channel conversion

Part of #27772

Merge remote-tracking branch 'tor-github/pr/402'

report origin of mystery core file

Report what program produced the mysterious core file that we
occasionally see on Travis CI during make distcheck.  Closes ticket
28024.

Merge remote-tracking branch 'tor-github/pr/380'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'maint-0.3.4'

Merge remote-tracking branch 'tor-github/pr/392' into maint-0.2.9

Merge branch 'bug27738-034-merged' into bug27738-master-merged

Merge branch 'bug27738-033' into bug27738-034

Merge remote-tracking branch 'teor/bug27738-032' into bug27738-033

Merge branch 'maint-0.3.4'

Merge branch 'appveyor_pacman_fixes_034' into maint-0.3.4

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.3

Update geoip and geoip6 to the October 9 2018 database.

Add sys/time.h include back to procmon.c

fixup! Changes file for apveyor changes

Changes file for apveyor changes

Appveyor: Don't try to install the mingw openssl package

This package conflicts with a preinstalled mingw openssl.

Fixes bug 27943. I think this was an appveyor change, not a Tor bug.

Install only one of the two mingw architectures in appveyor

Closes ticket 27765

This commit also changes the appveyor file to a unified line ending
style, since the old one was pretty confused.  Use "-b" to see the
real differences.

Merge remote-tracking branch 'tor-github/pr/384'

Merge remote-tracking branch 'tor-github/pr/385'

Merge remote-tracking branch 'public/ticket27856'

Add changes file

Add newline between hs_client_get_random_intro_from_edge() and hs_client_receive_introduce_ack()

Make sure we're removing conn from old_conns *once*

Refrain from using SMARTLIST_DEL_CURRENT in retry_all_listeners

Merge remote-tracking branch 'public/bug27893'

Merge remote-tracking branch 'public/bug27728'

Fix a bug where we would crash on --version.

Bug not in any released Tor.

test-stem would have caught this.

changes file for splitting routerparse.c (#27924)

Remove routerparse include from files that dont use it

Remove versions.h include from routerparse.h

Remove unused headers from routerparse.c

Extract addr-policy parsing code.

Move summarize_protover_flags to versions.c

extract networkstatus parsing to its own file.

Pull detached-signatures code into dirauth.

Split microdescriptor parser into its own file.

Remove addr_policy_assert_ok() as unused

Remove dump_distinct_digest_count()

It was disabled-by-default for ages, and it no longer compiles. I
think it's safe to call it obsolete.

Remove router_get_dir_hash as unused.

Move v2 hs parsing into feature/rend

Split the authority-cert and signature/hash code from routerparse

Extract the version-managing code from routerparse.c

Leave the versions.h include in routerparse.h for now; I'll remove
it later.

Extract logic for dumping unparseable junk from routerparse.c

Move routerparse and parsecommon to their own module.

Detect an unlikely integer overflow.

Merge branch 'split_stats'

Merge branch 'ticket_27861'

Prevent duplicate initialization of NSS DH module

Allowing this didn't do any actual harm, since there aren't any
shared structures or leakable objects here.  Still, it's bad style
and might cause trouble in the future.

Closes ticket 27856.

Always initialize the periodic events list.

Various places in our code try to activate these events or check
their status, so we should make sure they're initialized as early as
possible.  Fixes bug 27861; bugfix on 0.3.5.1-alpha.

Fix a memory leak in --dump-config

When freeing a configuration object from confparse.c in
dump_config(), we need to call the appropriate higher-level free
function (like or_options_free()) and not just config_free().

This only happens with options (since they're the one where
options_validate allocates extra stuff) and only when running
--dump-config with something other than minimal (since
OPTIONS_DUMP_MINIMAL doesn't hit this code).

Fixes bug 27893; bugfix on 0.3.2.1-alpha.

Mark a variable static.

geoip: fix windows compilation

Changes file for splitting stats

Extract the non-stats part of geoip into a new src/lib/geoip.

Remove excess dependencies from geoip.c

Split geoip from geoip-related stats.

This commit just moves the code to two separate files. The geoip
code still has a few needless dependencies on core/* and features/*.

Move the n_v3_ns_requests field out of geoip_country_t

This is preparation for splitting geoip.c into stats and non-stats
portions.

Move the predicted ports code out of rephist.c

It differs from the rest of the rephist code in that it's actually
necessary for Tor to operate, so it should probably go somewhere
else.  I'm not sure where yet, so I'll leave it in the same
directory, but give it its own file.

Merge branch 'router_split'

Merge remote-tracking branch 'dgoulet/bug27550_035_01'

Changes file for splitting router.c

Remove extra includes from router.c

Revise things that had included router.h before

Make them only include the headers that they needed, and sort their
headers while we're at it.

Move the "is the network disabled?" functions out of router.c

Since this is completely core functionality, I'm putting it in
core/mainloop, even though it depends on feature/hibernate. We'll
have to sort that out in the future.

Move routerinfo_t functions out of router.c

(It turns out that some of the functions in router.h didn't even
exist any more, so I just got to delete their declarations completely.)

Extract nickname-checking functions from router.c

Move the various _describe() functions out of router.c

Note that I haven't separated the headers yet (there's still an

Extract all the "am I a server" functions from router.c