1679. [bug] When there was a single nameserver with multiple

                        addresses for a zone not all addresses were tried.
                        [RT #11706]

diff --git a/CHANGES b/CHANGES
index d57c34d05ca82dd4e727e03a50461b5c02f952c2..e90fa11e1e99a6befe4709ac86a64cb723c77563 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,3 +1,7 @@
+1679.  [bug]           When there was a single nameserver with multiple
+                       addresses for a zone not all addresses were tried.
+                       [RT #11706]
+
 1678.  [bug]           RRSIG should use TYPEXXXXX for unknown types.
 
 1677.  [bug]           dig: +aaonly didn't work, +aaflag undocumented.

diff --git a/lib/dns/resolver.c b/lib/dns/resolver.c
index a24e7b36940b3c7ea08d241c8e52268d4d55657b..475e41978ca9b73fd74a1af01565001ecbb0fd34 100644 (file)
--- a/lib/dns/resolver.c
+++ b/lib/dns/resolver.c
@@ -15,7 +15,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-/* $Id: resolver.c,v 1.218.2.18.4.38 2004/06/07 03:24:57 marka Exp $ */
+/* $Id: resolver.c,v 1.218.2.18.4.39 2004/06/25 04:41:24 marka Exp $ */
 
 #include <config.h>
 
@@ -2102,23 +2102,25 @@ fctx_nextaddress(fetchctx_t *fctx) {
         * Find the first unmarked addrinfo.
         */
        addrinfo = NULL;
-       while (find != fctx->find) {
-               for (addrinfo = ISC_LIST_HEAD(find->list);
-                    addrinfo != NULL;
-                    addrinfo = ISC_LIST_NEXT(addrinfo, publink)) {
-                       if (!UNMARKED(addrinfo))
-                               continue;
-                       possibly_mark(fctx, addrinfo);
-                       if (UNMARKED(addrinfo)) {
-                               addrinfo->flags |= FCTX_ADDRINFO_MARK;
-                               break;
+       if (find != NULL) {
+               do {
+                       for (addrinfo = ISC_LIST_HEAD(find->list);
+                            addrinfo != NULL;
+                            addrinfo = ISC_LIST_NEXT(addrinfo, publink)) {
+                               if (!UNMARKED(addrinfo))
+                                       continue;
+                               possibly_mark(fctx, addrinfo);
+                               if (UNMARKED(addrinfo)) {
+                                       addrinfo->flags |= FCTX_ADDRINFO_MARK;
+                                       break;
+                               }
                        }
-               }
-               if (addrinfo != NULL)
-                       break;
-               find = ISC_LIST_NEXT(find, publink);
-               if (find != fctx->find && find == NULL)
-                       find = ISC_LIST_HEAD(fctx->finds);
+                       if (addrinfo != NULL)
+                               break;
+                       find = ISC_LIST_NEXT(find, publink);
+                       if (find != fctx->find && find == NULL)
+                               find = ISC_LIST_HEAD(fctx->finds);
+               } while (find != fctx->find);
        }
 
        fctx->find = find;
@@ -2144,23 +2146,25 @@ fctx_nextaddress(fetchctx_t *fctx) {
         * Find the first unmarked addrinfo.
         */
        addrinfo = NULL;
-       while (find != fctx->altfind) {
-               for (addrinfo = ISC_LIST_HEAD(find->list);
-                    addrinfo != NULL;
-                    addrinfo = ISC_LIST_NEXT(addrinfo, publink)) {
-                       if (!UNMARKED(addrinfo))
-                               continue;
-                       possibly_mark(fctx, addrinfo);
-                       if (UNMARKED(addrinfo)) {
-                               addrinfo->flags |= FCTX_ADDRINFO_MARK;
-                               break;
+       if (find != NULL) {
+               do {
+                       for (addrinfo = ISC_LIST_HEAD(find->list);
+                            addrinfo != NULL;
+                            addrinfo = ISC_LIST_NEXT(addrinfo, publink)) {
+                               if (!UNMARKED(addrinfo))
+                                       continue;
+                               possibly_mark(fctx, addrinfo);
+                               if (UNMARKED(addrinfo)) {
+                                       addrinfo->flags |= FCTX_ADDRINFO_MARK;
+                                       break;
+                               }
                        }
-               }
-               if (addrinfo != NULL)
-                       break;
-               find = ISC_LIST_NEXT(find, publink);
-               if (find != fctx->altfind && find == NULL)
-                       find = ISC_LIST_HEAD(fctx->altfinds);
+                       if (addrinfo != NULL)
+                               break;
+                       find = ISC_LIST_NEXT(find, publink);
+                       if (find != fctx->altfind && find == NULL)
+                               find = ISC_LIST_HEAD(fctx->altfinds);
+               } while (find != fctx->altfind);
        }
 
        faddrinfo = addrinfo;