Changeset 120eedd in code for dns.c

Timestamp:

Jun 17, 2014, 11:46:35 AM (11 years ago)

Author:

Mike Belopuhov <mike@…>

Branches:

master

Children:

e68221b

Parents:

9440414

Message:

prevent icbd_drop from freeing an icb session while dns is in progress

File:

• dns.c (modified) (4 diffs)

dns.c

@@ -56 +56 @@
                 icbd_log(is, LOG_WARNING, "dns resolution failed: %s",
                     gai_strerror(ar->ar_gai_errno));
+
+        if (ISSETF(is->flags, ICB_SF_PENDINGDROP)) {
+                free(is);
+                return;
+        }
+
+        CLRF(is->flags, ICB_SF_DNSINPROGRESS);
 }
 
@@ -65 +72 @@
         struct addrinfo hints;
 
+        if (ISSETF(is->flags, ICB_SF_PENDINGDROP)) {
+                free(is);
+                return;
+        }
+
         if (ar->ar_gai_errno == 0) {
                 icbd_log(is, LOG_DEBUG, "reverse dns resolved %s to %s",
@@ -73 +85 @@
                 as = getaddrinfo_async(is->hostname, NULL, &hints, NULL);
                 event_asr_run(as, dns_done_host, is);
-        } else
+        } else {
                 icbd_log(is, LOG_WARNING, "reverse dns resolution failed: %s",
                     gai_strerror(ar->ar_gai_errno));
+                CLRF(is->flags, ICB_SF_DNSINPROGRESS);
+        }
 }
 
@@ -86 +100 @@
                 return;
 
+        SETF(is->flags, ICB_SF_DNSINPROGRESS);
+
         if (verbose)
                 icbd_log(is, LOG_DEBUG, "resolving: %s", is->host);