DNS: Fix doxygen comments.
authorRichard Mudgett <rmudgett@digium.com>
Fri, 5 Jun 2015 16:41:54 +0000 (11:41 -0500)
committerRichard Mudgett <rmudgett@digium.com>
Wed, 10 Jun 2015 17:17:02 +0000 (12:17 -0500)
Change-Id: Icafea3fb4ea64ac027561b23cbfe2b17997dc549

include/asterisk/dns_internal.h
main/dns_query_set.c

index be8794b..d517748 100644 (file)
@@ -161,7 +161,7 @@ struct ast_dns_query_recurring {
 struct dns_query_set_query {
        /*! \brief Whether the query started successfully or not */
        unsigned int started;
-       /*! \brief THe query itself */
+       /*! \brief The query itself */
        struct ast_dns_query *query;
 };
 
@@ -169,7 +169,7 @@ struct dns_query_set_query {
 struct ast_dns_query_set {
        /*! \brief DNS queries */
        AST_VECTOR(, struct dns_query_set_query) queries;
-       /* \brief Whether the query set is in progress or not */
+       /*! \brief Whether the query set is in progress or not */
        int in_progress;
        /*! \brief The total number of completed queries */
        int queries_completed;
index 8dfc5ea..147c737 100644 (file)
@@ -117,6 +117,14 @@ int ast_dns_query_set_add(struct ast_dns_query_set *query_set, const char *name,
                return -1;
        }
 
+       /*
+        * We are intentionally passing NULL for the user data even
+        * though dns_query_set_callback() is not NULL tolerant.  Doing
+        * this avoids a circular reference chain until the queries are
+        * started.  ast_dns_query_set_resolve_async() will set the
+        * query user_data for us later when we actually kick off the
+        * queries.
+        */
        query.query = dns_query_alloc(name, rr_type, rr_class, dns_query_set_callback, NULL);
        if (!query.query) {
                return -1;