Re: [PATCH] Fix DNS error (timeout) when non-answer AAAA response arrives before CNAME

Ping!  Have you had time to glance at this patch yet?  I'm feeling a
bit left out :-).

On 9/30/09, Alan Jenkins <[email protected]> wrote:
> Symptoms:
>     Host www.kk.org has both AAAA and CNAME -- ignoring CNAME.
>     Host www.kk.org has both AAAA and CNAME -- ignoring CNAME.
>     Host www.kk.org has both AAAA and CNAME -- ignoring CNAME.
>     Host www.kk.org lookup failed: Timeout (131072).

>     The fix itself only affects a single if statement.  The remaining
>     bulk of the patch is needed to correct the same issue in an
>     assertion further down.

> --- a/dns.c
> +++ b/dns.c
> @@ -1181,7 +1181,8 @@ dnsReplyHandler(int abort, FdEventHandlerPtr event)
>          } else
>              releaseAtom(value);
>      } else if(af == 0) {
> -        if(query->inet4 || query->inet6) {
> +        if((query->inet4 && query->inet4->length > 0) ||
> +           (query->inet6 && query->inet6->length > 0)) {

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference