server/admin.c: Remove spurious `ping' in usage message.

Oops.

server/tests.at: Quote messages being echoed.

For consistency, and to make the literals stand out in my editor.

server/tripe.8.in: Fix the reference for trace options.

server/tripe-admin.5.in: Fix out-of-date information about `kill'.

It does sometimes send a `bye' message now.

svc/connect.in: Make `failures' info value be a string.

Otherwise we choke when presenting it for the `info' command.

peerdb/tripe-newpeers.in: Reflow an overlong comment.

common/defs.man: Set the font for the running headers as well as the body.

This involves switching environments.

peerdb/peers.in.5.in: Fix `.' which should have been `,' between list items.

keys/tripe-keys.conf.5.in: Delete entirely spurious `rijndael-cbc'.

No idea where that came from.  Looks like copy-and-paste debris from
when that table was originally added.

keys/tripe-keys.conf.5.in: Use `\-' (minus) for options, not `-' (hyphen).

keys/tripe-keys.conf.5.in: Generally enclose literal strings in quotes.

keys/tripe-keys.conf.5.in: Typeset parameter names in bold.

Not sure why I decided to set these in italics when I started.  It
doesn't make a great deal of sense.

Merge branch '1.0.0pre19.x'

* 1.0.0pre19.x:
  svc/connect.in: Squash newlines to spaces in `info' output.
  server/admin.c: Fix `=' vs `==' error in assertion.
  svc/tripe-ifup.in: Don't set remote IPv6 address until interface is up.

svc/connect.in: Squash newlines to spaces in `info' output.

Otherwise all sorts of things go horribly wrong.

server/admin.c: Fix `=' vs `==' error in assertion.

Oops.  Well, at least this /shouldn't/ have done anything bad...  I
guess GCC has only just started warning about this because older
versions of <assert.h> provide the necessary parentheses already.

server/peer.c: Always send `BYE' packets on peer death.

The peer can ignore them if it likes.  The point really is that
different ends of a connection can have different ideas about its
ephemeral-ness without any logical problems.

server/admin.c: Don't automatically mark `knock' peers as ephemeral.

This was a bad idea on my part.

Consider a server which lives behind NAT, and therefore needs to use
`knock' or some similar dynamic arrangement so as to inform its peers of
its current public address.  This shouldn't be considered ephemeral.  In
particular, if its peer decides, because of flaky networking, say, that
it's no longer available, and sends a `BYE' packet, our server shouldn't
actually stop trying to re-establish the connection.

server/tests.at (knock): Tell `bob' explicitly that `alice' is `ephemeral'.

This is currently redundant, but I'm planning to modify the way
ephemeral peers work.

t/Makefile.am: Run the tests in parallel.

Now that the ephemeral ports stuff works, this seems like a good plan.

svc/tripe-ifup.in: Don't set remote IPv6 address until interface is up.

This is annoying.  Later Linux kernels complain about adding routes to
an interface which is still down.

debian/: Bump to Debhelper 10.

server/bulkcrypto.c: Replace dynamic assertions with static ones.

This adds a dependency on mLib 2.4.1.

proxy/tripe-mitm.c, server/tests.at: Optionally use kernel-assigned ports.

Add an option for the `peer' ports in `tripe-mitm' to be allocated by
the kernel, and for the port numbers to be written to named files.  Use
this in the tests so as to avoid conflicts between concurrent instances
of the tests.  It's not perfect: some of the tests require that a new
`tripe-mitm' take over a port from an old one, and there's a chance that
some other process might have grabbed it in between -- but it's much
better than it was before.

I should also consider using `noip' for this testing.

Release 1.5.3.

peerdb/tripe-newpeers.in: Fix output crash when there are `user' records.

Oh, dear.  This really hasn't been a good release.

Release 1.5.2.

Fix bungled `tripe-wireshark' package.

Release 1.5.1.

server/tests.at: Add roundtrip test for the crypto transforms.

This should avoid future débâcles like the one we had this time.

server/test.c: Trace the key-exchange and bulk crypto details.

server/test.c: Don't try to trace if it's turned off.

server/test.c: Don't segfault if the keyring is bogus.

server/tripe-admin.5.in: Document the `unsuitable-aead-cipher' warnings.

This was missed out of the earlier AEAD work.

server/tripe-admin.5.in: Reorder the `KEYMGMT' warnings.

Now they're alphabetical by complaint.

server/bulkcrypto.c: Support smaller nonce spaces, down to 40 bits.

Pack the type into the available space, however small it is.

server/bulkcrypto.c: Abstract out the AEAD nonce formatting.

No functional change at this time.

server/bulkcrypto.c: Adjust the scale in the data-format diagrams.

Two spaces per byte seems to work well.

server/bulkcrypto.c: Fix segfault in `aead' algorithm setup.

I think I must have completely failed to test this prior to release.
Brown paper bag time.  Add a new keyring to test this sort-of properly.

Release 1.5.0.

configure.ac, debian/control: Set Catacomb version now that it's released.

Merge remote-tracking branch 'origin/1.0.0pre19.x'

* origin/1.0.0pre19.x:
  Release 1.0.0pre19.1.
  debian/: Use `dh_python2' for packaging.

debian/control: Move the Wireshark gubbins back to general `Build-Depends'.

The Wireshark plugin directory is architecture-specific, so, while the
Lua dissector is architecture-neutral, the install directory isn't and
we have to build a separate package for each architecture.

This is rather vexing.

server/bulkcrypto.c: Implement a bulk transform based on AEAD schemes.

Reimplement `naclbox' mostly in terms of this.  It doesn't quite fit,
because `naclbox' can't be used directly for challenges.

server/tripe.8.in: Use Rogaway's spelling `blockcipher' consistently.

server/keymgmt.c: Trace the bulk-transform name.

Most of the bulk-crypto trace is pretty good, but somehow I failed to do
this before.

pathmtu/pathmtu.c: Disable `raw' strategy if we don't have getifaddrs(3).

This isn't available on Android earlier than N.

pathmtu/pathmtu.c: Place `addreq' with the rest of the `raw' strategy.

It's not needed for the `linux' strategy.

pathmtu/pathmtu.c: Indent conditional `#define' properly.

server/Makefile.am: Build most of the server into a library.

As a bonus, we get to avoid having to mention most of the files twice to
build both the server proper and the test program.

server/: Make initialization errors be non-fatal and restartable.

It should now be possible to trap any error during startup, change any
part of the configuration, shut down the affected components, and
restart everything again, without taking down the whole process.

Mostly, this involves replacing the existing calls to `exit' with
goto-cleanup and return codes.  In some cases, the existing
functionality has been reordered to make cleanup easier.  I didn't leave
this for a separate commit, because including those changes here makes
it clearer what they're for and should make it easier to check that
they're the right fixes.

The details are:

  * admin.c (a_listen): Move resetting the `umask' to the end.

    Specifically, after the last part of the function that can fail.
    This avoids a double reset if the final part, the call to listen(2),
    fails.

  * admin.c (a_init): Move creation of the service table to the end.

    This isn't (currently) a thing that can be cleaned up, so do it only
    after the parts that can fail -- specifically, initializing ADNS.

  * keymgmt.c (kh_init): Use `kh->kf' as a flag to prevent double init.

    As mentioned earlier, this is cleared by the static initializer, so
    we can safely assume that `kh->kf' is null if and only if the
    keyhalf requires initialization.

    Also, reorder slightly, to establish the cache hashtable only after
    the keyring file has been read.

  * keymgmt.c (km_init): Refresh before fetching the master key.

    Suppose we fail to initialize because the master key is missing.
    The right fix is to update the keyring file with the proper key, and
    then retry.  But at this point the private keyhalf has been
    initialized; so we must force a refresh of the keyring data.

  * keymgmt.c (km_init): Be idempotent regarding the master key.

    If there's a master key cached, then don't clobber it if we can't
    find it again.  On the other hand, if we find a different one this
    time then switch.

server/tripe.c, server/standalone.c: Split `main' out into a seprate file.

We can also remove the temporary hacks from `test.c'.

server/: Build a proper interface for handling tunnel classes.

  * Introduce functions for enumerating the available tunnel types,
    looking one up by name, and returning a default tunnel.  This
    eliminates the direct access previously used by the admin code.

  * Introduce a registration interface.  This isn't necessary for our
    purposes, but, in case it's not been obvious enough, the recent
    changes have been directed towards making the server code suitable
    as a library, and a user of this library might well need a custom
    tunnel class.

server/keymgmt.c: Capture copies of the keyring and tag strings.

Slightly complicated because the private-key tag can legitimately be
null to indicate that the old-fashioned search by key-exchange group
thing should be done.

server/keymgmt.c, server/tripe.c: Add `km_clear' to shut down keyrings.

A bit more fiddly than our previous efforts, but this completes the job
of unwinding all of the configurable components of the server.  The main
thing that doesn't get reset is the set of extant admin connections, and
I intend to leave those running until we have no choice.

server/peer.c, server/tripe.c: Add `p_unbind' to tear down the UDP sockets.

server/keymgmt.c (kh_init): Remove pointless setting of `kh->kf'.

It's statically initialized to be null.  The only reason it might not be
is because we already changed it.  Hmm... that gives me an idea.

server/tripe.c, server/admin.c: Add a way to end the main loop.

This replaces the rather abrupt `a_quit' function.  Cleanup is handled
in `main', using the facilities added recently.

For now, there's a stub version of `lp_end' in `test.c', joining
`iv_...' from earlier.

server/peer.c, server/admin.c: Introduce `p_destroyall'.

Rather than having the loop in `a_quit' directly.

server/admin.c: Introduce `a_unlisten' to shut down the admin socket.

server/peer.c: Engage the interval timer only while there are peers.

The reason we need the interval timer is (a) to top up the entropy pool,
and (b) to clean up old keysets.  If there are no peers, we do no key
exchange, so we don't need any random bits, and there are no keysets to
clear away.  So we can save wakeups in this case.

For now, there are stub versions of `iv_(add|rm)reason' in `test.c', but
these will vanish once we split `main' out into its own file.  (Spoiler
alert!)

server/tripe.c: Formalize the main loop machinery.

The new `lp_init' function has taken on a number of miscellaneous
initialization tasks.  But nothing has really changed much.

server/tripe.c: Formalize the interval-timer arrangements.

The hard stuff is now (mostly) detached from the main initialization
code.  There's also some machinery, currently unused, for disabling the
interval timer while it's not doing any good, which might be useful in
energy-constrained devices.

server/keymgmt.c: Replace ad-hoc `die' with an `ABORT'-level warning.

It's either this or an `assert', and for some reason I didn't do that
the first time.

server/peer.c (p_bind): Add the selector after fetching the port number.

This will make more sense later.

server/: Issue `WARN' messages for (most) initialization errors.

Now, most initialization problems which aren't to do with duff arguments
are diagnosed as `WARN' messages rather than human-readable messages on
stderr.

Document the new warnings.

server/tripe.c: Establish the stdio admin client early.

This way, warnings from peer initialization, for example, are reported
to the stdio admin client rather than stderr.

server/admin.c (a_format): Introduce `?ERR' for explicitly named errors.

Judging by the documentation, `?ERRNO' is supposed to take an `int'
argument and format the error it specifies; but it actually works by
examining `errno', and all the callers know this.  Changing it now seems
pointless, but I do want to be able to report errors in cases where
`errno' is or might be stale, and stuffing an error code back into
`errno' just so that it can be reported seems rather ugly.

Instead, add `?ERR' which /does/ accept an `int' argument, and fix the
documentation so that it describes reality.

server/admin.c (a_listen): If stat(2) says the socket has gone, then retry.

This saves us from making an embarrassing `stat failed because ENOENT'
report.

server/keymgmt.c: Remove redundant error reports.

Any I/O errors encountered while reading the key file will have been
reported already.  In the same terms.

Previously, the final `die' message wasn't actually redundant, because
the earlier warnings got swallowed if we were going to run as a daemon.
It's obviously wrong that this happened, and the underlying cause has
been fixed properly now, so this isn't a concern any more.

server/tripe.c: Be more clever about when to switch error output.

If we're going to detach and be a daemon, then we don't adopt
stdin/stdout as an initial administration client -- because we're going
to close these streams and disassociate from the surrounding
environment.  On the other hand, if we /are/ going to adopt
stdin/stdout, it makes sense to report errors to the client listening
there as soon as we can, so that they can keep informed of our
progress.  So take advantage of the new flexibility afforded by the
`a_switcherr' function, and switch the error output early or late
depending on whether we're going to daemonize.

server/keymgmt.c (kh_reopen): Promote the happy path.

server/: Split peer and admin initialization into smaller pieces.

Split admin initialization into:

  * setting up internal data structures, which is `a_init' as before;

  * establishing the daemonic signal handlers, which is now `a_signals';

  * creating the admin socket and listening for incoming connections,
    which is now `a_listen'; and

  * switching alert reporting over to admin clients rather than stderr,
    which is now `a_switcherr'.

Given that `a_listen' now need not actually be called, only unlink the
socket object if we actually created it.

Split peer initialization into:

  * setting up the data structures, which is `p_init' as before; and

  * creating the UDP sockets for communicating with other peers, which
    is now `p_bind'.

This will make sense in context.

squash! server/: Split peer and admin initialization into smaller pieces.

server/admin.c: Split out signal-handler establishment.

server/admin.c: Note that it's safe to call `a_create' before `a_init'.

A good thing too, because we do exactly this.

server/tripe.c: Don't start privsep helper unless some tunnel needs it.

It just saves having a useless process hanging off the side.

server/: Record port number with UDP socket, rather than querying.

The old `p_port' function queried getsockname(2) at runtime, and bailed
in the (very unlikely) event that it failed.  Avoid this situation by
recording the port number when the socket is set up and just using the
recorded value when it's needed.

This was the only remaining abnormal exit after the server has started.

server/: Wrap `udpsock' selectors up in a custom structure.

Now we can easily add more per-socket information.  No functional change
at this stage.

Rename `common/libtripe.a' to `.../libcommon.a'.

The main server will be a library soon, and it deserves that name more.

common/Makefile.am: Zap spurious trailing blank line.

server/: Post-merge fixup.

There's a slightly nasty semantic conflict between the two branches:
mdw/knock introduces a new function which sends a packet to an address
which might not belong to a registered peer, and mdw/ipv6 means that
this now has to cope with multiple address families.  The mess is
surprisingly limited, all things considered.

Merge branches 'mdw/knock' and 'mdw/ipv6' into bleeding

* mdw/knock:
  Add notion of `ephemeral' associations and a goodbye protocol.
  Add new `knock' protocol.
  server/{keyexch,peer}.c: Maybe key-exchange messages come out of the blue.
  server/keyexch.c (kx_message): Squish vertically.
  server/keyexch.c: Abstract out the common message-handling behaviour.
  server/keymgmt.c: Track and find keys by their 32-bit IDs.
  server/test.c: Add a program to assist unit tests.
  server/servutil.c: Add utilities for plain asymmetric encryption.
  server/servutil.c: Add utilities for simple leaky-bucket rate limiting.
  server/keyexch.c: Rename kx_init => kx_setup.
  server/: Augment challenges to allow a payload.
  server/chal.c: Capture `master->algs.bulk' in a variable.
  server/chal.c: Rename bulk => bchal.
  server/: Expose and enhance the bulk-key-derivation protocol.

* mdw/ipv6: (64 commits)
  contrib/greet.in: Accept IPv6 addresses.
  contrib/tripe-ipif.in: Fixing for IPv6.
  svc/conntrack.in: Add IPv6 support.
  svc/conntrack.in: Split out a base class from `InetAddress'.
  svc/conntrack.in: Contemplate multiple address families.
  svc/conntrack.in: Allow multiple networks in a peer pattern.
  svc/conntrack.in (kickpeers): Refactor and reformat the search loop.
  svc/conntrack.in (kickpeers): Rename `map' variable.
  svc/conntrack.in: Process peer patterns in order.
  svc/conntrack.in: Maintain config groups in a dictionary.
  svc/conntrack.in: Make an `InetAddress' class to do address wrangling.
  svc/conntrack.in: Factor out network parsing.
  svc/conntrack.in: Gather address hacking functions into a new section.
  svc/conntrack.in: Introduce a function for parsing address strings.
  svc/conntrack.in (strmask): Consistently return a string object.
  svc/conntrack.in: Fix netmask parsing.
  svc/conntrack.in: Leave time for network configuration to settle.
  svc/conntrack.in: Hoist `netupdown' above `kickpeers'.
  server/, mon/: Introduce transport of TrIPE over IPv6.
  server/addrmap.c (hash): Visually tighten the arithmetic.
  ...

Add notion of `ephemeral' associations and a goodbye protocol.

When TrIPE kills an ephemeral peer, it sends a `bye' message to the
peer.  When TrIPE receives `bye' from an ephemeral peer, it kills the
peer (without sending `bye' back).

Augment the `connect' service to set appropriate flags when adding
peers, and the Wireshark dissector to understand the new message.

Release 1.0.0pre19.1.

debian/: Use `dh_python2' for packaging.

contrib/greet.in: Accept IPv6 addresses.

contrib/tripe-ipif.in: Fixing for IPv6.

Accept `INET6' addresses from the server; detect IPv6 literals from the
configuration file; and adjust the MTU accordingly to compensate for the
different IP header lengths.

svc/conntrack.in: Add IPv6 support.

This is now a simple matter of implementing an `Inet6Address' class and
tweaking the `parse_address' function.

svc/conntrack.in: Split out a base class from `InetAddress'.

Now adding more families should be pretty easy.

svc/conntrack.in: Contemplate multiple address families.

A number of relatively simple changes, with no overall functional change
except for a few diagnostic messages.

  * Attach the address-family code and a name string to the
    `InetAddress' class.  This will mean that we can add new address
    families without breaking things.

  * Make `testaddrs' (and related variables) be a dictionary, mapping
    address families to addresses, rather than just a lone address.

  * Ensure that the networks in a peer assignment belong to the same
    family.  They will do for now, because there's only one.

  * Have `kickpeers' maintain a local IP address for each family, rather
    than just a single one.

svc/conntrack.in: Allow multiple networks in a peer pattern.

This is sensible now that we're not trying to compare them for
specificness.

svc/conntrack.in (kickpeers): Refactor and reformat the search loop.

The `ip' variable wasn't doing anything except remembering whether we'd
found a match, so replace it with a flag.  This frees up the `ip' name,
so give that to the bizarrely named `ipq' variable which holds the
discovered local address.

svc/conntrack.in (kickpeers): Rename `map' variable.

We shall want to call the `map' function.

svc/conntrack.in: Process peer patterns in order.

Rewrite the configuration file parser entirely so as to process the
patterns in order, rather than messing about with topological sorting.
This will let us introduce various improvements to patterns which don't
have a clear specificness ordering.

svc/conntrack.in: Maintain config groups in a dictionary.

Nothing cares about the relative order of the groups, and the
`showgroup' command wants to be able to look one up by name.  Really,
the question is why I didn't do it like this the first time.

svc/conntrack.in: Make an `InetAddress' class to do address wrangling.

The name is a little misleading: it can also represent a network, but
separating the two turns out to be a little tedious, so I don't bother.

This means that the configuration now actually contains (PEER,
TEST-ADDRESS, LOCAL-NET) triples, rather than keeping the address and
mask portions of the LOCAL-NET separate.

This is rather an invasive change.  Sorry.

svc/conntrack.in: Factor out network parsing.

The new function is a little stricter than the old one.  The address
given must actually be the base address of the network, rather than any
old address within it.

svc/conntrack.in: Gather address hacking functions into a new section.

svc/conntrack.in: Introduce a function for parsing address strings.

It still returns a raw integer -- for now.  But it's a start.

svc/conntrack.in (strmask): Consistently return a string object.

Inexplicably, this used to return an integer if the mask covered a
prefix.  Fortunately, it was always invoked in order to produce a thing
which is printed via `%s', which stringifies anyway.

svc/conntrack.in: Fix netmask parsing.

  * Improve the checking for a prefix length: see if the thing is
    entirely made of digits, rather than searching for a `.'.

  * More importantly, if we have a general netmask, then parse the
    correct part of the network spec as the mask.

svc/conntrack.in: Leave time for network configuration to settle.

Introduce new machinery to delay `kicking' the peer configuration.

  * Changes as a result of reports from NetworkManager and similar are
    now delayed, currently by 2s.  Further reports restart the delay, so
    we update 2s after the last report in a sequence.  This long delay
    is unfortunate, but necessary according to my experiments.

  * The 30s interval timer no longer does anything if a kick is
    scheduled but delayed.

  * Kicks resulting from initial state inspections aren't delayed;
    neither are direct user requests via the `up' and `down' commands.

svc/conntrack.in: Hoist `netupdown' above `kickpeers'.

This will be part of a collection of little functions, and some will
definitely want to be above `kickpeers'.  It seems silly to separate
them with this monster function.