From f6712902a92eb82b6c8d7e7fd0980a20a716fa0d Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Thu, 27 Oct 2005 19:11:34 +0000 Subject: * format unknown DNS records according to RFC3597 * add some DNS class and type validity checks to avahi_server_add() * move DNS type and class definitions to avahi-common/defs.h git-svn-id: file:///home/lennart/svn/public/avahi/trunk@896 941a03a8-eaeb-0310-b9a0-b1bbd8fe43fe --- avahi-core/dns-test.c | 16 ++++++++++++- avahi-core/dns.c | 1 + avahi-core/entry.c | 9 ++++++++ avahi-core/rr.c | 63 +++++++++++++++++++++++++++++++++++---------------- avahi-core/rr.h | 28 +++++++---------------- 5 files changed, 76 insertions(+), 41 deletions(-) (limited to 'avahi-core') diff --git a/avahi-core/dns-test.c b/avahi-core/dns-test.c index 5634ca5..8a7586c 100644 --- a/avahi-core/dns-test.c +++ b/avahi-core/dns-test.c @@ -29,6 +29,7 @@ #include #include +#include #include #include "dns.h" @@ -36,7 +37,7 @@ #include "util.h" int main(AVAHI_GCC_UNUSED int argc, AVAHI_GCC_UNUSED char *argv[]) { - char t[AVAHI_DOMAIN_NAME_MAX]; + char t[AVAHI_DOMAIN_NAME_MAX], *m; const char *a, *b, *c, *d; AvahiDnsPacket *p; AvahiRecord *r, *r2; @@ -95,6 +96,19 @@ int main(AVAHI_GCC_UNUSED int argc, AVAHI_GCC_UNUSED char *argv[]) { /* Free the records */ avahi_record_unref(r); avahi_record_unref(r2); + + r = avahi_record_new_full("foobar", 77, 77, AVAHI_DEFAULT_TTL); + assert(r); + + assert(r->data.generic.data = avahi_memdup("HALLO", r->data.generic.size = 5)); + + m = avahi_record_to_string(r); + assert(m); + + avahi_log_debug(">%s<", m); + + avahi_free(m); + avahi_record_unref(r); return 0; } diff --git a/avahi-core/dns.c b/avahi-core/dns.c index d237d55..b91ccd0 100644 --- a/avahi-core/dns.c +++ b/avahi-core/dns.c @@ -30,6 +30,7 @@ #include #include +#include #include #include diff --git a/avahi-core/entry.c b/avahi-core/entry.c index fd0e233..3621fe3 100644 --- a/avahi-core/entry.c +++ b/avahi-core/entry.c @@ -176,6 +176,15 @@ static AvahiEntry * server_add_internal( AVAHI_CHECK_VALIDITY_RETURN_NULL(s, r->ttl != 0, AVAHI_ERR_INVALID_TTL); AVAHI_CHECK_VALIDITY_RETURN_NULL(s, !avahi_key_is_pattern(r->key), AVAHI_ERR_IS_PATTERN); AVAHI_CHECK_VALIDITY_RETURN_NULL(s, avahi_record_is_valid(r), AVAHI_ERR_INVALID_RECORD); + AVAHI_CHECK_VALIDITY_RETURN_NULL(s, r->key->clazz == AVAHI_DNS_CLASS_IN, AVAHI_ERR_INVALID_DNS_CLASS); + AVAHI_CHECK_VALIDITY_RETURN_NULL(s, + (r->key->type != 0) && + (r->key->type != AVAHI_DNS_TYPE_ANY) && + (r->key->type != AVAHI_DNS_TYPE_OPT) && + (r->key->type != AVAHI_DNS_TYPE_TKEY) && + (r->key->type != AVAHI_DNS_TYPE_TSIG) && + (r->key->type != AVAHI_DNS_TYPE_IXFR) && + (r->key->type != AVAHI_DNS_TYPE_AXFR), AVAHI_ERR_INVALID_DNS_TYPE); if (flags & AVAHI_PUBLISH_UPDATE) { AvahiRecord *old_record; diff --git a/avahi-core/rr.c b/avahi-core/rr.c index e88d3c6..57ddca4 100644 --- a/avahi-core/rr.c +++ b/avahi-core/rr.c @@ -32,6 +32,7 @@ #include #include +#include #include "rr.h" #include "log.h" @@ -224,18 +225,30 @@ const char *avahi_dns_type_to_string(uint16_t type) { } char *avahi_key_to_string(const AvahiKey *k) { + char class[16], type[16]; + const char *c, *t; + assert(k); assert(k->ref >= 1); + + /* According to RFC3597 */ + + if (!(c = avahi_dns_class_to_string(k->clazz))) { + snprintf(class, sizeof(class), "CLASS%u", k->clazz); + c = class; + } + + if (!(t = avahi_dns_type_to_string(k->type))) { + snprintf(type, sizeof(type), "TYPE%u", k->type); + t = type; + } - return avahi_strdup_printf("%s\t%s\t%s", - k->name, - avahi_dns_class_to_string(k->clazz), - avahi_dns_type_to_string(k->type)); + return avahi_strdup_printf("%s\t%s\t%s", k->name, c, t); } char *avahi_record_to_string(const AvahiRecord *r) { char *p, *s; - char buf[257], *t = NULL, *d = NULL; + char buf[1024], *t = NULL, *d = NULL; assert(r); assert(r->ref >= 1); @@ -274,10 +287,34 @@ char *avahi_record_to_string(const AvahiRecord *r) { r->data.srv.name); break; + + default: { + + uint8_t *c; + uint16_t n; + int i; + char *e; + + /* According to RFC3597 */ + + snprintf(t = buf, sizeof(buf), "\\# %u", r->data.generic.size); + + e = strchr(t, 0); + + for (c = r->data.generic.data, n = r->data.generic.size, i = 0; + n > 0 && i < 20; + c ++, n --) { + + sprintf(e, " %02X", *c); + e = strchr(e, 0); + } + + break; + } } p = avahi_key_to_string(r->key); - s = avahi_strdup_printf("%s %s ; ttl=%u", p, t ? t : "", r->ttl); + s = avahi_strdup_printf("%s %s ; ttl=%u", p, t, r->ttl); avahi_free(p); avahi_free(d); @@ -291,8 +328,6 @@ int avahi_key_equal(const AvahiKey *a, const AvahiKey *b) { if (a == b) return 1; -/* g_message("equal: %p %p", a, b); */ - return avahi_domain_equal(a->name, b->name) && a->type == b->type && a->clazz == b->clazz; @@ -302,8 +337,6 @@ int avahi_key_pattern_match(const AvahiKey *pattern, const AvahiKey *k) { assert(pattern); assert(k); -/* g_message("equal: %p %p", a, b); */ - assert(!avahi_key_is_pattern(k)); if (pattern == k) @@ -336,15 +369,6 @@ static int rdata_equal(const AvahiRecord *a, const AvahiRecord *b) { assert(b); assert(a->key->type == b->key->type); -/* t = avahi_record_to_string(a); */ -/* g_message("comparing %s", t); */ -/* avahi_free(t); */ - -/* t = avahi_record_to_string(b); */ -/* g_message("and %s", t); */ -/* avahi_free(t); */ - - switch (a->key->type) { case AVAHI_DNS_TYPE_SRV: return @@ -661,7 +685,6 @@ int avahi_record_is_valid(AvahiRecord *r) { return strlen(r->data.hinfo.os) <= 255 && strlen(r->data.hinfo.cpu) <= 255; - case AVAHI_DNS_TYPE_TXT: { diff --git a/avahi-core/rr.h b/avahi-core/rr.h index a2d430a..6bfe0ec 100644 --- a/avahi-core/rr.h +++ b/avahi-core/rr.h @@ -33,35 +33,23 @@ AVAHI_C_DECL_BEGIN -/** DNS record types, see RFC 1035 */ +/** DNS record types, see RFC 1035, in addition to those defined in defs.h */ enum { - AVAHI_DNS_TYPE_A = 0x01, - AVAHI_DNS_TYPE_NS = 0x02, - AVAHI_DNS_TYPE_CNAME = 0x05, - AVAHI_DNS_TYPE_SOA = 0x06, - AVAHI_DNS_TYPE_PTR = 0x0C, - AVAHI_DNS_TYPE_HINFO = 0x0D, - AVAHI_DNS_TYPE_MX = 0x0F, - AVAHI_DNS_TYPE_TXT = 0x10, - AVAHI_DNS_TYPE_AAAA = 0x1C, - AVAHI_DNS_TYPE_SRV = 0x21, - AVAHI_DNS_TYPE_ANY = 0xFF /**< Special query type for requesting all records */ + AVAHI_DNS_TYPE_ANY = 0xFF, /**< Special query type for requesting all records */ + AVAHI_DNS_TYPE_OPT = 41, /**< EDNS0 option */ + AVAHI_DNS_TYPE_TKEY = 249, + AVAHI_DNS_TYPE_TSIG = 250, + AVAHI_DNS_TYPE_IXFR = 251, + AVAHI_DNS_TYPE_AXFR = 252 }; -/** DNS record classes, see RFC 1035 */ +/** DNS record classes, see RFC 1035, in addition to those defined in defs.h */ enum { - AVAHI_DNS_CLASS_IN = 0x01, /**< Probably the only class we will ever use */ AVAHI_DNS_CLASS_ANY = 0xFF, /**< Special query type for requesting all records */ AVAHI_DNS_CACHE_FLUSH = 0x8000, /**< Not really a class but a bit which may be set in response packets, see mDNS spec for more information */ AVAHI_DNS_UNICAST_RESPONSE = 0x8000 /**< Not really a class but a bit which may be set in query packets, see mDNS spec for more information */ }; -/** The default TTL for RRs which contain a host name of some kind. */ -#define AVAHI_DEFAULT_TTL_HOST_NAME (120) - -/** The default TTL for all other records. */ -#define AVAHI_DEFAULT_TTL (75*60) - /** Encapsulates a DNS query key consisting of class, type and name. Use avahi_key_ref()/avahi_key_unref() for manipulating the reference counter. The structure is intended to be treated as "immutable", no -- cgit