fix segfault while dumping records

[project/mdnsd.git] / cache.c

diff --git a/cache.c b/cache.c
index 2658bd6..927b9cf 100644 (file)
--- a/cache.c
+++ b/cache.c
@@ -42,7 +42,8 @@
 #include "interface.h"
 
 static struct uloop_timeout cache_gc;
-struct avl_tree records, entries;
+struct avl_tree entries;
+static AVL_TREE(records, avl_strcmp, true, NULL);
 
 static void
 cache_record_free(struct cache_record *r)
@@ -93,7 +94,6 @@ int
 cache_init(void)
 {
        avl_init(&entries, avl_strcmp, true, NULL);
-       avl_init(&records, avl_strcmp, true, NULL);
 
        cache_gc.cb = cache_gc_timer;
        uloop_timeout_set(&cache_gc, 10000);
@@ -226,8 +226,7 @@ cache_answer(struct interface *iface, uint8_t *base, int blen, char *name, struc
        char *name_buf;
        void *rdata_ptr, *txt_ptr;
        int host_len = 0;
-
-       static char rdata_buffer[MAX_DATA_LEN + 1];
+       static char *rdata_buffer = (char *) mdns_buf;
 
        if (!(a->class & CLASS_IN))
                return;
@@ -341,15 +340,12 @@ cache_answer(struct interface *iface, uint8_t *base, int blen, char *name, struc
 void
 cache_dump_records(struct blob_buf *buf, const char *name)
 {
-       struct cache_record *r, *q = avl_find_element(&records, name, r, avl);
+       struct cache_record *r, *last, *next;
        const char *txt;
-       char buffer[MAX_NAME_LEN];
-
-       if (!q)
-               return;
+       char buffer[INET6_ADDRSTRLEN];
 
-       do {
-               r = q;
+       last = avl_last_element(&records, last, avl);
+       for (r = avl_find_element(&records, name, r, avl); r; r = next) {
                switch (r->type) {
                case TYPE_TXT:
                        if (r->txt && strlen(r->txt)) {
@@ -376,6 +372,12 @@ cache_dump_records(struct blob_buf *buf, const char *name)
                                blobmsg_add_string(buf, "ipv6", buffer);
                        break;
                }
-               q = avl_next_element(r, avl);
-       } while (q && !strcmp(r->record, q->record));
+
+               if (r == last)
+                       break;
+
+               next = avl_next_element(r, avl);
+               if (strcmp(r->record, next->record) != 0)
+                       break;
+       }
 }