]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
remove dns_name_fromstring2()
authorEvan Hunt <each@isc.org>
Wed, 16 Aug 2023 00:41:01 +0000 (17:41 -0700)
committerEvan Hunt <each@isc.org>
Thu, 31 Aug 2023 17:29:16 +0000 (10:29 -0700)
we don't need two versions of dns_name_fromstring() any longer; we
can just specify an origin value of dns_rootname for absolute names.

21 files changed:
bin/dig/dig.c
bin/dig/dighost.c
bin/named/server.c
bin/tests/system/dyndb/driver/instance.c
bin/tests/system/makejournal.c
bin/tools/named-rrchecker.c
fuzz/dns_message_checksig.c
lib/dns/catz.c
lib/dns/gssapictx.c
lib/dns/include/dns/name.h
lib/dns/name.c
lib/dns/rdata/in_1/srv_33.c
lib/isccfg/check.c
tests/dns/badcache_test.c
tests/dns/db_test.c
tests/dns/name_test.c
tests/dns/rdata_test.c
tests/dns/sigs_test.c
tests/dns/tsig_test.c
tests/libtest/dns.c
tests/libtest/ns.c

index 6c0dcb1f0d14fe0d93daedab8853d4d0a43c6cf1..54f7f81e33ddbb0115444411d9c6ee88030cbc3c 100644 (file)
@@ -487,7 +487,8 @@ dns64prefix_answer(dns_message_t *msg, isc_buffer_t *buf) {
        size_t i, count = 10;
 
        name = dns_fixedname_initname(&fixed);
-       result = dns_name_fromstring(name, "ipv4only.arpa", 0, NULL);
+       result = dns_name_fromstring(name, "ipv4only.arpa", dns_rootname, 0,
+                                    NULL);
        check_result(result, "dns_name_fromstring");
 
        result = dns_message_findname(msg, DNS_SECTION_ANSWER, name,
index bafa54bcd9f4605b81d36422b294c3c928e7d873..5e8e8422a5318a485e3380a7247d1fa46f654e9c 100644 (file)
@@ -2029,7 +2029,7 @@ next_origin(dig_lookup_t *oldlookup) {
         * Check for a absolute name or ndots being met.
         */
        name = dns_fixedname_initname(&fixed);
-       result = dns_name_fromstring2(name, oldlookup->textname, NULL, 0, NULL);
+       result = dns_name_fromstring(name, oldlookup->textname, NULL, 0, NULL);
        if (result == ISC_R_SUCCESS &&
            (dns_name_isabsolute(name) ||
             (int)dns_name_countlabels(name) > ndots))
index 9536617488d553ac6cedb91d40295318daed068c..324bc9587e5a1c06f4c94cff7a60b2fe62c6aac7 100644 (file)
@@ -2194,7 +2194,8 @@ configure_rpz_name(dns_view_t *view, const cfg_obj_t *obj, dns_name_t *name,
                   const char *str, const char *msg) {
        isc_result_t result;
 
-       result = dns_name_fromstring(name, str, DNS_NAME_DOWNCASE, view->mctx);
+       result = dns_name_fromstring(name, str, dns_rootname, DNS_NAME_DOWNCASE,
+                                    view->mctx);
        if (result != ISC_R_SUCCESS) {
                cfg_obj_log(obj, named_g_lctx, DNS_RPZ_ERROR_LEVEL,
                            "invalid %s '%s'", msg, str);
@@ -2207,8 +2208,8 @@ configure_rpz_name2(dns_view_t *view, const cfg_obj_t *obj, dns_name_t *name,
                    const char *str, const dns_name_t *origin) {
        isc_result_t result;
 
-       result = dns_name_fromstring2(name, str, origin, DNS_NAME_DOWNCASE,
-                                     view->mctx);
+       result = dns_name_fromstring(name, str, origin, DNS_NAME_DOWNCASE,
+                                    view->mctx);
        if (result != ISC_R_SUCCESS) {
                cfg_obj_log(obj, named_g_lctx, DNS_RPZ_ERROR_LEVEL,
                            "invalid zone '%s'", str);
@@ -3020,8 +3021,8 @@ configure_catz_zone(dns_view_t *view, dns_view_t *pview,
 
        str = cfg_obj_asstring(cfg_tuple_get(catz_obj, "zone name"));
 
-       result = dns_name_fromstring(&origin, str, DNS_NAME_DOWNCASE,
-                                    view->mctx);
+       result = dns_name_fromstring(&origin, str, dns_rootname,
+                                    DNS_NAME_DOWNCASE, view->mctx);
        if (result == ISC_R_SUCCESS && dns_name_equal(&origin, dns_rootname)) {
                result = DNS_R_EMPTYLABEL;
        }
@@ -3415,7 +3416,7 @@ create_empty_zone(dns_zone_t *pzone, dns_name_t *name, dns_view_t *view,
        {
                zconfig = cfg_listelt_value(element);
                str = cfg_obj_asstring(cfg_tuple_get(zconfig, "name"));
-               CHECK(dns_name_fromstring(zname, str, 0, NULL));
+               CHECK(dns_name_fromstring(zname, str, dns_rootname, 0, NULL));
                namereln = dns_name_fullcompare(zname, name, &order, &nlabels);
                if (namereln != dns_namereln_subdomain) {
                        continue;
@@ -3446,10 +3447,11 @@ create_empty_zone(dns_zone_t *pzone, dns_name_t *name, dns_view_t *view,
                                dns_name_clone(name, ns);
                        } else {
                                CHECK(dns_name_fromstring(ns, empty_dbtype[2],
-                                                         0, NULL));
+                                                         dns_rootname, 0,
+                                                         NULL));
                        }
-                       CHECK(dns_name_fromstring(contact, empty_dbtype[3], 0,
-                                                 NULL));
+                       CHECK(dns_name_fromstring(contact, empty_dbtype[3],
+                                                 dns_rootname, 0, NULL));
                        CHECK(add_soa(db, version, name, ns, contact));
                        CHECK(add_ns(db, version, name, ns));
                }
@@ -5648,7 +5650,7 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                result = named_config_get(maps, "empty-server", &obj);
                if (result == ISC_R_SUCCESS) {
                        CHECK(dns_name_fromstring(name, cfg_obj_asstring(obj),
-                                                 0, NULL));
+                                                 dns_rootname, 0, NULL));
                        isc_buffer_init(&buffer, server, sizeof(server) - 1);
                        CHECK(dns_name_totext(name, false, &buffer));
                        server[isc_buffer_usedlength(&buffer)] = 0;
@@ -5661,7 +5663,7 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                result = named_config_get(maps, "empty-contact", &obj);
                if (result == ISC_R_SUCCESS) {
                        CHECK(dns_name_fromstring(name, cfg_obj_asstring(obj),
-                                                 0, NULL));
+                                                 dns_rootname, 0, NULL));
                        isc_buffer_init(&buffer, contact, sizeof(contact) - 1);
                        CHECK(dns_name_totext(name, false, &buffer));
                        contact[isc_buffer_usedlength(&buffer)] = 0;
@@ -5701,7 +5703,8 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                        /*
                         * Look for zone on drop list.
                         */
-                       CHECK(dns_name_fromstring(name, empty, 0, NULL));
+                       CHECK(dns_name_fromstring(name, empty, dns_rootname, 0,
+                                                 NULL));
                        if (disablelist != NULL &&
                            on_disable_list(disablelist, name))
                        {
@@ -5804,8 +5807,9 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                        dns_forwarders_t *dnsforwarders = NULL;
                        dns_fwdpolicy_t fwdpolicy = dns_fwdpolicy_none;
 
-                       CHECK(dns_name_fromstring(
-                               name, zones[ipv4only_zone].name, 0, NULL));
+                       CHECK(dns_name_fromstring(name,
+                                                 zones[ipv4only_zone].name,
+                                                 dns_rootname, 0, NULL));
 
                        (void)dns_view_findzone(view, name, DNS_ZTFIND_EXACT,
                                                &zone);
@@ -5883,8 +5887,8 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
        result = named_config_get(maps, "nxdomain-redirect", &obj);
        if (result == ISC_R_SUCCESS) {
                dns_name_t *name = dns_fixedname_name(&view->redirectfixed);
-               CHECK(dns_name_fromstring(name, cfg_obj_asstring(obj), 0,
-                                         NULL));
+               CHECK(dns_name_fromstring(name, cfg_obj_asstring(obj),
+                                         dns_rootname, 0, NULL));
                view->redirectzone = name;
        } else {
                view->redirectzone = NULL;
@@ -5907,8 +5911,9 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
 
                        ntaname = dns_fixedname_initname(&fntaname);
                        obj = cfg_listelt_value(element);
-                       CHECK(dns_name_fromstring(
-                               ntaname, cfg_obj_asstring(obj), 0, NULL));
+                       CHECK(dns_name_fromstring(ntaname,
+                                                 cfg_obj_asstring(obj),
+                                                 dns_rootname, 0, NULL));
                        CHECK(dns_ntatable_add(ntatable, ntaname, true, 0,
                                               0xffffffffU));
                }
@@ -6158,7 +6163,8 @@ validate_tls(const cfg_obj_t *config, dns_view_t *view, const cfg_obj_t *obj,
             isc_log_t *logctx, const char *str, dns_name_t **name) {
        dns_fixedname_t fname;
        dns_name_t *nm = dns_fixedname_initname(&fname);
-       isc_result_t result = dns_name_fromstring(nm, str, 0, NULL);
+       isc_result_t result = dns_name_fromstring(nm, str, dns_rootname, 0,
+                                                 NULL);
 
        if (result != ISC_R_SUCCESS) {
                cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
@@ -7155,7 +7161,7 @@ get_tat_qname(dns_name_t *target, dns_name_t *keyname, dns_keynode_t *keynode) {
                isc_textregion_consume(&r, m);
        }
 
-       return (dns_name_fromstring2(target, label, keyname, 0, NULL));
+       return (dns_name_fromstring(target, label, keyname, 0, NULL));
 }
 
 static void
@@ -7759,7 +7765,7 @@ configure_zone_setviewcommit(isc_result_t result, const cfg_obj_t *zconfig,
        zname = cfg_obj_asstring(cfg_tuple_get(zconfig, "name"));
        origin = dns_fixedname_initname(&fixorigin);
 
-       result2 = dns_name_fromstring(origin, zname, 0, NULL);
+       result2 = dns_name_fromstring(origin, zname, dns_rootname, 0, NULL);
        if (result2 != ISC_R_SUCCESS) {
                return;
        }
@@ -10460,7 +10466,7 @@ zone_from_args(named_server_t *server, isc_lex_t *lex, const char *zonetxt,
        }
 
        name = dns_fixedname_initname(&fname);
-       CHECK(dns_name_fromstring(name, zonebuf, 0, NULL));
+       CHECK(dns_name_fromstring(name, zonebuf, dns_rootname, 0, NULL));
 
        /* Look for the optional class name. */
        classtxt = next_token(lex, text);
@@ -13347,7 +13353,7 @@ delete_zoneconf(dns_view_t *view, cfg_parser_t *pctx, const cfg_obj_t *config,
                cfg_listelt_t *e;
 
                zn = cfg_obj_asstring(cfg_tuple_get(zconf, "name"));
-               result = dns_name_fromstring(myname, zn, 0, NULL);
+               result = dns_name_fromstring(myname, zn, dns_rootname, 0, NULL);
                if (result != ISC_R_SUCCESS || !dns_name_equal(zname, myname)) {
                        continue;
                }
@@ -14193,7 +14199,8 @@ find_name_in_list_from_map(const cfg_obj_t *config,
        if (strcmp(map_key_for_list, "zone") == 0) {
                name1 = dns_fixedname_initname(&fixed1);
                name2 = dns_fixedname_initname(&fixed2);
-               result = dns_name_fromstring(name1, name, 0, NULL);
+               result = dns_name_fromstring(name1, name, dns_rootname, 0,
+                                            NULL);
                RUNTIME_CHECK(result == ISC_R_SUCCESS);
        }
 
@@ -14212,7 +14219,8 @@ find_name_in_list_from_map(const cfg_obj_t *config,
                }
 
                if (name1 != NULL) {
-                       result = dns_name_fromstring(name2, vname, 0, NULL);
+                       result = dns_name_fromstring(name2, vname, dns_rootname,
+                                                    0, NULL);
                        if (result == ISC_R_SUCCESS &&
                            dns_name_equal(name1, name2))
                        {
index af09f3551b74dba5dd6b3c0269206efd47c6750a..9fa597961c95ef92e753ec0c84b9321142b8985b 100644 (file)
@@ -81,14 +81,14 @@ parse_params(isc_mem_t *mctx, int argc, char **argv, dns_name_t *z1,
                result = ISC_R_FAILURE;
                goto cleanup;
        }
-       result = dns_name_fromstring2(z1, argv[0], dns_rootname, 0, mctx);
+       result = dns_name_fromstring(z1, argv[0], dns_rootname, 0, mctx);
        if (result != ISC_R_SUCCESS) {
                log_write(ISC_LOG_ERROR,
                          "parse_params: dns_name_fromstring2 -> %s",
                          isc_result_totext(result));
                goto cleanup;
        }
-       result = dns_name_fromstring2(z2, argv[1], dns_rootname, 0, mctx);
+       result = dns_name_fromstring(z2, argv[1], dns_rootname, 0, mctx);
        if (result != ISC_R_SUCCESS) {
                log_write(ISC_LOG_ERROR,
                          "parse_params: dns_name_fromstring2 -> %s",
index 03be86e14ad97555912f52269bebb71356a539ef..9bed375a87a084df656a1a50b488478fdaa9abe5 100644 (file)
@@ -62,7 +62,7 @@ loadzone(dns_db_t **db, const char *origin, const char *filename) {
 
        name = dns_fixedname_initname(&fixed);
 
-       result = dns_name_fromstring(name, origin, 0, NULL);
+       result = dns_name_fromstring(name, origin, dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                return (result);
        }
index d31ebf79e38e3903891646cb712c0a9ff597e59a..b20a7e7d280415f88cb79abfa3b21cd5b11eedc6 100644 (file)
@@ -185,7 +185,8 @@ main(int argc, char *argv[]) {
 
        if (origin != NULL) {
                name = dns_fixedname_initname(&fixed);
-               result = dns_name_fromstring(name, origin, 0, NULL);
+               result = dns_name_fromstring(name, origin, dns_rootname, 0,
+                                            NULL);
                if (result != ISC_R_SUCCESS) {
                        fatal("dns_name_fromstring: %s",
                              isc_result_totext(result));
index a253fa7027650b7cf841f89a2b495a906e09ab33..9129f540e6c111fbe9bf8393e98d78abb152a638 100644 (file)
@@ -193,7 +193,7 @@ LLVMFuzzerInitialize(int *argc ISC_ATTR_UNUSED, char ***argv ISC_ATTR_UNUSED) {
        dns_tsigkeyring_create(mctx, &ring);
        dns_tsigkeyring_create(mctx, &emptyring);
 
-       result = dns_name_fromstring(name, "tsig-key", 0, NULL);
+       result = dns_name_fromstring(name, "tsig-key", dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                fprintf(stderr, "dns_name_fromstring failed: %s\n",
                        isc_result_totext(result));
@@ -214,7 +214,7 @@ LLVMFuzzerInitialize(int *argc ISC_ATTR_UNUSED, char ***argv ISC_ATTR_UNUSED) {
                return (1);
        }
 
-       result = dns_name_fromstring(name, "sig0key", 0, NULL);
+       result = dns_name_fromstring(name, "sig0key", dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                fprintf(stderr, "dns_name_fromstring failed: %s\n",
                        isc_result_totext(result));
index 63fcc3dd0ee152387b22a0d59253920a4b506a20..d75f688433a698feea404eec17331e8d515a5650 100644 (file)
@@ -1439,7 +1439,8 @@ catz_process_primaries(dns_catz_zone_t *catz, dns_ipkeylist_t *ipkl,
                        memmove(keycbuf, rdatastr.data, rdatastr.length);
                        keycbuf[rdatastr.length] = 0;
                        dns_rdata_freestruct(&rdata_txt);
-                       result = dns_name_fromstring(keyname, keycbuf, 0, mctx);
+                       result = dns_name_fromstring(keyname, keycbuf,
+                                                    dns_rootname, 0, mctx);
                        if (result != ISC_R_SUCCESS) {
                                dns_name_free(keyname, mctx);
                                isc_mem_put(mctx, keyname, sizeof(*keyname));
@@ -2293,7 +2294,7 @@ dns__catz_update_cb(void *data) {
         * records might be processed differently depending on the version of
         * the catalog zone's schema.
         */
-       result = dns_name_fromstring2(name, "version", &updb->origin, 0, NULL);
+       result = dns_name_fromstring(name, "version", &updb->origin, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                dns_dbiterator_destroy(&updbit);
                isc_log_write(dns_lctx, DNS_LOGCATEGORY_GENERAL,
index 3ff8011f34627b41ae26fe58171e490d1db41a9a..631acd1bfa271b3c2a3c1f03be5ec68f0b958010 100644 (file)
@@ -431,7 +431,8 @@ dst_gssapi_identitymatchesrealmkrb5(const dns_name_t *signer,
                dns_name_t *machine;
 
                machine = dns_fixedname_initname(&fixed);
-               result = dns_name_fromstring(machine, sname, 0, NULL);
+               result = dns_name_fromstring(machine, sname, dns_rootname, 0,
+                                            NULL);
                if (result != ISC_R_SUCCESS) {
                        return (false);
                }
@@ -511,7 +512,7 @@ dst_gssapi_identitymatchesrealmms(const dns_name_t *signer,
                dns_name_t *machine;
 
                machine = dns_fixedname_initname(&fixed);
-               result = dns_name_fromstring2(machine, sbuf, realm, 0, NULL);
+               result = dns_name_fromstring(machine, sbuf, realm, 0, NULL);
                if (result != ISC_R_SUCCESS) {
                        return (false);
                }
index 51568b54bbf1fed237cca677a039cc26d95186a6..e02eca3b6861414f3c9d5818e9ea10db484e8edc 100644 (file)
@@ -1160,12 +1160,9 @@ dns_name_tostring(const dns_name_t *source, char **target, isc_mem_t *mctx);
  */
 
 isc_result_t
-dns_name_fromstring(dns_name_t *target, const char *src, unsigned int options,
+dns_name_fromstring(dns_name_t *target, const char *src,
+                   const dns_name_t *origin, unsigned int options,
                    isc_mem_t *mctx);
-isc_result_t
-dns_name_fromstring2(dns_name_t *target, const char *src,
-                    const dns_name_t *origin, unsigned int options,
-                    isc_mem_t *mctx);
 /*%<
  * Convert a string to a name and place it in target, allocating memory
  * as necessary.  'options' has the same semantics as that of
index fb6863231161b56bfeec3abc41ea8eaa7f8379de..fa1dda64d82d8e0cea3666545a5ce9ac62263180 100644 (file)
@@ -2104,20 +2104,10 @@ dns_name_tostring(const dns_name_t *name, char **target, isc_mem_t *mctx) {
        return (ISC_R_SUCCESS);
 }
 
-/*
- * dns_name_fromstring() -- convert directly from a string to a name,
- * allocating memory as needed
- */
 isc_result_t
-dns_name_fromstring(dns_name_t *target, const char *src, unsigned int options,
+dns_name_fromstring(dns_name_t *target, const char *src,
+                   const dns_name_t *origin, unsigned int options,
                    isc_mem_t *mctx) {
-       return (dns_name_fromstring2(target, src, dns_rootname, options, mctx));
-}
-
-isc_result_t
-dns_name_fromstring2(dns_name_t *target, const char *src,
-                    const dns_name_t *origin, unsigned int options,
-                    isc_mem_t *mctx) {
        isc_result_t result;
        isc_buffer_t buf;
        dns_fixedname_t fn;
index 2ebfc1db5b76029d9b266838049cb48e6acb9ca6..46da0d0326dd7dd2b82d1c113d8602c3c586db37 100644 (file)
@@ -335,8 +335,8 @@ additionaldata_in_srv(ARGS_ADDLDATA) {
 
        dns_fixedname_init(&fixed);
        snprintf(buf, sizeof(buf), "_%u._tcp", port);
-       result = dns_name_fromstring2(dns_fixedname_name(&fixed), buf, NULL, 0,
-                                     NULL);
+       result = dns_name_fromstring(dns_fixedname_name(&fixed), buf, NULL, 0,
+                                    NULL);
        if (result != ISC_R_SUCCESS) {
                return (ISC_R_SUCCESS);
        }
index 2098030283f22b5ac7bf7bb513cacf31a42fdf04..c55d9fa24190773826d421786a18020c865eb50f 100644 (file)
@@ -278,7 +278,8 @@ validate_tls(const cfg_obj_t *config, const cfg_obj_t *obj, isc_log_t *logctx,
             const char *str) {
        dns_fixedname_t fname;
        dns_name_t *nm = dns_fixedname_initname(&fname);
-       isc_result_t result = dns_name_fromstring(nm, str, 0, NULL);
+       isc_result_t result = dns_name_fromstring(nm, str, dns_rootname, 0,
+                                                 NULL);
 
        if (result != ISC_R_SUCCESS) {
                cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
@@ -975,7 +976,8 @@ check_name(const char *str) {
        dns_fixedname_t fixed;
 
        dns_fixedname_init(&fixed);
-       return (dns_name_fromstring(dns_fixedname_name(&fixed), str, 0, NULL));
+       return (dns_name_fromstring(dns_fixedname_name(&fixed), str,
+                                   dns_rootname, 0, NULL));
 }
 
 static bool
@@ -2344,7 +2346,8 @@ resume:
                                const char *str = cfg_obj_asstring(key);
                                dns_fixedname_t fname;
                                dns_name_t *nm = dns_fixedname_initname(&fname);
-                               tresult = dns_name_fromstring(nm, str, 0, NULL);
+                               tresult = dns_name_fromstring(
+                                       nm, str, dns_rootname, 0, NULL);
                                if (tresult != ISC_R_SUCCESS) {
                                        cfg_obj_log(key, logctx, ISC_LOG_ERROR,
                                                    "'%s' is not a valid name",
@@ -2358,7 +2361,8 @@ resume:
                                const char *str = cfg_obj_asstring(tls);
                                dns_fixedname_t fname;
                                dns_name_t *nm = dns_fixedname_initname(&fname);
-                               tresult = dns_name_fromstring(nm, str, 0, NULL);
+                               tresult = dns_name_fromstring(
+                                       nm, str, dns_rootname, 0, NULL);
                                if (tresult != ISC_R_SUCCESS) {
                                        cfg_obj_log(tls, logctx, ISC_LOG_ERROR,
                                                    "'%s' is not a valid name",
@@ -2491,7 +2495,7 @@ check_update_policy(const cfg_obj_t *policy, isc_log_t *logctx) {
                }
 
                str = cfg_obj_asstring(identity);
-               tresult = dns_name_fromstring(id, str, 1, NULL);
+               tresult = dns_name_fromstring(id, str, dns_rootname, 1, NULL);
                if (tresult != ISC_R_SUCCESS) {
                        cfg_obj_log(identity, logctx, ISC_LOG_ERROR,
                                    "'%s' is not a valid name", str);
@@ -2509,7 +2513,8 @@ check_update_policy(const cfg_obj_t *policy, isc_log_t *logctx) {
                        str = cfg_obj_asstring(dname);
                }
                if (tresult == ISC_R_SUCCESS) {
-                       tresult = dns_name_fromstring(name, str, 0, NULL);
+                       tresult = dns_name_fromstring(name, str, dns_rootname,
+                                                     0, NULL);
                        if (tresult != ISC_R_SUCCESS) {
                                cfg_obj_log(dname, logctx, ISC_LOG_ERROR,
                                            "'%s' is not a valid name", str);
@@ -4492,7 +4497,7 @@ record_static_keys(isc_symtab_t *symtab, isc_mem_t *mctx,
                const char *str = cfg_obj_asstring(cfg_tuple_get(obj, "name"));
                isc_symvalue_t symvalue;
 
-               result = dns_name_fromstring(name, str, 0, NULL);
+               result = dns_name_fromstring(name, str, dns_rootname, 0, NULL);
                if (result != ISC_R_SUCCESS) {
                        continue;
                }
@@ -4565,7 +4570,7 @@ check_initializing_keys(isc_symtab_t *symtab, const cfg_obj_t *keylist,
                }
 
                str = cfg_obj_asstring(cfg_tuple_get(obj, "name"));
-               result = dns_name_fromstring(name, str, 0, NULL);
+               result = dns_name_fromstring(name, str, dns_rootname, 0, NULL);
                if (result != ISC_R_SUCCESS) {
                        continue;
                }
@@ -4613,7 +4618,7 @@ record_ds_keys(isc_symtab_t *symtab, isc_mem_t *mctx,
                const char *str = cfg_obj_asstring(cfg_tuple_get(obj, "name"));
                isc_symvalue_t symvalue;
 
-               result = dns_name_fromstring(name, str, 0, NULL);
+               result = dns_name_fromstring(name, str, dns_rootname, 0, NULL);
                if (result != ISC_R_SUCCESS) {
                        continue;
                }
@@ -4805,7 +4810,8 @@ check_rpz_catz(const char *rpz_catz, const cfg_obj_t *rpz_obj,
                        }
                }
 
-               tresult = dns_name_fromstring(name, zonename, 0, NULL);
+               tresult = dns_name_fromstring(name, zonename, dns_rootname, 0,
+                                             NULL);
                if (tresult != ISC_R_SUCCESS) {
                        cfg_obj_log(nameobj, logctx, ISC_LOG_ERROR,
                                    "bad domain name '%s'", zonename);
@@ -4864,7 +4870,8 @@ check_rpz(const cfg_obj_t *rpz_obj, isc_log_t *logctx) {
                nameobj = cfg_tuple_get(obj, "zone name");
                zonename = cfg_obj_asstring(nameobj);
 
-               tresult = dns_name_fromstring(name, zonename, 0, NULL);
+               tresult = dns_name_fromstring(name, zonename, dns_rootname, 0,
+                                             NULL);
                if (tresult != ISC_R_SUCCESS) {
                        cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
                                    "bad domain name '%s'", zonename);
@@ -4922,7 +4929,8 @@ check_catz(const cfg_obj_t *catz_obj, const char *viewname, isc_mem_t *mctx,
                nameobj = cfg_tuple_get(obj, "zone name");
                zonename = cfg_obj_asstring(nameobj);
 
-               tresult = dns_name_fromstring(name, zonename, 0, NULL);
+               tresult = dns_name_fromstring(name, zonename, dns_rootname, 0,
+                                             NULL);
                if (tresult != ISC_R_SUCCESS) {
                        cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
                                    "bad domain name '%s'", zonename);
index dfe8adf6dffbdf4761c05b95b8a06e17f69692cc..f7151b73fdfcec8a3710943dfbdca27fe6d64e03 100644 (file)
@@ -67,7 +67,7 @@ ISC_LOOP_TEST_IMPL(basic) {
        isc_result_t result;
        uint32_t flags = BADCACHE_TEST_FLAG;
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
 
        bc = dns_badcache_new(mctx);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
@@ -95,7 +95,7 @@ ISC_LOOP_TEST_IMPL(expire) {
        isc_result_t result;
        uint32_t flags = BADCACHE_TEST_FLAG;
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
 
        bc = dns_badcache_new(mctx);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
@@ -138,7 +138,7 @@ ISC_LOOP_TEST_IMPL(print) {
        uint8_t buf[4096];
        size_t len;
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
 
        bc = dns_badcache_new(mctx);
        dns_badcache_add(bc, name, dns_rdatatype_a, false, flags, expire);
@@ -169,7 +169,7 @@ ISC_LOOP_TEST_IMPL(flush) {
        isc_result_t result;
        uint32_t flags = BADCACHE_TEST_FLAG;
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
 
        bc = dns_badcache_new(mctx);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
@@ -197,22 +197,22 @@ ISC_LOOP_TEST_IMPL(flushname) {
 
        bc = dns_badcache_new(mctx);
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
 
-       dns_name_fromstring2(name, "sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
 
-       dns_name_fromstring2(name, "sub.sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.sub.example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
 
-       dns_name_fromstring2(name, "sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.example.com.", NULL, 0, NULL);
        dns_badcache_flushname(bc, name);
 
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
@@ -220,11 +220,11 @@ ISC_LOOP_TEST_IMPL(flushname) {
        result = dns_badcache_find(bc, name, dns_rdatatype_a, &flags, now);
        assert_int_equal(result, ISC_R_NOTFOUND);
 
-       dns_name_fromstring2(name, "sub.sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.sub.example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
 
@@ -243,36 +243,36 @@ ISC_LOOP_TEST_IMPL(flushtree) {
 
        bc = dns_badcache_new(mctx);
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
        assert_int_equal(flags, BADCACHE_TEST_FLAG);
 
-       dns_name_fromstring2(name, "sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
        assert_int_equal(flags, BADCACHE_TEST_FLAG);
 
-       dns_name_fromstring2(name, "sub.sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.sub.example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now + 60);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
        assert_int_equal(flags, BADCACHE_TEST_FLAG);
 
-       dns_name_fromstring2(name, "sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.example.com.", NULL, 0, NULL);
        dns_badcache_flushtree(bc, name);
 
-       dns_name_fromstring2(name, "sub.sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.sub.example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_NOTFOUND);
 
-       dns_name_fromstring2(name, "sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_NOTFOUND);
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags, now);
        assert_int_equal(result, ISC_R_SUCCESS);
        assert_int_equal(flags, BADCACHE_TEST_FLAG);
@@ -292,19 +292,19 @@ ISC_LOOP_TEST_IMPL(purge) {
 
        bc = dns_badcache_new(mctx);
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags,
                                   now - 60);
        assert_int_equal(result, ISC_R_SUCCESS);
 
-       dns_name_fromstring2(name, "sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags,
                                   now - 60);
        assert_int_equal(result, ISC_R_SUCCESS);
 
-       dns_name_fromstring2(name, "sub.sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.sub.example.com.", NULL, 0, NULL);
        dns_badcache_add(bc, name, dns_rdatatype_aaaa, false, flags, now);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags,
                                   now - 60);
@@ -314,17 +314,17 @@ ISC_LOOP_TEST_IMPL(purge) {
                                   now + 30);
        assert_int_equal(result, ISC_R_NOTFOUND);
 
-       dns_name_fromstring2(name, "sub.sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.sub.example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags,
                                   now + 30);
        assert_int_equal(result, ISC_R_NOTFOUND);
 
-       dns_name_fromstring2(name, "sub.example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "sub.example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags,
                                   now + 30);
        assert_int_equal(result, ISC_R_NOTFOUND);
 
-       dns_name_fromstring2(name, "example.com.", NULL, 0, NULL);
+       dns_name_fromstring(name, "example.com.", NULL, 0, NULL);
        result = dns_badcache_find(bc, name, dns_rdatatype_aaaa, &flags,
                                   now + 30);
        assert_int_equal(result, ISC_R_NOTFOUND);
index 7a6e436230ab0f49b46414e4e8d54ade39f90ecb..ffcd65d27564c1c7ab916cc330041e44099ac380 100644 (file)
@@ -114,7 +114,7 @@ ISC_RUN_TEST_IMPL(dns_dbfind_staleok) {
        example = dns_fixedname_initname(&example_fixed);
        found = dns_fixedname_initname(&found_fixed);
 
-       result = dns_name_fromstring(example, "example", 0, NULL);
+       result = dns_name_fromstring(example, "example", dns_rootname, 0, NULL);
        assert_int_equal(result, ISC_R_SUCCESS);
 
        /*
index f5b517bc0d50915e25c0f9022b32fe7d68072ec5..bb4d2679ce783ac7aa92f9823670be25ceae62be 100644 (file)
@@ -93,15 +93,15 @@ ISC_RUN_TEST_IMPL(fullcompare) {
                if (data[i].name1[0] == 0) {
                        dns_fixedname_init(&fixed1);
                } else {
-                       result = dns_name_fromstring2(name1, data[i].name1,
-                                                     NULL, 0, NULL);
+                       result = dns_name_fromstring(name1, data[i].name1, NULL,
+                                                    0, NULL);
                        assert_int_equal(result, ISC_R_SUCCESS);
                }
                if (data[i].name2[0] == 0) {
                        dns_fixedname_init(&fixed2);
                } else {
-                       result = dns_name_fromstring2(name2, data[i].name2,
-                                                     NULL, 0, NULL);
+                       result = dns_name_fromstring(name2, data[i].name2, NULL,
+                                                    0, NULL);
                        assert_int_equal(result, ISC_R_SUCCESS);
                }
                relation = dns_name_fullcompare(name1, name1, &order, &nlabels);
@@ -493,7 +493,8 @@ ISC_RUN_TEST_IMPL(istat) {
        name = dns_fixedname_initname(&fixed);
 
        for (i = 0; i < (sizeof(data) / sizeof(data[0])); i++) {
-               result = dns_name_fromstring(name, data[i].name, 0, NULL);
+               result = dns_name_fromstring(name, data[i].name, dns_rootname,
+                                            0, NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
                assert_int_equal(dns_name_istat(name), data[i].istat);
        }
@@ -621,11 +622,11 @@ ISC_RUN_TEST_IMPL(hash) {
                n1 = dns_fixedname_initname(&f1);
                n2 = dns_fixedname_initname(&f2);
 
-               result = dns_name_fromstring2(n1, testcases[i].name1, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n1, testcases[i].name1, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
-               result = dns_name_fromstring2(n2, testcases[i].name2, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n2, testcases[i].name2, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
 
                /* Check case-insensitive hashing first */
@@ -680,11 +681,11 @@ ISC_RUN_TEST_IMPL(issubdomain) {
                n1 = dns_fixedname_initname(&f1);
                n2 = dns_fixedname_initname(&f2);
 
-               result = dns_name_fromstring2(n1, testcases[i].name1, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n1, testcases[i].name1, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
-               result = dns_name_fromstring2(n2, testcases[i].name2, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n2, testcases[i].name2, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
 
                if (verbose) {
@@ -730,8 +731,8 @@ ISC_RUN_TEST_IMPL(countlabels) {
 
                name = dns_fixedname_initname(&fname);
 
-               result = dns_name_fromstring2(name, testcases[i].namestr, NULL,
-                                             0, NULL);
+               result = dns_name_fromstring(name, testcases[i].namestr, NULL,
+                                            0, NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
 
                if (verbose) {
@@ -771,11 +772,11 @@ ISC_RUN_TEST_IMPL(getlabel) {
                n1 = dns_fixedname_initname(&f1);
                n2 = dns_fixedname_initname(&f2);
 
-               result = dns_name_fromstring2(n1, testcases[i].name1, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n1, testcases[i].name1, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
-               result = dns_name_fromstring2(n2, testcases[i].name2, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n2, testcases[i].name2, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
 
                dns_name_getlabel(n1, testcases[i].pos1, &l1);
@@ -819,11 +820,11 @@ ISC_RUN_TEST_IMPL(getlabelsequence) {
                n1 = dns_fixedname_initname(&f1);
                n2 = dns_fixedname_initname(&f2);
 
-               result = dns_name_fromstring2(n1, testcases[i].name1, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n1, testcases[i].name1, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
-               result = dns_name_fromstring2(n2, testcases[i].name2, NULL, 0,
-                                             NULL);
+               result = dns_name_fromstring(n2, testcases[i].name2, NULL, 0,
+                                            NULL);
                assert_int_equal(result, ISC_R_SUCCESS);
 
                dns_name_getlabelsequence(n1, testcases[i].pos1,
@@ -849,11 +850,12 @@ ISC_RUN_TEST_IMPL(maxlabels) {
                "a.b.c.";
 
        name = dns_fixedname_initname(&fixed);
-       result = dns_name_fromstring(name, one_too_many, 0, NULL);
+       result = dns_name_fromstring(name, one_too_many, dns_rootname, 0, NULL);
        assert_int_equal(result, ISC_R_NOSPACE);
 
        name = dns_fixedname_initname(&fixed);
-       result = dns_name_fromstring(name, one_too_many + 2, 0, NULL);
+       result = dns_name_fromstring(name, one_too_many + 2, dns_rootname, 0,
+                                    NULL);
        assert_int_equal(result, ISC_R_SUCCESS);
        assert_true(dns_name_isvalid(name));
        assert_int_equal(dns_name_countlabels(name), DNS_NAME_MAXLABELS);
index d960b529f3325c6314b20f95c42adf333263c455..befe57986c184a3fa46891e46ee72566f4fea847 100644 (file)
@@ -224,17 +224,19 @@ rdata_checknames(dns_rdata_t *rdata) {
        (void)dns_rdata_checknames(rdata, dns_rootname, NULL);
        (void)dns_rdata_checknames(rdata, dns_rootname, bad);
 
-       result = dns_name_fromstring(name, "example.net", 0, NULL);
+       result = dns_name_fromstring(name, "example.net", dns_rootname, 0,
+                                    NULL);
        assert_int_equal(result, ISC_R_SUCCESS);
        (void)dns_rdata_checknames(rdata, name, NULL);
        (void)dns_rdata_checknames(rdata, name, bad);
 
-       result = dns_name_fromstring(name, "in-addr.arpa", 0, NULL);
+       result = dns_name_fromstring(name, "in-addr.arpa", dns_rootname, 0,
+                                    NULL);
        assert_int_equal(result, ISC_R_SUCCESS);
        (void)dns_rdata_checknames(rdata, name, NULL);
        (void)dns_rdata_checknames(rdata, name, bad);
 
-       result = dns_name_fromstring(name, "ip6.arpa", 0, NULL);
+       result = dns_name_fromstring(name, "ip6.arpa", dns_rootname, 0, NULL);
        assert_int_equal(result, ISC_R_SUCCESS);
        (void)dns_rdata_checknames(rdata, name, NULL);
        (void)dns_rdata_checknames(rdata, name, bad);
index d521757d87a6106c5c29843b26a9b6e5a6dcf1c0..94d94fbb39ba426cce0e3185838d823db414a8f2 100644 (file)
@@ -129,7 +129,8 @@ compare_tuples(const zonediff_t *expected, dns_difftuple_t *found,
         * Check owner name.
         */
        expected_name = dns_fixedname_initname(&expected_fname);
-       result = dns_name_fromstring(expected_name, expected->owner, 0, mctx);
+       result = dns_name_fromstring(expected_name, expected->owner,
+                                    dns_rootname, 0, mctx);
        assert_int_equal(result, ISC_R_SUCCESS);
        dns_name_format(&found->name, found_name, sizeof(found_name));
        assert_true(dns_name_equal(expected_name, &found->name));
index 6c6935e7c9380fbf2faa0434a20a144c61913c5b..bab3156da75676f8ab857fb9ce6923896138157d 100644 (file)
@@ -287,7 +287,7 @@ ISC_RUN_TEST_IMPL(tsig_tcp) {
        /* isc_log_setdebuglevel(lctx, 99); */
 
        keyname = dns_fixedname_initname(&fkeyname);
-       result = dns_name_fromstring(keyname, "test", 0, NULL);
+       result = dns_name_fromstring(keyname, "test", dns_rootname, 0, NULL);
        assert_int_equal(result, ISC_R_SUCCESS);
 
        dns_tsigkeyring_create(mctx, &ring);
index de0545be8b1805813cf154c2a0077f2f8b4142db..d50c012caf1c9adcf26b2b5eb558ab14338c6027 100644 (file)
@@ -127,7 +127,7 @@ dns_test_makezone(const char *name, dns_zone_t **zonep, dns_view_t *view,
         */
        dns_zone_settype(zone, dns_zone_primary);
        origin = dns_fixedname_initname(&fixed_origin);
-       result = dns_name_fromstring(origin, name, 0, NULL);
+       result = dns_name_fromstring(origin, name, dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                goto detach_zone;
        }
@@ -219,7 +219,7 @@ dns_test_loaddb(dns_db_t **db, dns_dbtype_t dbtype, const char *origin,
 
        name = dns_fixedname_initname(&fixed);
 
-       result = dns_name_fromstring(name, origin, 0, NULL);
+       result = dns_name_fromstring(name, origin, dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                return (result);
        }
@@ -455,7 +455,8 @@ dns_test_difffromchanges(dns_diff_t *diff, const zonechange_t *changes,
                 * Parse owner name.
                 */
                name = dns_fixedname_initname(&fixedname);
-               result = dns_name_fromstring(name, changes[i].owner, 0, mctx);
+               result = dns_name_fromstring(name, changes[i].owner,
+                                            dns_rootname, 0, mctx);
                if (result != ISC_R_SUCCESS) {
                        break;
                }
index 8f7f1302533af023efd08765e91767e300df1c0e..19b2f676991a0534e6f8c4b89ab8c2151efc5d15 100644 (file)
@@ -285,7 +285,7 @@ attach_query_msg_to_client(ns_client_t *client, const char *qnamestr,
         * class IN and type "qtype", link the two and add the result to the
         * QUESTION section of the query.
         */
-       result = dns_name_fromstring(qname, qnamestr, 0, mctx);
+       result = dns_name_fromstring(qname, qnamestr, dns_rootname, 0, mctx);
        if (result != ISC_R_SUCCESS) {
                goto put_name;
        }
@@ -533,7 +533,7 @@ ns_test_loaddb(dns_db_t **db, dns_dbtype_t dbtype, const char *origin,
 
        name = dns_fixedname_initname(&fixed);
 
-       result = dns_name_fromstring(name, origin, 0, NULL);
+       result = dns_name_fromstring(name, origin, dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                return (result);
        }