Do address-family dispatch with lots of switchy functions.

Instead of the previous table-of-function-pointers approach.  This seems
to have succeeded in pushing more work from call sites into addrfam,
which I think ought to be considered a win.

src/setup.c, src/transmit.c: New function finds udpsocket by AF.

This eliminates a slightly fiddly loop from two call sites.

query.c: Use a null `zone' pointer for `_submit_reverse' signalling.

Currently undocumented.  Might need to revert to using a magic token if
we come up with a better meaning for a null `zone' pointer to
`adns_submit_reverse_any'.

src/types.c: Use the new `checklabel' machinery for PTR queries.

Previously we'd check the query name and parse it into an address while
parsing the answer to the initial query, which means that you only get
an error about the name being malformed after waiting for the answer.
Now we have a better placed hook for dealing with this, so we should use
it.

Turn query domain parsing upside-down.

The `qdparselabel' hook is no more.  Unescaping of the query domain is
now done in one place, and always in the same way.  Checking of the
domain name is done through a new `checklabel' hook, with a more
convenient interface.

The checklabel hook does not have the original possibly-escaped version
of the query domain available to it; it can only check the name being
submitted on the wire, not the form in which it was provided to the
library.  This eliminates the inconsistency in SRV record handling,
where

\095http._tcp.distorted.org.uk

is acceptable, but

_\104ttp._tcp.distorted.org.uk

is not.  It also now prohibits SRV queries on `_wrong', which I think
was always intended but didn't actually work.

All queries are checked, including internally generated ones, so this
can be used to store information about the query domain for later use.

src/query.c: Remove flag hacking from adns_submit.

This is done in addr_rrtypes now.

Change how addr queries track which subqueries to make.

Now we have a vector of query types we might want to make, and a pair of
bitmaps tracking whether (a) we want to issue that kind of query,
and (b) whether we've had an answer to it yet.  This replaces
the (sometimes tricky) messing about with raw rrtype vectors.

As a bonus, these bitmaps are now set in the context structure, where
they can be pushed through adns__internal_submit and into qs_addr, so we
no longer need adns__qf_nosend.  This is, accordingly, killed.

src/types.c: Fix obvious stupid mistake.

And a sneaky style hack.

Introduce custom typeinfo for addr subqueries; kill __qf_senddirect.

Reinstate avoidance of pointless zero-length allocation.

This was removed while I tested hacks to the lower-level functions.

Move type-specific state into the query context structure.

The type state is now `tinfo'.  Rename the parent state to `pinfo' to
make sure we catch callers and get them to zero-initialize the tinfo.

Don't use adns__must_gettimeofday in callbacks.

There's potential re-entrancy and it's very bad.  Instead, call
gettimeofday(2) directly and cope with the fallout locally.

Make adns__sockaddr_ntop use a caller-supplied buffer.

Using an internal static buffer violates the (weak) thread-safety
guarantees.

Add a type hook for reporting the output record size.

Otherwise adns_rr_info doesn't work properly.

src/types.c (addr_submit): Make child queries from correct template.

src/types.c (append_addrs): Skip allocation and copying of empty vectors.

src/types.c (icb_hostaddr): Report correct output status and address count.

src/query.c: Handle the magic zero-byte-allocation convention.

regress/Makefile.in: More out-of-tree build hacking.

New init flags and options to influence address family selection.

Fix CNAME hack.

We must track which records have been returned explicitly, because we
don't want to resubmit queries which returned NODATA unnecessarily.

This lopens up the possibility of not cancelling the previous
outstanding queries, and taking their results if they're consistent with
our chosen CNAME, but I'm not doing that yet.

Finished CNAME handling.  Except that there's no testing, and it
probably doesn't actually work.

Start work on handling inconsistent CNAMEs in addr queries.

Fold expiry-time tracking into append_addrs.

More progress on addr support.

Fix a bunch of bugs to do with merging query results: in particular,
coping with NODATA responses.

Handle hostaddr correctly, working out which address families I didn't
get from the additional section and submitting queries for them.

Handle (and support in adnshost) the various address family flags
properly.

src/types.c: Support multiple address families in adns_rr_addr lookups.

This is now largely working.  A query for adns_r_addr now causes (via a
new type hook) concurrent child queries for all of the wanted address
families; we then stitch the answers together and report the result.

Astonishingly, we have NS, MX, and SRV queries doing approximately the
right thing.

Significant pieces missing:

  * init flags and configuration aren't yet processed;

  * (pap_hostaddr) we should query for wanted rrtypes but which aren't
    in the additional section, even if we got some other rrtypes for the
    name;

  * CNAME records aren't handled at all.

Support transport over IPv6.

We need multiple UDP sockets, for each address family required, with the
knock-on complexity to the sending and receiving code.

[[REBASE NOTE: this commit commutes syntatically with the adns_r_addr
enhancement patch, but won't actually work.  Before the series is
submitted for review, it wants to be hoisted /after/ that patch.  This
note can then be removed.]]

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>

Support IPv6 PTR lookups.

  * Support parsing of reversed IPv6 addresses, so as to check the
    forward address against the result of the reverse lookup.

  * Support reversing AF_INET6 socket addresses in adns_submit_reverse.

  * Parse IPv6 literal addresses in adnshost `-i' arguments and pass
    them on.

A side effect of this work is that PTR lookups now issue adns_r_a or
adns_r_aaaa child queries, as opposed to adns_a_addr queries.  I found
that this made things simpler -- especially as we don't have adns_r_addr
support for IPv6 yet.

Knowledge about the reverse-lookup zones is not kept in addrfam.  I want
to leave open the possibility of direct support for the obsolete
`ip6.int' zone, and other similar wrinkles.  This knowledge has moved
from adns_submit_reverse to adns_submit_reverse_any, which also needs
to find the appropriate name-reversal algorithm, and it seemed sensible
to keep them together.  The magical `default_zone' token used to pass on
this responsibility is a little grim.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>

src/types.c: IPv4 networks match IPv6-mapped IPv4 addresses in sortlist.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>

src/: Support IPv6 addresses in `sortlist'.

This involves a fair amount of messing about.  There's a new file
`addrfam.c' containing a number of address-family-specific facts and
utilities, packaged together in a handy structure.  The `sortlist'
representation now attaches one of these to each entry, and now keeps
a union of possible address types.

Lookup of addresses in the sortlist is also modified, to check the
address family before going any further.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>

src/setup.c: Check netmask more carefully.

[crybaby ~/src/adns/build]RES_CONF_TEXT="sortlist 172.29.199.1/24" RES_OPTIONS="adns_checkc:entex" adnshost -ta www-cache.distorted.org.uk
adnshost: ../src/check.c:83: checkc_global: Assertion `!(ads->sortlist[i].base.s_addr & ~ads->sortlist[i].mask.s_addr)' failed.
Aborted

Unfortunately, `ccf_sortlist' only checked that the free bits of the
network address are clear when given a general netmask, so if you give a
prefix length or leave it implicit then you hit a consistency check
failure later.

Check the mask later on, after all of the variants are handled.  There's
a little bit of variation in the message.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>

Hacking to make out-of-tree builds work properly.

This is mostly standard.  The regression-test scripts have had an
interface enhancement: they will look for source-like files using the
$srcdir environment variable, which defaults to `.' for compatibility.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>

FOUND on davenant
/u/ian/things/Adns/adns
-rw-rw-r-- 1 ian ian 12465 Oct 18  2006 client/adnsresfilter.c

FOUND on davenant
/u/ian/things/Adns/adns
-rw-r--r-- 1 ian ian 9594 Jan  2  2007 README.html

finalise version 1.4

@@ -1,3 +1,14 @@
+adns (1.3.0.99.1); urgency=low
+
+  Improvements for multithreaded programs:
+  * New documentation comment in adns.h explaining thread guarantees
+    (or lack of them), replaces `single-threaded' note at the top.
+  * Fix string conversion of adns_r_addr not to use a static buffer
+    (function csp_addr) so as to make thread promise true.
+  * Make an internal variable const-correct (expectdomain in pa_ptr).
+
+ --
+
 adns (1.3); urgency=low

   Portability fixes:

MINOR=3

get rid of junk file x.gdb

DISTVERSION 1.3

finalise adns 1.3

@@ -10,6 +10,9 @@
     (Report from Mihai Ibanescu.)
+  * Do away with `mismatch' variable in parse.c:adns__findrr_anychk so that
+    overzealous GCC cannot complain about members of eo_fls being
+    uninitialised.  (Report from Jim Meyering.)

@@ -1,13 +1,15 @@
 adns (1.3); urgency=low

-  Bugfixes:
-  * In configure.in, quote macro name argument to define() to
-    suppress spurious autoconf error.  (Report from Mihai Ibanescu.)
+  Portability fixes:
   * Cast ptrdiff_t to int for %.*s length in adnsheloex and adnslogres,
     as is required.  (Report from Jim Meyering.)
-  * Use autoconf's values for {bin,lib,include}_dir rather than inventing
+  * In configure.in, quote macro name argument to define() to
+    suppress spurious autoconf error.  (Report from Mihai Ibanescu.)
+  * Use autoconf's values for {bin,lib,include}dir rather than inventing
     our own from @exec_prefix@, making configure --libdir work.
     (Patch from Mihai Ibanescu.)
+  * Remove spurious `_' from {bin,lib,include}dir Makefile variables.
+    (Report from Mihai Ibanescu.)

  --

@@ -5,6 +5,9 @@
     as is required.  (Report from Jim Meyering.)
+  * Use autoconf's values for {bin,lib,include}_dir rather than inventing
+    our own from @exec_prefix@, making configure --libdir work.
+    (Patch from Mihai Ibanescu.)

@@ -3,6 +3,8 @@
     suppress spurious autoconf error.  (Report from Mihai Ibanescu.)
+  * Cast ptrdiff_t to int for %.*s length in adnsheloex and adnslogres,
+    as is required.  (Report from Jim Meyering.)

@@ -1,3 +1,11 @@
+adns (1.3); urgency=low
+
+  Bugfixes:
+  * In configure.in, quote macro name argument to define() to
+    suppress spurious autoconf error.  (Report from Mihai Ibanescu.)
+
+ --
+
 adns (1.2); urgency=medium

   New features:

changelog version finalised, 1.2

update DISTVERSION

update copyright dates

update copyright dates and ADNS_VERSION_STRING

restructured and improved changelog

A few more links for the README.

licence waiver section improved

fix up adnstest for r_unknown; tests for r_unknown

unknown rr types seem to work

spec for adns_r_unknown

SRV supported

SERVFAIL is definitely not conclusive.  Eg, recursive server is supposedly slave for a zone but it has timed out or something - this gives SERVFAIL in BIND8 at least.  But another server (eg not with broken config) may be able to answer the question

semicolon _is_ comment in resolv.conf now

SRV is now supported

regression tests for srv, part 2

regression tests for srv, part 1

remove debugging from srv with postsort

srv with postsort seems to work

revert: put back postsort - the RFC doesn't say that the server does that for us

do away with postsort - the server does that for us

no, do not add new types to adnstest - it breaks regress totally!

fix parsing of SRVs

srv processing written (except for sorting); now to be debugged

further wip on SRV - need per-type functions now including postsort

initial SRV WIP - adns.h

*** empty log message ***

  * adnstest converts some errno values to EFOOBAR: all of the ones
    mentioned in adns.h, at least.  This makes the regression test
    more portable (fixes problem noticed by Bernd Eckenfels).

@@ -5,6 +5,7 @@
     and patch from Nix of esperi.org.uk.)
+  * Update MINOR to 2 because of adns_init_logfn.

@@ -2,6 +2,9 @@
     in README.html.
+  * Create $(bin_dir) and $(lib_dir) on `make install', and also
+    make a libadns.so.1 -> libadns.so.1.<minor> link.  (Suggestions
+    and patch from Nix of esperi.org.uk.)

Include reference to Peter Simons's Haskell bindings

@@ -3,10 +3,12 @@
   * Include reference to Peter Simons's Haskell bindings
     in README.html.

-  Fixes from Bernd Eckenfels, the Debian maintainer:
+  Fixes from or suggested by Bernd Eckenfels, the Debian maintainer:
   * Correct type of various printf arguments: ptrdiff_t != int.
   * Do not print size of leaked blocks of memory (this causes
-    a spurious regression test failure).
+    a spurious regression test failure on some platforms)
+  * Provide adns_if_none and adns_qf_none (which will help with compilers
+    which complain about plain `0' being passed where an enum is wanted).

  --

Do not print size of leaked blocks of memory (this causes a spurious regression test failure on some platforms).

Correct type of various printf arguments: ptrdiff_t != int

@@ -1,3 +1,10 @@
+adns (1.1.999.0.2); urgency=medium
+
+  * Include reference to Peter Simons's Haskell bindings
+    in README.html.
+
+ --
+
 adns (1.1.999.0.1); urgency=medium

   * Fix error in prototype in definition of adns__parse_domain.

finalise

@@ -1,4 +1,4 @@
-adns (1.2); urgency=medium
+adns (1.1.999.0.1); urgency=medium

   * Fix error in prototype in definition of adns__parse_domain.
   * New LICENCE.WAIVERS file for GPL-incompatility workarounds.
@@ -16,6 +16,8 @@
   * Add bind(2) and listen(2) to test harness (for epithet's benefit, but
     harmless in adns).
   * Add missing ENOTSOCK to hcommon.c.m4 (was already in hcommon.c!)
+  * New adns_init_logfn etc. for having logging done with a callback
+    function.

  --

@@ -11,6 +11,11 @@
     updated).
+  * Call Tensurerecordfile in Hgettimeofday (Tensurerecordfile was
+    Tensureinputfile/Tensureoutputfile).
+  * Add bind(2) and listen(2) to test harness (for epithet's benefit, but
+    harmless in adns).
+  * Add missing ENOTSOCK to hcommon.c.m4 (was already in hcommon.c!)

@@ -10,6 +10,7 @@
     updated).
+  * Add -Wno-pointer-sign if GCC has that option.

@@ -9,6 +9,7 @@
     updated).
+  * Add documentation comment by definition of adns_r_ptr_raw type enum.

@@ -7,6 +7,8 @@
+  * New adnsheloex client courtesy of Tony Finch (and copyright notices
+    updated).

@@ -6,6 +6,7 @@
+  * Darwin listed in INSTALL.

@@ -5,6 +5,7 @@
+  * Add .PHONY: install to Makefile, to help people with demented fs's.

@@ -4,6 +4,7 @@
+  * Added a TODO list item.

@@ -3,8 +3,9 @@
   * Fix error in prototype in definition of adns__parse_domain.
   * New LICENCE.WAIVERS file for GPL-incompatility workarounds.
   * Clarified GPL-vs-LGPL: a bit less hostile and a bit more mercenary.
+  * Allow `;'-comments in resolv.conf (report from Colin Charles).

-  --
+ --

 adns (1.1); urgency=medium

semicolon comments request

@@ -1,8 +1,10 @@
 adns (1.2); urgency=medium

   * Fix error in prototype in definition of adns__parse_domain.
+  * New LICENCE.WAIVERS file for GPL-incompatility workarounds.
+  * Clarified GPL-vs-LGPL: a bit less hostile and a bit more mercenary.

- --
+  --

 adns (1.1); urgency=medium

@@ -1,3 +1,9 @@
+adns (1.2); urgency=medium
+
+  * Fix error in prototype in definition of adns__parse_domain.
+
+ --
+
 adns (1.1); urgency=medium

   Major bugfixes:

update version

Releasing 1.1.

categorise changelog

fixes from www.gnu.org

update latest html version

improve wrapping in adns.h

@@ -4,9 +4,12 @@
+  * Fix bogus multiple updates to p in transmit.c (!)
+  * Fix up spurious #undef's in hredirect.h.
     (and there doesn't seem to be much explanation why).
+  * #include <stdlib.h> in internal.h (for abort etc).

@@ -1,6 +1,7 @@
   Bugfixes:
+  * Do not spin if connect() fails immediately (!)