s4-dns: Fix the comments about ignoring zones in internal server
authorMatthieu Patou <mat@matws.net>
Sat, 13 Oct 2012 08:36:06 +0000 (01:36 -0700)
committerKai Blin <kai@samba.org>
Sat, 13 Oct 2012 10:37:53 +0000 (12:37 +0200)
Acked-By: Kai Blin <kai@samba.org>
Autobuild-User(master): Kai Blin <kai@samba.org>
Autobuild-Date(master): Sat Oct 13 12:37:53 CEST 2012 on sn-devel-104

source4/dns_server/dns_server.c

index d139eb7e91514fb22a8896768b99382ddafebb4f..8e2539664c1b4b4e19204b2ff8a9d740eececf82 100644 (file)
@@ -832,7 +832,13 @@ static void dns_task_init(struct task_server *task)
 
                z->name = ldb_msg_find_attr_as_string(res->msgs[i], "name", NULL);
                z->dn = talloc_move(z, &res->msgs[i]->dn);
-               /* Ignore the RootDNSServers zone and zones that we don't support yet */
+               /*
+                * Ignore the RootDNSServers zone and zones that we don't support yet
+                * RootDNSServers should never be returned (Windows DNS server don't)
+                * ..TrustAnchors should never be returned as is, (Windows returns
+                * TrustAnchors) and for the moment we don't support DNSSEC so we'd better
+                * not return this zone.
+                */
                if ((strcmp(z->name, "RootDNSServers") == 0) ||
                        (strcmp(z->name, "..TrustAnchors") == 0)) {
                        DEBUG(10, ("Ignoring zone %s\n", z->name));