2922. [bug] 'dig +trace' could drop core after "connection

                        timeout". [RT #21514]

diff --git a/CHANGES b/CHANGES
index e9d560f8f58b34c065a5f416d505fec6b6a696f1..cd7fbea0b2afcb8dfb06c3497971e3dd4cdb6d89 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,3 +1,6 @@
+2922.  [bug]           'dig +trace' could drop core after "connection
+                       timeout". [RT #21514]
+
 2922.  [contrib]       Update zkt to version 1.0.
 
 2921.  [bug]           The resolver could attempt to destroy a fetch context

diff --git a/bin/dig/dighost.c b/bin/dig/dighost.c
index 1c206b3fc54772e92c396f10f31f6468831195a0..85ad056efaeaeea990961fd80b6480fe66d35bd4 100644 (file)
--- a/bin/dig/dighost.c
+++ b/bin/dig/dighost.c
@@ -15,7 +15,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-/* $Id: dighost.c,v 1.330 2010/05/18 02:38:10 tbox Exp $ */
+/* $Id: dighost.c,v 1.331 2010/06/24 07:22:18 marka Exp $ */
 
 /*! \file
  *  \note
@@ -2401,6 +2401,15 @@ force_timeout(dig_lookup_t *l, dig_query_t *query) {
                      isc_result_totext(ISC_R_NOMEMORY));
        }
        isc_task_send(global_task, &event);
+
+       /*
+        * The timer may have expired if, for example, get_address() takes
+        * long time and the timer was running on a different thread.
+        * We need to cancel the possible timeout event not to confuse
+        * ourselves due to the duplicate events.
+        */
+       if (l->timer != NULL)
+               isc_timer_detach(&l->timer);
 }
 
 
@@ -2424,7 +2433,7 @@ send_tcp_connect(dig_query_t *query) {
        query->waiting_connect = ISC_TRUE;
        query->lookup->current_query = query;
        result = get_address(query->servname, port, &query->sockaddr);
-       if (result == ISC_R_NOTFOUND) {
+       if (result != ISC_R_SUCCESS) {
                /*
                 * This servname doesn't have an address.  Try the next server
                 * by triggering an immediate 'timeout' (we lie, but the effect
@@ -2506,7 +2515,7 @@ send_udp(dig_query_t *query) {
                /* XXX Check the sense of this, need assertion? */
                query->waiting_connect = ISC_FALSE;
                result = get_address(query->servname, port, &query->sockaddr);
-               if (result == ISC_R_NOTFOUND) {
+               if (result != ISC_R_SUCCESS) {
                        /* This servname doesn't have an address. */
                        force_timeout(l, query);
                        return;