From: Lennart Poettering <lennart@poettering.net>
Date: Wed, 30 Jul 2014 15:52:58 +0000 (+0200)
Subject: resolved: properly pass empty answers back to bus clients
X-Git-Tag: v216~362
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=3339cb71d44c5198f9546f113674f06dc7b01a6f;hp=79266746741b42da878663faddd1e570f8f38e74

resolved: properly pass empty answers back to bus clients
---

diff --git a/src/resolve/resolved-bus.c b/src/resolve/resolved-bus.c
index 1244b6dd0..d0c21894e 100644
--- a/src/resolve/resolved-bus.c
+++ b/src/resolve/resolved-bus.c
@@ -149,33 +149,36 @@ static void bus_method_resolve_hostname_complete(DnsQuery *q) {
         if (r < 0)
                 goto finish;
 
-        answer = dns_answer_ref(q->answer);
         ifindex = q->answer_ifindex;
 
-        for (i = 0; i < answer->n_rrs; i++) {
-                r = dns_question_matches_rr(q->question, answer->rrs[i]);
-                if (r < 0)
-                        goto parse_fail;
-                if (r == 0) {
-                        /* Hmm, if this is not an address record,
-                           maybe it's a cname? If so, remember this */
-                        r = dns_question_matches_cname(q->question, answer->rrs[i]);
+        if (q->answer) {
+                answer = dns_answer_ref(q->answer);
+
+                for (i = 0; i < answer->n_rrs; i++) {
+                        r = dns_question_matches_rr(q->question, answer->rrs[i]);
                         if (r < 0)
                                 goto parse_fail;
-                        if (r > 0)
-                                cname = dns_resource_record_ref(answer->rrs[i]);
+                        if (r == 0) {
+                                /* Hmm, if this is not an address record,
+                                   maybe it's a cname? If so, remember this */
+                                r = dns_question_matches_cname(q->question, answer->rrs[i]);
+                                if (r < 0)
+                                        goto parse_fail;
+                                if (r > 0)
+                                        cname = dns_resource_record_ref(answer->rrs[i]);
 
-                        continue;
-                }
+                                continue;
+                        }
 
-                r = append_address(reply, answer->rrs[i], ifindex);
-                if (r < 0)
-                        goto finish;
+                        r = append_address(reply, answer->rrs[i], ifindex);
+                        if (r < 0)
+                                goto finish;
 
-                if (!canonical)
-                        canonical = dns_resource_record_ref(answer->rrs[i]);
+                        if (!canonical)
+                                canonical = dns_resource_record_ref(answer->rrs[i]);
 
-                added ++;
+                        added ++;
+                }
         }
 
         if (added <= 0) {
@@ -347,20 +350,22 @@ static void bus_method_resolve_address_complete(DnsQuery *q) {
         if (r < 0)
                 goto finish;
 
-        answer = dns_answer_ref(q->answer);
+        if (q->answer) {
+                answer = dns_answer_ref(q->answer);
 
-        for (i = 0; i < answer->n_rrs; i++) {
-                r = dns_question_matches_rr(q->question, answer->rrs[i]);
-                if (r < 0)
-                        goto parse_fail;
-                if (r == 0)
-                        continue;
+                for (i = 0; i < answer->n_rrs; i++) {
+                        r = dns_question_matches_rr(q->question, answer->rrs[i]);
+                        if (r < 0)
+                                goto parse_fail;
+                        if (r == 0)
+                                continue;
 
-                r = sd_bus_message_append(reply, "s", answer->rrs[i]->ptr.name);
-                if (r < 0)
-                        goto finish;
+                        r = sd_bus_message_append(reply, "s", answer->rrs[i]->ptr.name);
+                        if (r < 0)
+                                goto finish;
 
-                added ++;
+                        added ++;
+                }
         }
 
         if (added <= 0) {