Add helper for checking if cached entry is for a host
authorRafał Miłecki <rafal@milecki.pl>
Tue, 14 Feb 2017 12:31:07 +0000 (13:31 +0100)
committerRafał Miłecki <rafal@milecki.pl>
Tue, 14 Feb 2017 12:31:49 +0000 (13:31 +0100)
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
cache.h
ubus.c

diff --git a/cache.h b/cache.h
index 6c38977b08265d2da74ca95e2d6b9139435e87d3..7830b4fd96f1b06c11f150cebe499f4698a07c6a 100644 (file)
--- a/cache.h
+++ b/cache.h
@@ -54,6 +54,9 @@ void cache_cleanup(struct interface *iface);
 void cache_answer(struct interface *iface, uint8_t *base, int blen,
                  char *name, struct dns_answer *a, uint8_t *rdata, int flush);
 int cache_host_is_known(char *record);
+static inline bool cache_service_is_host(struct cache_service *s) {
+       return *((char *)s->avl.key) != '_';
+}
 void cache_dump_records(struct blob_buf *buf, const char *name);
 void cache_dump_recursive(struct blob_buf *b, const char *name, uint16_t type, struct interface *iface);
 
diff --git a/ubus.c b/ubus.c
index 69db2ac8c34f60aa07a91f08f97477d33ef11a41..a74df6cf5682e9e1f5ab757ea0f39d3bf2da7c2c 100644 (file)
--- a/ubus.c
+++ b/ubus.c
@@ -59,7 +59,7 @@ umdns_browse(struct ubus_context *ctx, struct ubus_object *obj,
        blob_buf_init(&b, 0);
        avl_for_each_element(&services, s, avl) {
                char *local;
-               if (*((char *) s->avl.key) != '_')
+               if (cache_service_is_host(s))
                        continue;
                snprintf(buffer, MAX_NAME_LEN, "%s", (const char *) s->avl.key);
                local = strstr(buffer, ".local");
@@ -103,7 +103,7 @@ umdns_hosts(struct ubus_context *ctx, struct ubus_object *obj,
        blob_buf_init(&b, 0);
        avl_for_each_element(&services, s, avl) {
                char *local;
-               if (*((char *) s->avl.key) == '_')
+               if (!cache_service_is_host(s))
                        continue;
                snprintf(buffer, MAX_NAME_LEN, "%s", (const char *) s->entry);
                local = strstr(buffer, "._");