gethostbyname errata

[Ted Unangst]

Due to a bug in the libc asr resolver, querying an invalid hostname can
cause a crash. Patches are available for 5.5 and 5.6.

untrusted comment: signature from openbsd 5.6 base private key
RWR0EANmo9nqhl31oIXbJYtUWXNHHNzHGhJ+v2XZAAlwH5TwYDkTp2NHqjhnrJayp37glapQejDsm/LDGm1M5bnpkmHh7FGNGQ4=

OpenBSD 5.6 errata 8, Nov 17, 2014:  Querying an invalid hostname with
gethostbyname(3) could cause a NULL deref.

Apply patch using:

    signify -Vep /etc/signify/openbsd-56-base.pub -x 008_asr.patch.sig \
        -m - | (cd /usr/src && patch -p0)

Then build and install libc

    cd /usr/src/lib/libc
    make obj
    make depend
    make
    make install

Also recompile any statically-linked binaries:

    cd /usr/src/bin
    make obj
    make depend
    make
    make install

    cd /usr/src/sbin
    make obj
    make depend
    make
    make install

Index: lib/libc/asr/gethostnamadr_async.c
===================================================================
RCS file: /cvs/src/lib/libc/asr/gethostnamadr_async.c,v
retrieving revision 1.30
diff -u -p -r1.30 gethostnamadr_async.c
--- lib/libc/asr/gethostnamadr_async.c  23 Jul 2014 21:26:25 -0000      1.30
+++ lib/libc/asr/gethostnamadr_async.c  6 Nov 2014 13:00:40 -0000
@@ -357,13 +357,12 @@ gethostnamadr_async_run(struct asr_query
                }
 
                /*
-                * No address found in the dns packet. The blocking version
-                * reports this as an error.
+                * No valid hostname or address found in the dns packet.
+                * Ignore it.
                 */
                if ((as->as_type == ASR_GETHOSTBYNAME &&
                     h->h.h_addr_list[0] == NULL) ||
-                   (as->as_type == ASR_GETHOSTBYADDR &&
-                    h->h.h_name == NULL)) {
+                   h->h.h_name == NULL) {
                        free(h);
                        async_set_state(as, ASR_STATE_NEXT_DB);
                        break;
Index: lib/libc/asr/getnetnamadr_async.c
===================================================================
RCS file: /cvs/src/lib/libc/asr/getnetnamadr_async.c,v
retrieving revision 1.16
diff -u -p -r1.16 getnetnamadr_async.c
--- lib/libc/asr/getnetnamadr_async.c   23 Jul 2014 21:26:25 -0000      1.16
+++ lib/libc/asr/getnetnamadr_async.c   6 Nov 2014 13:00:40 -0000
@@ -239,11 +239,11 @@ getnetnamadr_async_run(struct asr_query 
                        n->n.n_net = as->as.netnamadr.addr;
 
                /*
-                * No address found in the dns packet. The blocking version
-                * reports this as an error.
+                * No valid hostname or address found in the dns packet.
+                * Ignore it.
                 */
-               if (as->as_type == ASR_GETNETBYNAME && n->n.n_net == 0) {
-                        /* XXX wrong */
+               if ((as->as_type == ASR_GETNETBYNAME && n->n.n_net == 0) ||
+                   n->n.n_name == NULL) {
                        free(n);
                        async_set_state(as, ASR_STATE_NEXT_DB);
                        break;