uslip: Associate a done-function with gobblers.

These work just like the dribbler done-functions.  We're going to need
this soon.

uslip: Don't let gobblers leak file descriptors.

The otherwise aptly-named gobbler_close function neglected to actually
close its file descriptor.

uslip: Fix SLIP escape handling.

In particular, do_slip_in failed to leave ESC mode having handled the
escaped character.

server/admin: Fix client destruction some more.

It's possible that finally destroying a client can kill others.  For
example, if the second client (a) has sent EOF, and (b) has a background
command outstanding with the first; then when the first sends EOF, the
second gets taken down too.

Unfortunately, what actually happens in this case is that the newly
defunct clients get put on the dead list -- and then ignored because the
dead list is silently killed at the end of a_destroypending.  Fix by
clearing the list at the top of a_destroypending, and doing the whole
job repeatedly until there are no more cascades.

The change is mostly indenting a loop: it looks less scary with -b.

server/keyexch: Store check-value key hash in the right place.

Another botch from de7bd20be1c41f8f70e98ab498ffb4a82800a9d8.  The
respond function stashes a hash of the peer's check value so we can
recognize it again without having to verify it fully.  Unfortunately, it
stashes it in the wrong slot, so (a) it gets overwritten immediately
afterwards, and (b) the code which actually tries to do the checking
finds some rubbish instead.

The most prominent symptom of this bug under normal circumstances is a
large number of bad-expected-reply-log warnings in the log file.  What
happens is that, after a successful key exchange, the two peers end up
quite precisely synchronized -- so much so, indeed, that they're very
likely to run the entire key exchange protocol truly simultaneously.  As
a result, we have to process both a full challenge and a reply, both of
which contain a check field.  The second time, this fails because of the
bug.  This continues until the two fall out of lock-step with each
other.

server/keyexch: Fix message name table in trace output.

In de7bd20be1c41f8f70e98ab498ffb4a82800a9d8, I removed the pre-challenge
message in order to streamline the protocol.  Unfortunately, I forgot to
update the pkname table which maps message numbers to human-readable
name strings, which makes decoding trace messages rather difficult.

maint-utils: Handy script to make the `bleeding' branch.

server/admin: Brown-paper-bag fix.

Commit 165efde7 changed a_resolve to not require a port number.
Unfortunately, I botched it and wrote a test for av[i + 1] as
av[i + i].  Result with i = 0: very different.

While investigating this bug, I became nervous about the number of
is-av[i]-null tests going on when str_qsplit doesn't actually guarantee
to leave a null terminator behind if it uses all the array slots.  So
I've allocated an extra slot and zeroed it explicitly.

init/tripe-init: Pass correct options for setting the user and group.

They're capitals, -U and -G, not -u and -g.

vars.am: Actually apply PACKAGE and VERSION substitutions.

How embarrassing to have forgotten this!

server: Test script.

server: Option `-F' to run the server in the `foreground'.

This means that the server will quit when it sees EOF on standard
input.  Useful for testing, since it means that your processes aren't
going to continue running after you let them off the leash.

Also make tripectl pass `-F' to a tripe that it's starting up so that it
won't continue without (say) logging.

server: Actually recognize -n on the command line.

For some reason this got missed out of the short-options list.  I've now
eyeballed the long- and short-options lists and checked that there
aren't any other missing entries.

server: Introduce another temporary buffer for debugging output.

Debugging output trashed important things like hashes in buf_t.
Introduce a new buf_u and use that in functions like mpstr and gestr.

uslip: New program providing a fake SLIP interface.

This is useful for doing testing on a single machine.  We'll work out
how to test the system-specific tunnel drivers later, but that really
will need multiple machines.

server, common: New header slip.h contains definitions for SLIP.

This will be needed by another program as well.

keys: Add test script.

I've recently found a number of bugs in this area, so it's probably
worth writing a test script to make sure they stay dead.

This wouldn't be complete without some edits to the tripe-keys script,
so (a) make it executable in the build tree, and (b) flush output before
running subprocesses so that the output appears in the correct order
even when it's redirected to a file.

Add testing infrastructure.

infra: Ignore log files.

Debhelper seems to have started leaving these log files behind, which
may be useful but they certainly have no business upsetting Git.

server, common: Split more code into utilities.

  * Move mystrieq to server/servutil.c where it belongs.

  * Move code to resolve user and group names, and to set user and
    group, into common/util.c where the client can get at it.  (This
    will eventually be useful for privilege separation.)

Various minor cleanups.

  * Spacing fix in server/peer.c.

  * Minor reformatting in tunnel drivers and client.

  * Remove bogus declaration for long-dead u_daemon in common/util.h.

  * Reformatting of client/tripectl.c.

tripe.h: Make job index be unsigned int, not unsigned short.

This is mainly (a) because we don't actually save space by using a
short, and (b) because it shuts up a compiler warning.

The warning is annoying.  The compiler complains that it has proven that
an assertion is always true.  Of course, it's nice when it can prove my
assertions, but they're /meant/ to be provably true!

This particular instance is particularly annoying, since it's only
/trivially/ true in the sense the compiler is warning about as a result
of a coincidence of data-type ranges, which may not hold on other
architectures -- on which the assertion is nontrivial but still
important.

Duh.

server/peer: Use hash tables to find peer records.

configure.ac: Trivial whitespace fixes.

Pesky spaces sneaking in before tabs.

python: Replace sre by plain re.

Annoyingly, Python 2.4 documentation suggests that sre is the right
module to use.

debian: Don't depend on sysvinit.

It seems that this is bad form.  Nobody else does it, and it stops the
package working on Ubuntu.

keys: Reformat in line with my newer commenting conventions.

Overhaul manual pages.

Make the source code look prettier, and attach copyright notices to them
all.

tripe-keys.conf.5: Description of sig-url shouldn't be circular!

Just a typo.

tripe-keys: Don't hard-code the `-l' flag when generating master keys.

It makes automated testing painful.  Instead, provide a configuration
parameter master-keygen-flags which defaults to -l but can be overridden
by people who know what they're doing.

tripe-keys: Provide upload-hook for more complicated publishing.

The existing arrangement of writing the updated files to a local directory
are satisfactory for simple situations, but it's not actually desirable
to handle sensitive cryptographic keys (e.g., the TrIPE master key!) on
the same machine as a public-facing web server.

The upload-hook can contain an arbitrary shell-command, though it'll
typically be an invocation of rsync or similar.

tripe-keys: Fix iteration over keyring.

In catacomb-python 426e898f..., KeyFile was changed to iterate over
key-ids rather than keys, for consistency with other mapping objects.
This means that we need to use itervalues explicitly now.

Build system overhaul to conform to new standards.

  * Rename configure.in to configure.ac, and rewrite using modern
    Autoconf macros.

  * Rewrite the Makefiles to be cleaner and more readable, using
    Automake conditionals and appending.

  * Move the manpages to live with their respective code components
    rather than languishing in their own subdirectory.

  * Switch the Debian build process over to using CDBS.

doc/tripe.8: Fix layout of synopsis section.

Something got upset by the whitespace stripping, I think.

server: Add a peer without sending key-exchange packets.

When contacting a passive peer, the initial pre-challenge will cause the
peer to complain about an unsolicited packet.  The -cork option to ADD
makes the new peer silent until contacted by the (presumably awoken)
passive peer.

Provide a pkg-config file for use by separately compiled services.

debian: Pre-emptive reorganization.

Use dh_install to install things into the correct packages.  It's
getting too cumbersome to do all this by hand.

Remove crufty old CVS $Id$ markers.

Infrastructure: Get it building again, using pkg-config.

Use the new official IANA-allocated port number 4070.

This is now the default port selected by the server (say -p0 for
explicit dynamic allocation).  Also let the ADDR command default to 4070
so that nobody needs to remember it.

server/tun-unet: Fix stupidity in t_create.

Obviously I never watched this build carefully enough.

cleanup: Whitespaces fixes, left right and centre.

doc: Fix distribution of generated manpages.

Oops, this is leftover from change 797cf76b...

Also fix the tripemon(8) manpage to use configured paths, and (cheekily)
fix tripemon itself to support TRIPESOCK.

server: Fix some bad warning messages.

Oops, the arguments to some calls to a_warn are malformed and can cause
crashes.  Fix them.

server: Correct handling of interface names in tun interface.

Now that interface names can be changed, we need to let the tunnel
driver know of changes so that it can produce the right messages when
things go weird.  It's better to let the peer edifice handle interface
names, so

  * let `create' return the interface name rather than have p_ifname ask
    the tunnel explicitly, and

  * replace the `ifname' query with an optional `setifname'
    notification, which is currently used only by the SLIP driver to
    maintain the correct name for its persistent interfaces.

configure.in: Reformat --with-logfile help text.

Useful functions (u_daemon and versioncmp) moved to mLib.

Remove the code here, and use the mLib versions.

debian: Still making tripe-ethereal, so ignore directory

New environment variable TRIPESOCK.

Provides a handy default for the `--admin-socket' option.  Added
documentation.

Also fixed paths in documentation to reflect the configuration choices;
in particular, they were wrong in the Debian build.

client/tripectl: Flush output after each line.

Otherwise script users can be left hanging for long periods of time.

server/admin: Fix core dump if ADD wasn't given enough arguments.

Just a stupid off-the-end-of-the-array error.

tripectl: Quotify the arguments rather than hoping for the best.

To make this work, move quotify from admin to the shared utilities.

Merge branches 'cleanup' and 'services'

* cleanup:
  server/admin: Fix tokenization of statistics output.
  Fix typos in messages.
  doc: Various small cleanups to tripe-admin.5.
  cleanup: Various simple whitespace changes.
  cleanup: Rename a few badly-chosen variables.

* services:
  doc: Document the services messages.
  admin: Implement the main job commands.
  admin: Implement job table infrastructure.
  admin: Service ownership infrastructure and commands.
  servutil: Implement version number comparison.
  admin: New ?TOKENS formatting directive.
  admin: Rename the unknown-service error.
  admin: Improve handling of background jobs.
  admin: Option parser macros.
  admin: Put all command options at the start of the command-line.
  admin: Fix premature close in a_bgrelease.
  admin: Remove locking; new safe client destruction.

Conflicts:

server/admin.c

doc: Document the services messages.

admin: Implement the main job commands.

That's SVCSUBMIT to submit new jobs, and SVCINFO, SVCOK and SVCFAIL for
responding to them.

admin: Implement job table infrastructure.

We'll want this to keep track of which jobs a service provider has
outstanding.

admin: Service ownership infrastructure and commands.

  * Implement a table of named and versioned services.

  * Allow clients to claim and release services, and find out about
    claimed services.

  * Notify clients when services are claimed and released.

  * Notify a service provider when another client provides a later
    version.

servutil: Implement version number comparison.

We use an extended version of the dpkg version comparison algorithm.
See the code for a description.

admin: New ?TOKENS formatting directive.

Format a vector of pretokenized strings.  This obviates the need for
a complicated @alertcmd@ and suchlike.  It also means we can, at last,
resurrect @a_alert@.

admin: Rename the unknown-service error.

We'll want that when we report errors about client-provided services.
Renamed to unknown-port.

admin: Improve handling of background jobs.

New commands BGCANCEL (cancel an outstanding job) and JOBS (list
outstanding jobs).

admin: Option parser macros.

There's too much clone and hack in options parsing.  Cover up the
repeated code with macros.

Also, sneak in a little fix: complain if there are too many arguments to
a PING or EPING command.

admin: Put all command options at the start of the command-line.

This makes life much easier for client libraries, which can (for
example) put `-background TAG' in a standard place to obtain
background operation.

admin: Fix premature close in a_bgrelease.

The function incorrectly destroys the connection at the release of the
first background job after close; it should wait until they've all
finished.

admin: Remove locking; new safe client destruction.

The locking stuff is fiddly, and prevents cleanup of pending background
operations during destruction.  Instead, we add `destroyed' clients to a
list which we clean up just before going back to select.  This ensures
that they get cleaned up at a safe place, when there aren't functions
threaded on the stack which will be upset by the admin block vanishing
under their feet.

server/admin: Fix tokenization of statistics output.

Fix typos in messages.

  tripe-init: `want do die'.

doc: Various small cleanups to tripe-admin.5.

  * Duplicate the timespec description for PING.

  * Put BGDETACH in the summary of responses.

  * Tag EPING as a command to include in the summary.

  * Fix the description of the INET address family now that we have
    background commands.

cleanup: Various simple whitespace changes.

cleanup: Rename a few badly-chosen variables.

  * admin: A few old `tag' arguments renamed to `status'.

admin: New command SETIFNAME to change an interface's recorded name.

Also the machinery in peer.c to make it work, and documentation for the
new command and matching notification.

Great reorganization.

Split all the sources into subdirectories.  This will look rather more
sensible when we add the Python support stuff.

build: Distribute make-summary to fix distcheck.

I've been concentrating too much on Debian packaging and the standard
autoconf install has rotted a bit.  Oops.

gitignore: Track rename of Ethereal => Wireshark in debian subdirectory.

configure: Fix formatting.

Rename ethereal -> wireshark.

Also make the plugin actually work.  Probably.  Involves messing with
the Debian packaging and search-and-replace on the build system mainly.

keyexch: Simplify key-exchange protocol.

Remove pointless cookie message.  Only allocate challenge blocks on
receipt of a full challenge.

Also fix bugs introduced in previous changes.

keyexch, keymgmt: Include the peer's public key in the check hash.

This turns out to be necessary for security: otherwise an adversary can
cross over two sessions, which is probably undesirable.  This also
requires that we know our own public key, which was previously
unnecessary.

Except for session-ids (we don't care if two `sessions' with the same
peer get crossed over, because we don't distinguish them anyway), the
protocol now matches the one described and proved secure in the crypto
paper.

keyexch: Group element encodings must have constant length.

SECURITY FIX.  COMPATIBILITY BREAK.

Using the `buf' encodings of group elements leaks length information.
Use the `raw' encoding instead.  I suspect that this really is a bad
security property to have; certainly the security proof for the key
exchange protocol depends on all encodings having the same length.

tripe-admin manpage: Generate a command and message summary.

There's such a big pile of information it seems nice to provide a quick
index at the end.  An unpleasant awk script seems to do the job nicely.

admin, resolve ops bugfix: initialize tag in resop early.

We can call the completion function before detachment, and it might try
to trace its job tag.  This initialization just makes sure that it
doesn't do something stupid here.

admin: Initialize reference counter for client blocks.

Good ol' Valgrind!

Merge branch 'tokens'

* tokens:
  Correctly tokenize output to admin clients.

Correctly tokenize output to admin clients.

This is a wide ranging change, and there may well be bugs in some of the
message reporting code.

tripe-keys: Use curl instead of wget.

It's generally much better, and in particular, it can cope with file:
URLs, which wget chokes on, which is a nuisance for testing.

infra: Clean up project setup

Extract Subversion ignore data.

Greetings and challenges.

Packet loss percentages.

Stupid bug.

Various little fixes.

Weird.  tripe-keys got backdates somewhere.

One that got away.

More support scripts and other cool stuff.

Backgrounded commands and RELOAD.

Fix uninitialized variable in p_create.

Various fixes.

Fix for new key-data interface.

Stupid labelling bug.

Keepalives and pings.