X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=src%2Freply.c;h=4b3d74fa6b5ad6427f3f064da00c0ee0ceb5220f;hb=94be415aea9fb1e252a9a51af34dce5bdb7d115a;hp=ab7c8ff4aadbc37c6cdb589baf43b49aa384cc88;hpb=660d7d3b4af029c3e86fdfe1ac078815c41ed79e;p=adns.git diff --git a/src/reply.c b/src/reply.c index ab7c8ff..4b3d74f 100644 --- a/src/reply.c +++ b/src/reply.c @@ -3,7 +3,12 @@ * - main handling and parsing routine for received datagrams */ /* - * This file is part of adns, which is Copyright (C) 1997-1999 Ian Jackson + * This file is + * Copyright (C) 1997-1999 Ian Jackson + * + * It is part of adns, which is + * Copyright (C) 1997-1999 Ian Jackson + * Copyright (C) 1999 Tony Finch * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -21,12 +26,11 @@ */ #include -#include #include "internal.h" void adns__procdgram(adns_state ads, const byte *dgram, int dglen, - int serv, struct timeval now) { + int serv, int viatcp, struct timeval now) { int cbyte, rrstart, wantedrrs, rri, foundsoa, foundns, cname_here; int id, f1, f2, qdcount, ancount, nscount, arcount; int flg_ra, flg_rd, flg_tc, flg_qr, opcode; @@ -89,6 +93,12 @@ void adns__procdgram(adns_state ads, const byte *dgram, int dglen, dgram+DNS_HDRSIZE, qu->query_dglen-DNS_HDRSIZE)) continue; + if (viatcp) { + if (qu->state != query_tcpsent) continue; + } else { + if (qu->state != query_tosend) continue; + if (!(qu->udpsent & (1<flags & adns_qf_cname_forbid) { adns__query_fail(qu,adns_s_prohibitedcname); return; - } else if (!qu->cname_dgram) { /* Ignore second and subsequent CNAMEs */ + } else if (qu->cname_dgram) { /* Ignore second and subsequent CNAME(s) */ + adns__debug(ads,serv,qu,"allegedly canonical name %s is actually alias for %s", + qu->answer->cname, + adns__diag_domain(ads,serv,qu, &qu->vb, dgram,dglen,rdstart)); + adns__query_fail(qu,adns_s_prohibitedcname); + return; + } else if (wantedrrs) { /* Ignore CNAME(s) after RR(s). */ + adns__debug(ads,serv,qu,"ignoring CNAME (to %s) coexisting with RR", + adns__diag_domain(ads,serv,qu, &qu->vb, dgram,dglen,rdstart)); + } else { qu->cname_begin= rdstart; qu->cname_dglen= dglen; st= adns__parse_domain(ads,serv,qu, &qu->vb, @@ -170,7 +189,7 @@ void adns__procdgram(adns_state ads, const byte *dgram, int dglen, if (!qu->vb.used) goto x_truncated; if (st) { adns__query_fail(qu,st); return; } l= strlen(qu->vb.buf)+1; - qu->answer->cname= adns__alloc_interim(qu,l); + qu->answer->cname= adns__alloc_preserved(qu,l); if (!qu->answer->cname) { adns__query_fail(qu,adns_s_nomemory); return; } qu->cname_dgram= adns__alloc_mine(qu,dglen); @@ -184,10 +203,6 @@ void adns__procdgram(adns_state ads, const byte *dgram, int dglen, * TCP. If there is no truncation we can use the whole answer if * it contains the relevant info. */ - } else { - adns__debug(ads,serv,qu,"ignoring duplicate CNAME (%s, as well as %s)", - adns__diag_domain(ads,serv,qu, &qu->vb, dgram,dglen,rdstart), - qu->answer->cname); } } else if (rrtype == (qu->typei->type & adns__rrt_typemask)) { wantedrrs++; @@ -307,7 +322,6 @@ void adns__procdgram(adns_state ads, const byte *dgram, int dglen, LIST_LINK_TAIL(ads->childw,qu); return; } - adns__query_done(qu); return; @@ -321,7 +335,6 @@ void adns__procdgram(adns_state ads, const byte *dgram, int dglen, qu->flags |= adns_qf_usevc; x_restartquery: - if (qu->cname_dgram) { st= adns__mkquery_frdgram(qu->ads,&qu->vb,&qu->id, qu->cname_dgram, qu->cname_dglen, qu->cname_begin, @@ -336,6 +349,7 @@ void adns__procdgram(adns_state ads, const byte *dgram, int dglen, memcpy(newquery,qu->vb.buf,qu->vb.used); } - adns__reset_cnameonly(qu); - adns__query_udp(qu,now); + if (qu->state == query_tcpsent) qu->state= query_tosend; + adns__reset_preserved(qu); + adns__query_send(qu,now); }