Release 1.2.3.

noip.c (do_implicit_bind): Handle `SAME' impbind entries properly.

For some reason this just got left out.

noip.c (do_implicit_bind): Set wildcard port if we find a match.

Otherwise it's left uninitialized, and there is sadness.

noip.c, uopen.c: Provide fallback implementation of `SUN_LEN'.

Apparently Android's `libc' doesn't have this.

Release 1.2.2.

noip.c (getpeername): Don't swallow the return code.

How did this ever work?

Release 1.2.1.

noip.c: Support `SO_REUSEADDR', rather sketchily.

If the caller tries to bind, and has previously set the `SO_REUSEADDR'
flag, then unlink any existing socket unless there's an active listener
already.

noip.c (unix_socket_status): Return whether anyone's listening.

Add a new `LISTEN' flag which says whether anyone's listening on the
socket.  Maybe the listener has gone away and we can only see the
leftover connections.

noip.c (encode_single_inet_addr): Make the logic less weird.

Now it doesn't need to know quite so much about all of the possible
status values.

noip.c (unix_socket_status): Refactor status-file parsing.

Remove the early exit: we always process all of the entries now.  The
socket is `STALE' if we can't find any entries.

noip.c: Make the `unix_socket_status' return a bitfield.

The constant names are the same, but they're processed differently.

noip.c (recvfrom, recvmsg): Don't swallow the underlying system error.

Move the call to `real_recvfoo' outside of `PRESERVING_ERRNO'.

Release 1.2.0.

debian/rules: Use old-fashioned `dpkg-buildflags' options.

I'm still trying to build packages for `wheezy' sometimes.

debian/control: `noip' only works on Linux.

Because it trawls through magic Linux `/proc/' files.  Patches welcome
for BSD support.

noip.1: Clarify ACL list syntax.

The ACLish keywords add lists rather than individual entries.  The
`impbind' keyword had no explanation of how entries were separated.  Fix
this garbled mess.

noip.1: Associate connection-to-remote-host aside with `realconnect'.

It seemed like a non-sequitur where it was, especially after the
`impbind' section was added.

noip.1: Fix bungled description of comment syntax.

build, debian/: Add a `configure' script to make things easier.

Amazingly, this is good enough to work with Debhelper, so throw out the
earlier fiddling.

Makefile: Support out-of-tree builds.

The `distcheck' target shows how you can do this, though it's rather
cumbersome.

Makefile (distcheck): Propagate jobserver parameters to the test build.

Makefile (distdir): Copy files rather than linking.

Linking will hardlink symlinks, and that won't end well.  All our files
are tiny, so it's not worth worrying about the performance effect.

Makefile: Sink the silent-rules machinery.

This is not really user-serviceable.

Makefile: Refactor the silent-rules machinery.

Now it's more like what I do elsewhere.

Makefile: Generate and include dependency information for headers.

Not that there are any right now, but it's the principle of the thing.

Makefile (all, clean): Make these be double-colon targets.

Makefile (clean): Have a variable for things to clean.

Makefile: Factor out common build arrangements for hacks.

  * Name the `..._SOURCES' variable literally after the hack, so that it
    can be recovered mechanically.

  * Accumulate all of the sources together, into an `ALL_SOURCES' and
    dump that into `DISTFILES', rather than marking them separately for
    distribution.

  * Replace the separate linker rules with a single static-pattern rule.
    This unfortunately needs `.SECONDEXPANSION' to work properly.

debian/.gitignore: Ignore Debhelper's little droppings.

.gitignore: Rearrange and make more sensible.

Things like `deb-build' are my local conventions, so leave them to my
common ignore file.  Ignore all `.so' files rather than just the ones we
know about.  Alas, we can't bulk-ignore the generated scripts, but at
least we can mark the rules for fixed files as being absolute pathnames.

Also add an entry for `/debian/debhelper-build-stamp', since that's
turned up.

noip.c: Support IPv6-mapped IPv4 addresses (nearly) correctly.

There's a longstanding bug here.  If a program makes an `AF_INET6'
socket, and then tries to connect to a v6-mapped v4 address, then `noip'
/ought/ to match that against an existing binding of the corresponding
real IPv4 address, and /vice versa/.

Also, to support non-`IPV6_V6ONLY' server sockets, when trying to
resolve an IPv4 connection, consider the IPv6 wildcard address as a
suitable match.

There's unfortunately quite a lot here, but it all really needs to be
done in one go to work properly.

  * Add `map_ipv4_sockaddr' and `unmap_ipv4_sockaddr' functions to
    do the grunt-work of the address handling.

  * In `encode_unused_inet_addr', check that a proposed v4 socket
    address doesn't conflict with any v6-mapped and/or wildcard sockets.

  * In `encode_inet_addr', try to match v4 remote addresses against
    local v6-mapped and/or wildcard sockets.

  * Complicate `return_fake_name' to optionally v6-map an IPv4 address.

  * Introduce and deploy a new `return_fake_peer' function, which
    inspects a socket's local address to discover what its `native'
    address family is, so that it can map the remote address if
    necessary.

  * Complicate `do_implicit_bind' so that it will v6-map the local
    address it's decided on, if necessary.  This is essential now as a
    means of recording the socket's `native' address family, as used by
    `return_fake_peer'.

  * Complicate `fixup_client_socket' to unmap remote v6-mapped v4
    addresses before trying to encode them as Unix-domain addresses.

noip.c (do_implicit_bind): Report the local address that we chose.

noip.c (fixup_client_socket): Reorder the preflight checks.

Superficially, it looks as if `acl_allows_p' will be cheaper than
`real_getsockname'.  But actually, if `acl_allows_p' passes the remote
address then we'll end up doing a `real_getsockname' anyway in order to
decide whether we need to substitute an unfaked socket, so this is a
false economy.

This shouldn't actually change the effective behaviour, but it will
improve performance -- not that this hack is especially performance
critical.

noip.c: Factor out the non-implicit-binding parts of `do_implicit_bind'.

The function's other responsibilities are taken over by a new function
`fixup_client_socket'.  When I did this, I anticipated that I was going
to have to make the latter rather more complicated, but it turns out
that it's not actually that far off.

No functional change, though the new function takes a different approach
to producing the same effective control flow.

noip.c (encode_..._inet_addr: Abstract out checking a single address.

There's this common pattern of building the Unix-domain socket address,
probing its status, maybe unlinking the socket if it's stale, and
returning the result.  Let's only write that once.

No functional change.

noip.c: Add commentary to some hairier functions.

No code change.

noip.c (encode_inet_addr): Exit early rather than branch round.

Promote the wildcard probe to the main line, and leave early -- to the
handy `found' label we just hoisted -- if the exact match was
acceptable.  This structure is more convenient for adding a more
complicated search strategy later.

No functional change.

noip.c (encode_inet_addr): Hoist `found' label to top-level.

No functional change: the control flow is as it was before, but this
structure invites use of the label from the other branch of the main
`if'.

noip.c (get{sock,peer}name): Don't clobber `errno'.

We might return a failure here, if the underlying syscall fails, and
should propagate the `errno' it returned.

Also hoist out some block-scope variables now that their enclosing block
has vanished.

noip.c ({get,set}sockopt): Stub support for IPv6 options too.

These are many and hairy, and this might cause trouble, but I'll risk it
for now.

noip.c: Replace `SOL_...' with `IPPROTO_...'.

Apparently this is the POSIX way.  The numbers are the same, so
whatever.

noip.c: Replace `WANT_...' constants with flags for `encode_inet_addr'.

I originally expected that I'd want to add some more flags here, but it
turned out that I was mistaken.  I think this is better anyway, and I
intend to continue this pattern elsewhere.

No functional change.

noip.c (encode_..._inet_addr): Only call `port_from_sockaddr' once.

Just a small cleanup; no functional change.

noip.c (acl_allows_p): Check that the ACL node address family is right.

Oops.  `sockaddr_in_range_p' can't check this, because it can only see
one address family.

noip.c: Placate GCC warning about `misleading' layout.

I think I prefer the way it was, but I'll go with having a quiet life.

noip.[c1]: New configuration feature for setting local addresses.

Some servers are picky about which addresses they're willing to accept
connections from.  Unfortunately, `noip' doesn't pick sensible local
addresses when implicitly binding sockets.  It can't usefully consult
the routing table (because we may be trying to simulate an entirely
fictional network of which the kernel knows nothing).  About the best we
can do is allow the user to configure the local address selection.

Add a new `impbind' configuration command which adds an entry to a list
of rules for choosing implicit binding addresses.

noip.1: Describe ACL entries in terms of <address-range> and <port-range>.

This corresponds with the newly refactored code.

noip.c: Factor out address-range handling functions.

  * Introduce `parse_addrrange' and `foreach_addrrange' for parsing.
    The former captures a representation of the range syntax, which can
    contain things like `local' that actually cover multiple ranges, and
    the latter iterates over the implied address ranges.

  * Introduce `dump_addrrange' to produce a readable description of a
    range in the debugging output.

No functional changes.

noip.c (parse_acl_env): Move next to `parse_acl_line'.

noip.c (encode_inet_addr): Avoid collisions with wildcard addresses.

If the caller is wanting to `encode' a currently floating socket (i.e.,
no explicit port number has been set and we're meant to pick one), then
make sure we pick a port number which doesn't collide with either the
chosen address /or/ the address-family wildcard address.  Otherwise, we
can get into the situation where process A listens on a floating socket
with a wildcard address, tells process B which port was allocated, and
then process B binds to localhost, tries to connect to A, and is stymied
because B actually allocated the same port number itself.

Really we ought to seed the RNG separately for each process.

noip.c: Add debugging to most of the syscall wrappers.

noip.c (present_sockaddr): Handle a null address pointer.

We'll want this functionality soon.

noip.c: Have parsers fail if there's trailing junk.

noip.c: Include the process-id in debugging output.

Reading dumps from complicated multi-process systems can be tricky
otherwise.

noip.c (parse_acl_line): Continue correctly after ADDRESS/PREFIX-LEN.

After an ADDRESS/PREFIX-LEN entry, the following delimiter will have
been consumed; make sure to put it back.

noip.c: Hack ioctl(2) as well.

It appears that Java's network machinery does lots of probing of network
addresses with ioctl(2), and /some/ of these ioctls don't work well with
Unix-domain sockets.  If we see one of these, then make a temporary
Internet socket and do the ioctl on that instead.

This really is quite unpleasant, but it seems to work well enough to
make Gradle work, for example.

noip.c (fixup_real_ip_socket): Support for temporary fixups.

Add a new argument to `fixup_real_ip_socket' where it can return a
temporary fixed-up copy of the input socket.  Sort out the call sites to
pass a null pointer (requesting the previous behaviour).

noip.c: Fix some holdovers with hardcoded address families.

Now all of the references to `_INET' are in the `Address-type hacking'
section where they belong.  This should really have been done as part of
the IPv6 work, but unaccountably wasn't.

noip.1: Fix source formatting.

noip.c: Make sure parsers update the cursor when they finish.

Even the ones which don't expect anything else to follow.

noip.c (parse_acl_line): Don't skip on past a zero byte.

noip.c: Add `const' qualifiers on `aclnode *' arguments.

noip.c: Missing blank line.

Very important.

noip.c (decode_inet_addr): Fix doc comment.

Makefile: Actually distribute `debian/source/format'.

Did it in too much of a rush.

Release 1.1.3.

noip.c: Fix memset length error.

Reported by Ian Jackson; thanks.

Release 1.1.2.

noip.c (create_sockdir): Make sure the socket directory isn't a symlink.

Oh, dear.  This is a long-standing bug, and a bad one for systems which
don't set per-user $TMPDIR directories.

Release 1.1.1.

debian/: Multi-Arch support.

debian/control: Fix Build-Depends.

debian/source/format: Apparently we need one of these nowadays.

debian/control: `fw' changed name to `fwd' in 2008.

Maybe it's time this package caught up.

Release 1.1.0.

Now with IPv6.

README: Fettle to include IPv6, and update configuration.

noip.c, noip.1: Add IPv6 support.

Just like that.  Of course, the hard work was done earlier.

noip.c, noip.1: Multiple address family support.

Abstract out all of the address-family-specific hacking into a
collection of utility functions.  Now, with a little luck, adding
additional address families later will be straightforward.

The main casualty is the notional support for arbitrary netmasks, though
in fact they never worked correctly.  They've now been silently dropped:
the new parser simply refuses to try.

noip.c (get_local_ipaddrs): Rewrite to use getifaddrs(3).

This is a much better function to use because it actually gives us all
of the addresses on each interface rather than just the first.

Also, discard duplicate addresses, because they're more likely now.  We
should have done this already, but unaccountably neglected to.

Fix the GPL notices.

noip.c: Allow PF_NETLINK sockets.

Anything fiddling with them probably knows what its doing.  Chromium
fails unless it can get one.

debian/rules: Switch to using debhelper 9 directly.

noip.c: Shut up some GCC warnings.

I have no idea what fixup_p was meant to mean, but it's gone now.

withlib.in: Fix bashism (`export VAR=VALUE').

For some reason, under `dash', this causes (say) `noip' to work
correctly for exactly one level of nesting.  An attempt to invoke a
nested `noip' results in

export: 4: noip.so: bad variable name

This appears to be a result of `dash' doing word splitting too early.

Release 1.0.6.

noip.c: Remove hints that we might intercept close(2).

We don't because glibc makes it too hard.

noip.c: Make sure it's an AF_UNIX address before counting the length.

Makefile: Cardboard cutout silent rules build.

noip.c: Silence GCC warnings about unused return values.

Actually act on errors from fgets while skipping the header on
/proc/net/unix, even though it's unlikely to be bad and we'll fail
later anyway.  The write in printerr is not worth worrying about.

noip.c: Boring whitespace fixes.

Release 1.0.5.

Merge branch 'master' of metalzone:public-git/preload-hacks

* 'master' of metalzone:public-git/preload-hacks:
  Version 1.0.4.
  noip (decode_inet_addr): Be more careful when converting addresses.
  noip: Don't try to support families other than AF_UNIX and AF_INET.

noip.c: Only fiddle with `errno' from `connect' if we faked things.

Otherwise some programs get confused.

README: Add some more notes about how useful `noip' is.

Version 1.0.4.

noip (decode_inet_addr): Be more careful when converting addresses.

The old version of decode_inet_addr would convert empty Unix-domain
addresses to wildcard Internet addresses, which is erroneous in a number
of situations.  In particular, this causes problems in some servers
which maintain parallel Unix-domain and Internet listening sockets, and
use the address family reported by accept(2) to decide what to do: if
the incoming connection is from an unbound (but real) Unix-domain
socket, it gets misinterpreted.

This fixes decode_inet_addr to be more selective about its decoding of
empty addresses.  It must do the decoding when swapping in a genuine IP
socket, but when called via return_fake_name it's wrong to do this: a
remote socket which ought to be decoded will have been bound either
explicitly by the peer or implicitly by do_implicit_bind.  (Actually,
getsockname might now be wrong when called on an unbound socket, but
that's probably a small price to pay -- and there's no way of returning
the right answer in this case anyway.)

noip: Don't try to support families other than AF_UNIX and AF_INET.

If there's demand, I might try to hack in IPv6 later.  But not now.

Version 1.0.3.

debian: Add lintian-overrides.

The libraries we ship aren't for linking against, so they don't need
sonames.