Remove a 9-function strongly connected component of tor's callgraph.

microdesc_free_() called get_microdesc_cache(), which had the fun
side-effect of potentially reloading the whole cache from disk.
Replace it with a variant that doesn't.

Update callgraph code to find and output strongly connected components

Merge remote-tracking branch 'dgoulet/bug16274_027_02'

Merge remote-tracking branch 'public/decouple-write'

Merge remote-tracking branch 'public/bug16286'

add dSYM files to gitignore (they are an osx thing)

Update tor-fw-helper URL

Set the open file limit to the current value before changing it

If setrlimit() failed, max_out wasn't set in set_max_file_descriptors()
ending in a state where we don't use ULIMIT_BUFFER for things like tor
private key files.

Also fix the set_max_file_descriptors() documentation.

Fixes #16274

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

Write a bunch more test for doc/WritingTests

Allow cov-display to take directories as arguments

Let's try to get test_workqueue working on windows

Merge remote-tracking branch 'public/bug13338'

Switch order of unblocking threads and releasing the mutex.

According to POSIX, the mutex must be locked by the thread calling the signal
functions to ensure predictable scheduling behavior.

Found the issue using Helgrind which gave the warning `dubious: associated lock
is not held by any thread`.

Update doc/HACKING with more coverage instructions

When building with coverage, run chutney with coverage

Previously, this required me to do stuff like
  'cp src/or/tor-cov src/or/tor' ,
which is pretty embarrassing.

Improved targets for "run all the tests, no, all of them."

Move formatting functions around.

The base64 and base32 functions used to be in crypto.c;
crypto_format.h had no header; some general-purpose functions were in
crypto_curve25519.c.

This patch makes a {crypto,util}_format.[ch], and puts more functions
there.  Small modules are beautiful!

Improve log messages for problems about ed25519 keypinning

Fixes 16286; bugfix on 0.2.7.2-alpha.

Add get_max_sockets() and remove dead code

The control port was using set_max_file_descriptors() with a limit set to 0
to query the number of maximum socket Tor can use. With the recent changes
to that function, a check was introduced to make sure a user can not set a
value below the amount we reserved for non socket.

This commit adds get_max_sockets() that returns the value of max_sockets so
we can stop using that "setter" function to get the current value.

Finally, the dead code is removed that is the code that checked for limit
equal to 0. From now on, set_max_file_descriptors() should never be used
with a limit set to 0 for a valid use case.

Fixes #16697

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

Do not autoflush control connections as their outbufs get big

Doing this is no longer necessary, and it leads to weird recursions in
our call graph.  Closes ticket 16480.

whitespace fix

changes file for callgraph scripts

Check in some scripts I use for callgraph analysis.

Wrap windows-only C files in #ifdef _WIN32

This should make some scripts and IDEs happier.

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

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

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

Merge remote-tracking branch 'teor/feature14175-chutney-performance-v2'

Update geoip and geoip6 to the July 8 2015 database.

Merge remote-tracking branch 'linus/bug_11061'

Bump version to 0.2.7.2-alpha-dev

Fold final entries into changelog

tighten and tidy the changelog; pick a date

Merge remote-tracking branch 'yawning/bug16674'

Allow a single trailing `.` when validating FQDNs from SOCKS.

URI syntax (and DNS syntax) allows for a single trailing `.` to
explicitly distinguish between a relative and absolute
(fully-qualified) domain name. While this is redundant in that RFC 1928
DOMAINNAME addresses are *always* fully-qualified, certain clients
blindly pass the trailing `.` along in the request.

Fixes bug 16674; bugfix on 0.2.6.2-alpha.

Fix ed25519-donna with SSP on non-x86.

The only reason 16 byte alignment is required is for SSE2 load and
store operations, so only align datastructures to 16 byte boundaries
when building with SSE2 support.

This fixes builds with GCC SSP on platforms that don't have special
case code to do dynamic stack re-alignment (everything not x86/x86_64).

Fixes bug #16666.

Move the note about non-localhost SOCKSPort usage up to where it belongs.

I think this section slipped downwards when flags where added.

Bump version (and explain how)

Re-run trunnel to capture change for msvc.

Fix a couple more msvc issues, reported by gisle vanem

A nit found by qwerty1

Changelog tweaks from harmony

Minor changelog tweaks

Edit changelog entries

Move around and reformat items in 0.2.7.2-alpha changelog

Merge branch 'ticket2325_squashed'

Actually, write the torrc format in ABNF

This should make it more clear what I meant, if you know how to read ABNF.

(Thanks to rl1987 for correcting numerous issues here)

Remove casting of void pointers when handling signals.

Improve the workqueue test help message.

The workqueue test help message has two issues. First, the message uses 4 space
indentation when 2 space indentation seems more common. Second, the help
message misses some options.

This commit fixes both issues.

Limit the number of inflight items to the number of items.

changes file for bug16631

Fix some potential memory leaks in the thread pool code.

typo fix; spotted by arma

Document the torrc format as thoroughly as possible

Closes ticket 2325

Merge remote-tracking branch 'public/bug16162_026'

Use a more recommended syntax for the systemd unit file

closes 16162.

Fix testcases where scrypt APIs are used wrong.

Add the remaining changes items to ChangeLog for 0.2.7.2-alpha

Make all changes files work with lintChanges

Start changelog sections that are straight copies from 0.2.6

Merge remote-tracking branch 'public/feature_16580'

Merge remote-tracking branch 'public/ticket16543'

Merge remote-tracking branch 'sysrqb/bug15220_026_sysrqb'

Add my script to dump per-file or per-function coverage

Break some wide lines

Fix most check-spaces issues

Use C99 variadic macros when not on GCC.

1) We already require C99.

2) This allows us to support MSVC again (thanks to Gisle Vanem for
   this part)

3) This change allows us to dump some rotten old compatibility code
   from log.c

Merge branches 'feature_16582' and 'feature_16581'

Don't allow INIT_ED_KEY_{NO_REPAIR,NEEDCERT} to be used together.

We haven't implemented NO_REPAIR for NEEDCERT, and we don't need it:
but it's safest to stop any attempt to use it that way.

Add more EINVAL errno setting on key read failures

Teor found these.  This is for part of #16582.

Remove tor-fw-helper code

It did a good idea, but the code-quality of libupnpc and libnatpnp
is so dodgy that I'm not really comfortable including them alongside
Tor proper.  Instead, we'll recommend that people do the pure-go
reimplementation instead.  Closes ticket 13338.

Add changes file for 8712; apply it to HSDir flag as well.

Vote for relays to be fast and stable even when they aren't currently active.

Change the name for the keypinning file; delete the old one if found

This is a brute-force fix for #16580, wherein #16530 caused some
routers to do bad things with the old keypinning journal.

Add more consistency checks in load_ed_keys

Make sure that signing certs are signed by the right identity key,
to prevent a recurrence of #16530.  Also make sure that the master
identity key we find on disk matches the one we have in RAM, if we
have one.

This is for #16581.

Do more consistency checks in ed_key_init_from_file()

When there is a signing key and the certificate lists a key, make
sure that the certificate lists the same signing key.

When there are public key and secret key stored in separate files,
make sure they match.

Use the right file name when we load an encrypted secret key and
then find a problem with it.

This is part of 16581.

If loading an ed25519 master key fails with errno != ENOENT, give up.

This implements feature 16582: if we get EMFILE or something when
loading our master key, we should not at that point attempt to
overwrite it.

Preserve errno when loading encrypted ed25519 keys.

Make file-reading and key-reading preserve errno

This is an important part of #16582.

Remove empty rend_service_validate_intro_early function

Rename ambiguous introduction point functions

Task #15824

Try to convince coverity not to worry about this loop either

Try to convince coverity not to worryh about this loop

Tell coverity to tolerate ed25519_verify

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

This is an 'ours' merge, to avoid taking the bump to 0.2.6.10-dev

Bump maint-0.2.6 to 0.2.6.10-dev

Forward-port the changelog and releasenotes

Okay, this time it was my fault

Fine, mingw! fine! are you happy now??

This should be the last SecureZeroMemory fix

Dammit, autoconf!

Disable -Wstack-protector diagnostics in curve25519-donna-helpers.h

This was causing a false positive on our arm builders.

Nth time is maybe the charm for fixing windows readpassword build errors

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

This is an "ours" merge to avoid bumping the version number in master.

Bump maint-0.2.6 to 0.2.6.10

note some dead code in set_max_file_descriptors

Fix a leak in test_microdesc (CID 1311631)

Try one more one more time to get tor-ci-windows working

Apparently its mingw headers are missing some stuff.

Attempt yet again to make the tor-ci-windows builder happy

Remove the HidServDirV2 and VoteOnHidServDirectoriesV2 options

(Mark them as obsolete)

Closes 16543.