regress/r1test: Support a `$debug' prefix variable.

Amazingly, sometimes the tests don't pass.  When that happens, it's
handy to be able to debug the library and find out why.

It's a little tricky to attach GDB to the running process directly.  I
used the following:

$ debug="noip gdbserver localhost:666" ./r1test CASE

and then, in another terminal, something like

$ noip gdb adnstest_s
(gdb) target remote localhost:666

to debug the program.

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

src/: Change how query domain names are checked.

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,
so we let the hook store information in the qcontext structure.

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

src/: adns__internal_submit can modify ctx->tinfo.

It will be convenient for type hooks to store information in their part
of the query context before the query is properly established.  We could
have adns__internal_submit make a copy of the caller's context, but it
turns out that none of them actually care if we change it.

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

src/internal.h: Hoist the qcontext definition to before typeinfo.

So that typeinfo functions can have argument types which involve the
qcontext structure.

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

src/types.c: Rearrange the main typeinfo table.

Kill XTRA_TYPE, which won't scale at all well as new features are added
to the typeinfo structure.  Instead, allow additional arguments to
FLAT/DEEP_TYPE, which are used to carry designated initializers for the
more unusual fields.

I took the opportunity to factor out some common prefixes in the names
of the functions listed in the table, which gives us some additional
horizontal space.  This will come in useful later.

All of this involves a little nontrivial macrology, now in internal.h.

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

regress/: Don't track the size of the socket-address buffer in recvfrom(3).

This is system specific, and (more annoyingly) will break the tests if
additional address families are supported in the future.

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

src/: Separate context state into parts for the type itself and its parent.

The `qcontext.info' union has two distinct uses.  A PTR query uses it to
store the initial query address, against which it compares the answer to
its child A query.  On the other hand, `pap_hostaddr' uses this space in
each child query it makes, to keep track of where to put the socket
addresses when the answers come back.

These two kinds of uses are incompatible, so split `qcontext.info' into
two pieces: one for the internal use of a query type, and one for the
use of a child query's parent -- and specifically for its completion
callback.

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

src/types.c: Fix up the table-of-contents comments.

Cheesy but useful sed(1) rune:

sed -n '
  /order of sections/,/\*\//d
  /^ \* _/s///p
  /^static \+[a-z0-9_]\+ \+\**\([0-9a-z_]\+\).*$/s//  \1/p'

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

client/adh-query.c: Surprising comma rather than semicolon.

This doesn't change the meaning of the code, but it's really surprising.

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

src/adns-internal.h: Kill decoy internal header file.

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

src/internal.h: Delete decoy type `rr_align'.

The droid you were looking for is called `union maxalign'.

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

src/types.c: Remove some unused macros.

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

Remove unused variables.

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>

src/setup.c: Insert missing parentheses.

Found from GCC warning.  This code never worked.

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>

.gitignore: Add subdirectories' .cvsignores

 egrep . */.cvsignore | perl -pe 's#/\.cvsignore:#/#' >>.gitignore
 git-rm */.cvsignore

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

.gitignore: Add *.o *.so *.a and *~

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

.gitignore: Rename from .cvsignore

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

CVS to git conversion

Record the scripts etc. we used (and their history) for posterity,
by binding them into the history.

make script's actual tag anyway

merge the script's history into the generated history

Merge remote-tracking branch 'mdw/mdw/convert-script'

"iwj's original" 8298cc36e3a7eef91f722e7115ea8b5a9366fb55
is actually identical to 8e04e8e83b27a61d1471c81de5f8aabe29dda30a
apart from the .gitignore

Conflicts:
convert

fix

Diff the changelog by hand if the commit message is `empty'.

Wrap long single-line comment messages.

Avoid printing removed lines unless necessary; avoid version headers.

Use `git mumble' consistently in place of `git-mumble'.

Newer versions of Git don't leave the implementations on the PATH.

iwj's original.

script is in new location

merge rather than abandon pollfds

move graftmergein (nfc)

tidy up a bit

found

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

+  Improvements for multithreaded programs:

@@ -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

+  * Do away with `mismatch' variable in parse.c:adns__findrr_anychk so that

@@ -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.)

+  Portability fixes:

@@ -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.)

  --

+  * Use autoconf's values for {bin,lib,include}_dir rather than inventing

@@ -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.)

+  * Cast ptrdiff_t to int for %.*s length in adnsheloex and adnslogres,

@@ -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.)

+  Bugfixes:

@@ -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

+  * Document in adns.h EINVAL from adns_init meaning bad configuration.

@@ -9,6 +9,7 @@ adns (1.1.999.0.2); urgency=medium
   * 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).
+  * Document in adns.h EINVAL from adns_init meaning bad configuration.

   Fixes from or suggested by Bernd Eckenfels, the Debian maintainer:
   * Correct type of various printf arguments: ptrdiff_t != int.

  * 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).

+  * Update MINOR to 2 because of adns_init_logfn.

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

+  * Create $(bin_dir) and $(lib_dir) on `make install', and also

@@ -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

+  Fixes from or suggested by Bernd Eckenfels, the Debian maintainer:

@@ -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

+  * Include reference to Peter Simons's Haskell bindings

@@ -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

+  * New adns_init_logfn etc. for having logging done with a callback

@@ -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.

  --

branch is closed

+  * Call Tensurerecordfile in Hgettimeofday (Tensurerecordfile was

@@ -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!)

+  * Add -Wno-pointer-sign if GCC has that option.

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

+  * Add documentation comment by definition of adns_r_ptr_raw type enum.

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

+  * New adnsheloex client courtesy of Tony Finch (and copyright notices

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

+  * Darwin listed in INSTALL.

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

+  * Add .PHONY: install to Makefile, to help people with demented fs's.

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

+  * Added a TODO list item.

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

document log message format

document the fact that you have to reassemble the messages

+  * New adns_init_logfn etc. for having logging done with a callback

@@ -1,3 +1,10 @@
+adns (1.3); urgency=medium
+
+  * New adns_init_logfn etc. for having logging done with a callback
+    function.
+
+ --
+
 adns (1.2); urgency=medium

   * Fix error in prototype in definition of adns__parse_domain.

+  * Allow `;'-comments in resolv.conf (report from Colin Charles).

@@ -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

+  * New LICENCE.WAIVERS file for GPL-incompatility workarounds.

@@ -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

+  * Fix error in prototype in definition of adns__parse_domain.

@@ -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.