]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
clean up uses of ISC_R_NOMEMORY
authorEvan Hunt <each@isc.org>
Wed, 8 Jan 2025 03:03:07 +0000 (19:03 -0800)
committerEvan Hunt <each@isc.org>
Thu, 23 Jan 2025 23:54:57 +0000 (15:54 -0800)
the isc_mem allocation functions can no longer fail; as a result,
ISC_R_NOMEMORY is now rarely used: only when an external library
such as libjson-c or libfstrm could return NULL. (even in
these cases, arguably we should assert rather than returning
ISC_R_NOMEMORY.)

code and comments that mentioned ISC_R_NOMEMORY have been
cleaned up, and the following functions have been changed to
type void, since (in most cases) the only value they could
return was ISC_R_SUCCESS:

- dns_dns64_create()
- dns_dyndb_create()
- dns_ipkeylist_resize()
- dns_kasp_create()
- dns_kasp_key_create()
- dns_keystore_create()
- dns_order_create()
- dns_order_add()
- dns_peerlist_new()
- dns_tkeyctx_create()
- dns_view_create()
- dns_zone_setorigin()
- dns_zone_setfile()
- dns_zone_setstream()
- dns_zone_getdbtype()
- dns_zone_setjournal()
- dns_zone_setkeydirectory()
- isc_lex_openstream()
- isc_portset_create()
- isc_symtab_create()

(the exception is dns_view_create(), which could have returned
other error codes in the event of a crypto library failure when
calling isc_file_sanitize(), but that should be a RUNTIME_CHECK
anyway.)

88 files changed:
bin/check/check-tool.c
bin/delv/delv.c
bin/named/controlconf.c
bin/named/include/named/tkeyconf.h
bin/named/include/named/tsigconf.h
bin/named/main.c
bin/named/server.c
bin/named/tkeyconf.c
bin/named/zoneconf.c
bin/nsupdate/nsupdate.c
bin/tests/system/dyndb/driver/zone.c
bin/tests/system/pipelined/pipequeries.c
bin/tools/mdig.c
bin/tools/named-rrchecker.c
fuzz/dns_message_checksig.c
lib/dns/catz.c
lib/dns/client.c
lib/dns/dlz.c
lib/dns/dns64.c
lib/dns/ds.c
lib/dns/dyndb.c
lib/dns/include/dns/adb.h
lib/dns/include/dns/cache.h
lib/dns/include/dns/catz.h
lib/dns/include/dns/db.h
lib/dns/include/dns/diff.h
lib/dns/include/dns/dispatch.h
lib/dns/include/dns/dns64.h
lib/dns/include/dns/dnssec.h
lib/dns/include/dns/dyndb.h
lib/dns/include/dns/forward.h
lib/dns/include/dns/ipkeylist.h
lib/dns/include/dns/kasp.h
lib/dns/include/dns/keystore.h
lib/dns/include/dns/master.h
lib/dns/include/dns/masterdump.h
lib/dns/include/dns/message.h
lib/dns/include/dns/name.h
lib/dns/include/dns/order.h
lib/dns/include/dns/peer.h
lib/dns/include/dns/rdataslab.h
lib/dns/include/dns/resolver.h
lib/dns/include/dns/rriterator.h
lib/dns/include/dns/ssu.h
lib/dns/include/dns/tkey.h
lib/dns/include/dns/tsig.h
lib/dns/include/dns/view.h
lib/dns/include/dns/zone.h
lib/dns/include/dst/dst.h
lib/dns/ipkeylist.c
lib/dns/journal.c
lib/dns/kasp.c
lib/dns/keystore.c
lib/dns/master.c
lib/dns/order.c
lib/dns/peer.c
lib/dns/sdlz.c
lib/dns/tkey.c
lib/dns/view.c
lib/dns/zone.c
lib/isc/file.c
lib/isc/httpd.c
lib/isc/include/isc/buffer.h
lib/isc/include/isc/file.h
lib/isc/include/isc/ht.h
lib/isc/include/isc/interfaceiter.h
lib/isc/include/isc/lex.h
lib/isc/include/isc/log.h
lib/isc/include/isc/mem.h
lib/isc/include/isc/portset.h
lib/isc/include/isc/radix.h
lib/isc/include/isc/symtab.h
lib/isc/lex.c
lib/isc/portset.c
lib/isc/symtab.c
lib/isccfg/check.c
lib/isccfg/include/isccfg/cfg.h
lib/isccfg/include/isccfg/kaspconf.h
lib/isccfg/kaspconf.c
lib/isccfg/parser.c
lib/ns/client.c
lib/ns/include/ns/hooks.h
lib/ns/query.c
tests/dns/nsec3param_test.c
tests/dns/sigs_test.c
tests/isc/symtab_test.c
tests/libtest/dns.c
tests/libtest/ns.c

index 98924e3dfc0d17699c77d6f27df0c2fc2b33543e..f502840f3fc04dfe968bd6449c6285ff46364d88 100644 (file)
@@ -112,11 +112,8 @@ add(char *key, int value) {
        }
 
        if (symtab == NULL) {
-               result = isc_symtab_create(sym_mctx, 100, freekey, sym_mctx,
-                                          false, &symtab);
-               if (result != ISC_R_SUCCESS) {
-                       return;
-               }
+               isc_symtab_create(sym_mctx, 100, freekey, sym_mctx, false,
+                                 &symtab);
        }
 
        key = isc_mem_strdup(sym_mctx, key);
@@ -660,17 +657,17 @@ load_zone(isc_mem_t *mctx, const char *zonename, const char *filename,
        isc_buffer_add(&buffer, strlen(zonename));
        origin = dns_fixedname_initname(&fixorigin);
        CHECK(dns_name_fromtext(origin, &buffer, dns_rootname, 0, NULL));
-       CHECK(dns_zone_setorigin(zone, origin));
+       dns_zone_setorigin(zone, origin);
        dns_zone_setdbtype(zone, 1, (const char *const *)dbtype);
        if (strcmp(filename, "-") == 0) {
-               CHECK(dns_zone_setstream(zone, stdin, fileformat,
-                                        &dns_master_style_default));
+               dns_zone_setstream(zone, stdin, fileformat,
+                                  &dns_master_style_default);
        } else {
-               CHECK(dns_zone_setfile(zone, filename, fileformat,
-                                      &dns_master_style_default));
+               dns_zone_setfile(zone, filename, fileformat,
+                                &dns_master_style_default);
        }
        if (journal != NULL) {
-               CHECK(dns_zone_setjournal(zone, journal));
+               dns_zone_setjournal(zone, journal);
        }
 
        region.base = UNCONST(classname);
index c757a41246ea01e09c0d30f0250f002183163855..f6ac3d25428f4e75ec676038c6be82e41dd77a84 100644 (file)
@@ -2202,8 +2202,8 @@ run_server(void *arg) {
        CHECK(ns_interfacemgr_create(mctx, sctx, loopmgr, netmgr, dispatchmgr,
                                     NULL, &interfacemgr));
 
-       CHECK(dns_view_create(mctx, loopmgr, dispatchmgr, dns_rdataclass_in,
-                             "_default", &view));
+       dns_view_create(mctx, loopmgr, dispatchmgr, dns_rdataclass_in,
+                       "_default", &view);
        CHECK(dns_cache_create(loopmgr, dns_rdataclass_in, "", mctx, &cache));
        dns_view_setcache(view, cache, false);
        dns_cache_detach(&cache);
index 145e0cb43bfea3b185ae25af6f4ab189bd737448..1a96565bd992cfc55864dd88e466a77a5bb0938e 100644 (file)
@@ -842,15 +842,12 @@ get_rndckey(isc_mem_t *mctx, controlkeylist_t *keyids) {
        CHECK(cfg_map_get(config, "key", &key));
 
        keyid = isc_mem_get(mctx, sizeof(*keyid));
+       *keyid = (controlkey_t){
+               .algorithm = DST_ALG_UNKNOWN,
+               .link = ISC_LINK_INITIALIZER,
+       };
        keyid->keyname = isc_mem_strdup(mctx,
                                        cfg_obj_asstring(cfg_map_getname(key)));
-       keyid->secret.base = NULL;
-       keyid->secret.length = 0;
-       keyid->algorithm = DST_ALG_UNKNOWN;
-       ISC_LINK_INIT(keyid, link);
-       if (keyid->keyname == NULL) {
-               CHECK(ISC_R_NOMEMORY);
-       }
 
        CHECK(isccfg_check_key(key));
 
index ad6a297dacbcf5dbd612cbff7e5348c2b87fe959..2ea4063d3579d18ea1f1be68afc935362f2517d1 100644 (file)
@@ -34,5 +34,5 @@ named_tkeyctx_fromconfig(const cfg_obj_t *options, isc_mem_t *mctx,
  *
  *     Returns:
  *\li          ISC_R_SUCCESS
- *\li          ISC_R_NOMEMORY
+ *\li          return codes from dns_name_fromtext()
  */
index 7adaf2ed8ba8e9f1b815ee10c3cd3f958195cda4..60a8bf2d5c70e777e527a289f942ecc418b27bf5 100644 (file)
@@ -32,5 +32,6 @@ named_tsigkeyring_fromconfig(const cfg_obj_t *config, const cfg_obj_t *vconfig,
  *
  *     Returns:
  *     \li     ISC_R_SUCCESS
- *     \li     ISC_R_NOMEMORY
+ *     \li     DNS_R_BADALG
+ *     \li     return codes from dns_name_fromtext()
  */
index b28d20700563d4c632d49f98889324abbcc03339..c7547da9c96c9dbc4e51270669f8d08d8e357855 100644 (file)
@@ -1414,14 +1414,7 @@ named_smf_get_instance(char **ins_name, int debug, isc_mem_t *mctx) {
                return ISC_R_FAILURE;
        }
 
-       if ((instance = isc_mem_allocate(mctx, namelen + 1)) == NULL) {
-               UNEXPECTED_ERROR("named_smf_get_instance memory "
-                                "allocation failed: %s",
-                                isc_result_totext(ISC_R_NOMEMORY));
-               scf_handle_destroy(h);
-               return ISC_R_FAILURE;
-       }
-
+       instance = isc_mem_allocate(mctx, namelen + 1);
        if (scf_myname(h, instance, namelen + 1) == -1) {
                if (debug) {
                        UNEXPECTED_ERROR("scf_myname() failed: %s",
index 0f06cd9c19faa646456909d0fac0684e136e38c8..346a13d27614d63940b5b876e0496ead42a8573f 100644 (file)
@@ -1317,15 +1317,12 @@ configure_order(dns_order_t *order, const cfg_obj_t *ent) {
         * explicit entry for "." when the name is "*".
         */
        if (addroot) {
-               result = dns_order_add(order, dns_rootname, rdtype, rdclass,
-                                      mode);
-               if (result != ISC_R_SUCCESS) {
-                       return result;
-               }
+               dns_order_add(order, dns_rootname, rdtype, rdclass, mode);
        }
 
-       return dns_order_add(order, dns_fixedname_name(&fixed), rdtype, rdclass,
-                            mode);
+       dns_order_add(order, dns_fixedname_name(&fixed), rdtype, rdclass, mode);
+
+       return ISC_R_SUCCESS;
 }
 
 static isc_result_t
@@ -1684,7 +1681,7 @@ check_dbtype(dns_zone_t *zone, unsigned int dbtypec, const char **dbargv,
        unsigned int i;
        isc_result_t result = ISC_R_SUCCESS;
 
-       CHECK(dns_zone_getdbtype(zone, &argv, mctx));
+       dns_zone_getdbtype(zone, &argv, mctx);
 
        /*
         * Check that all the arguments match.
@@ -1853,7 +1850,7 @@ dns64_reverse(dns_view_t *view, isc_mem_t *mctx, isc_netaddr_t *na,
        isc_buffer_add(&b, strlen(reverse));
        CHECK(dns_name_fromtext(name, &b, dns_rootname, 0, NULL));
        dns_zone_create(&zone, mctx, 0);
-       CHECK(dns_zone_setorigin(zone, name));
+       dns_zone_setorigin(zone, name);
        dns_zone_setview(zone, view);
        CHECK(dns_zonemgr_managezone(named_g_server->zonemgr, zone));
        dns_zone_setclass(zone, view->rdclass);
@@ -3223,7 +3220,7 @@ create_empty_zone(dns_zone_t *pzone, dns_name_t *name, dns_view_t *view,
 
        if (pzone == NULL) {
                CHECK(dns_zonemgr_createzone(named_g_server->zonemgr, &zone));
-               CHECK(dns_zone_setorigin(zone, name));
+               dns_zone_setorigin(zone, name);
                CHECK(dns_zonemgr_managezone(named_g_server->zonemgr, zone));
                if (db == NULL) {
                        dns_zone_setdbtype(zone, empty_dbtypec, empty_dbtype);
@@ -3330,7 +3327,7 @@ create_ipv4only_zone(dns_zone_t *pzone, dns_view_t *view,
                 * Create the actual zone.
                 */
                dns_zone_create(&zone, mctx, 0);
-               CHECK(dns_zone_setorigin(zone, name));
+               dns_zone_setorigin(zone, name);
                CHECK(dns_zonemgr_managezone(named_g_server->zonemgr, zone));
                dns_zone_setclass(zone, view->rdclass);
                dns_zone_settype(zone, dns_zone_primary);
@@ -3913,11 +3910,6 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                        const cfg_obj_t *name, *search = NULL;
                        char *s = isc_mem_strdup(mctx, cfg_obj_asstring(obj));
 
-                       if (s == NULL) {
-                               result = ISC_R_NOMEMORY;
-                               goto cleanup;
-                       }
-
                        result = isc_commandline_strtoargv(mctx, s, &dlzargc,
                                                           &dlzargv, 0);
                        if (result != ISC_R_SUCCESS) {
@@ -4147,12 +4139,9 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                                dns64options |= DNS_DNS64_BREAK_DNSSEC;
                        }
 
-                       result = dns_dns64_create(mctx, &na, prefixlen, sp,
-                                                 clients, mapped, excluded,
-                                                 dns64options, &dns64);
-                       if (result != ISC_R_SUCCESS) {
-                               goto cleanup;
-                       }
+                       dns_dns64_create(mctx, &na, prefixlen, sp, clients,
+                                        mapped, excluded, dns64options,
+                                        &dns64);
                        dns_dns64_append(&view->dns64, dns64);
                        view->dns64cnt++;
                        result = dns64_reverse(view, mctx, &na, prefixlen,
@@ -4748,7 +4737,7 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                dns_peerlist_t *newpeers = NULL;
 
                (void)named_config_get(cfgmaps, "server", &peers);
-               CHECK(dns_peerlist_new(mctx, &newpeers));
+               dns_peerlist_new(mctx, &newpeers);
                for (element = cfg_list_first(peers); element != NULL;
                     element = cfg_list_next(element))
                {
@@ -4770,7 +4759,7 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
                const cfg_obj_t *rrsetorder = NULL;
 
                (void)named_config_get(maps, "rrset-order", &rrsetorder);
-               CHECK(dns_order_create(mctx, &order));
+               dns_order_create(mctx, &order);
                for (element = cfg_list_first(rrsetorder); element != NULL;
                     element = cfg_list_next(element))
                {
@@ -5316,9 +5305,9 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
 
                if (dctx == NULL) {
                        const void *hashinit = isc_hash_get_initializer();
-                       CHECK(dns_dyndb_createctx(mctx, hashinit, view,
-                                                 named_g_server->zonemgr,
-                                                 named_g_loopmgr, &dctx));
+                       dns_dyndb_createctx(mctx, hashinit, view,
+                                           named_g_server->zonemgr,
+                                           named_g_loopmgr, &dctx);
                }
 
                CHECK(configure_dyndb(dyndb, mctx, dctx));
@@ -6160,12 +6149,8 @@ create_view(const cfg_obj_t *vconfig, dns_viewlist_t *viewlist,
        }
        INSIST(view == NULL);
 
-       result = dns_view_create(named_g_mctx, named_g_loopmgr,
-                                named_g_dispatchmgr, viewclass, viewname,
-                                &view);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_view_create(named_g_mctx, named_g_loopmgr, named_g_dispatchmgr,
+                       viewclass, viewname, &view);
 
        isc_nonce_buf(view->secret, sizeof(view->secret));
 
@@ -6369,7 +6354,7 @@ configure_zone(const cfg_obj_t *config, const cfg_obj_t *zconfig,
                } else {
                        CHECK(dns_zonemgr_createzone(named_g_server->zonemgr,
                                                     &zone));
-                       CHECK(dns_zone_setorigin(zone, origin));
+                       dns_zone_setorigin(zone, origin);
                        dns_zone_setview(zone, view);
                        CHECK(dns_zonemgr_managezone(named_g_server->zonemgr,
                                                     zone));
@@ -6472,7 +6457,7 @@ configure_zone(const cfg_obj_t *config, const cfg_obj_t *zconfig,
                 * to create a new one.
                 */
                CHECK(dns_zonemgr_createzone(named_g_server->zonemgr, &zone));
-               CHECK(dns_zone_setorigin(zone, origin));
+               dns_zone_setorigin(zone, origin);
                dns_zone_setview(zone, view);
                CHECK(dns_zonemgr_managezone(named_g_server->zonemgr, zone));
                dns_zone_setstats(zone, named_g_server->zonestats);
@@ -6530,7 +6515,7 @@ configure_zone(const cfg_obj_t *config, const cfg_obj_t *zconfig,
                if (raw == NULL) {
                        dns_zone_create(&raw, dns_zone_getmem(zone),
                                        dns_zone_gettid(zone));
-                       CHECK(dns_zone_setorigin(raw, origin));
+                       dns_zone_setorigin(raw, origin);
                        dns_zone_setview(raw, view);
                        dns_zone_setstats(raw, named_g_server->zonestats);
                        CHECK(dns_zone_link(zone, raw));
@@ -6631,14 +6616,14 @@ add_keydata_zone(dns_view_t *view, const char *directory, isc_mem_t *mctx) {
 
        /* No existing keydata zone was found; create one */
        CHECK(dns_zonemgr_createzone(named_g_server->zonemgr, &zone));
-       CHECK(dns_zone_setorigin(zone, dns_rootname));
+       dns_zone_setorigin(zone, dns_rootname);
 
        defaultview = (strcmp(view->name, "_default") == 0);
        CHECK(isc_file_sanitize(
                directory, defaultview ? "managed-keys" : view->name,
                defaultview ? "bind" : "mkeys", filename, sizeof(filename)));
-       CHECK(dns_zone_setfile(zone, filename, dns_masterformat_text,
-                              &dns_master_style_default));
+       dns_zone_setfile(zone, filename, dns_masterformat_text,
+                        &dns_master_style_default);
 
        dns_zone_setview(zone, view);
        dns_zone_settype(zone, dns_zone_key);
@@ -8251,20 +8236,8 @@ load_configuration(const char *filename, named_server_t *server,
        /*
         * Configure sets of UDP query source ports.
         */
-       result = isc_portset_create(named_g_mctx, &v4portset);
-       if (result != ISC_R_SUCCESS) {
-               isc_log_write(NAMED_LOGCATEGORY_GENERAL, NAMED_LOGMODULE_SERVER,
-                             ISC_LOG_ERROR, "creating UDP/IPv4 port set: %s",
-                             isc_result_totext(result));
-               goto cleanup_bindkeys_parser;
-       }
-       result = isc_portset_create(named_g_mctx, &v6portset);
-       if (result != ISC_R_SUCCESS) {
-               isc_log_write(NAMED_LOGCATEGORY_GENERAL, NAMED_LOGMODULE_SERVER,
-                             ISC_LOG_ERROR, "creating UDP/IPv6 port set: %s",
-                             isc_result_totext(result));
-               goto cleanup_v4portset;
-       }
+       isc_portset_create(named_g_mctx, &v4portset);
+       isc_portset_create(named_g_mctx, &v6portset);
 
        result = isc_net_getudpportrange(AF_INET, &udpport_low, &udpport_high);
        if (result != ISC_R_SUCCESS) {
@@ -8272,7 +8245,7 @@ load_configuration(const char *filename, named_server_t *server,
                              ISC_LOG_ERROR,
                              "get the default UDP/IPv4 port range: %s",
                              isc_result_totext(result));
-               goto cleanup_v6portset;
+               goto cleanup_portsets;
        }
 
        isc_portset_addrange(v4portset, udpport_low, udpport_high);
@@ -8290,7 +8263,7 @@ load_configuration(const char *filename, named_server_t *server,
                              ISC_LOG_ERROR,
                              "get the default UDP/IPv6 port range: %s",
                              isc_result_totext(result));
-               goto cleanup_v6portset;
+               goto cleanup_portsets;
        }
        isc_portset_addrange(v6portset, udpport_low, udpport_high);
        if (!ns_server_getoption(server->sctx, NS_SERVER_DISABLE6)) {
@@ -8369,7 +8342,7 @@ load_configuration(const char *filename, named_server_t *server,
        } else {
                result = named_config_getport(config, "port", &listen_port);
                if (result != ISC_R_SUCCESS) {
-                       goto cleanup_v6portset;
+                       goto cleanup_portsets;
                }
        }
 
@@ -8416,13 +8389,13 @@ load_configuration(const char *filename, named_server_t *server,
 
                result = named_config_get(maps, "listen-on", &clistenon);
                if (result != ISC_R_SUCCESS) {
-                       goto cleanup_v6portset;
+                       goto cleanup_portsets;
                }
                result = listenlist_fromconfig(
                        clistenon, config, named_g_aclconfctx, named_g_mctx,
                        AF_INET, server->tlsctx_server_cache, &listenon);
                if (result != ISC_R_SUCCESS) {
-                       goto cleanup_v6portset;
+                       goto cleanup_portsets;
                }
                if (listenon != NULL) {
                        ns_interfacemgr_setlistenon4(server->interfacemgr,
@@ -8440,13 +8413,13 @@ load_configuration(const char *filename, named_server_t *server,
 
                result = named_config_get(maps, "listen-on-v6", &clistenon);
                if (result != ISC_R_SUCCESS) {
-                       goto cleanup_v6portset;
+                       goto cleanup_portsets;
                }
                result = listenlist_fromconfig(
                        clistenon, config, named_g_aclconfctx, named_g_mctx,
                        AF_INET6, server->tlsctx_server_cache, &listenon);
                if (result != ISC_R_SUCCESS) {
-                       goto cleanup_v6portset;
+                       goto cleanup_portsets;
                }
                if (listenon != NULL) {
                        ns_interfacemgr_setlistenon6(server->interfacemgr,
@@ -8476,7 +8449,7 @@ load_configuration(const char *filename, named_server_t *server,
                                      "unable to listen on any configured "
                                      "interfaces");
                        result = ISC_R_FAILURE;
-                       goto cleanup_v6portset;
+                       goto cleanup_portsets;
                }
        }
 
@@ -9283,10 +9256,8 @@ cleanup_keystorelist:
                dns_keystore_detach(&keystore);
        }
 
-cleanup_v6portset:
+cleanup_portsets:
        isc_portset_destroy(named_g_mctx, &v6portset);
-
-cleanup_v4portset:
        isc_portset_destroy(named_g_mctx, &v4portset);
 
 cleanup_bindkeys_parser:
index 9fff9b27d167b250c4d1a04b93b582e84ce53522..8538b15fa3de3a00a26a25fe51db58dba3c11bda 100644 (file)
@@ -53,10 +53,7 @@ named_tkeyctx_fromconfig(const cfg_obj_t *options, isc_mem_t *mctx,
        isc_buffer_t b;
        const cfg_obj_t *obj;
 
-       result = dns_tkeyctx_create(mctx, &tctx);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_tkeyctx_create(mctx, &tctx);
 
        obj = NULL;
        result = cfg_map_get(options, "tkey-domain", &obj);
index f2b178b872276512b6e9e672b959cf8afa8873ad..00be0fe2a6c945a8502de1fc4ac987878a887b0f 100644 (file)
@@ -978,9 +978,6 @@ named_zone_configure(const cfg_obj_t *config, const cfg_obj_t *vconfig,
        if (result == ISC_R_SUCCESS) {
                cpval = isc_mem_strdup(mctx, cfg_obj_asstring(obj));
        }
-       if (cpval == NULL) {
-               CHECK(ISC_R_NOMEMORY);
-       }
 
        obj = NULL;
        result = cfg_map_get(zoptions, "dlz", &obj);
@@ -1109,24 +1106,20 @@ named_zone_configure(const cfg_obj_t *config, const cfg_obj_t *vconfig,
                size_t signedlen = strlen(filename) + sizeof(SIGNED);
                char *signedname;
 
-               CHECK(dns_zone_setfile(raw, filename, masterformat,
-                                      masterstyle));
+               dns_zone_setfile(raw, filename, masterformat, masterstyle);
                signedname = isc_mem_get(mctx, signedlen);
 
                (void)snprintf(signedname, signedlen, "%s" SIGNED, filename);
-               result = dns_zone_setfile(zone, signedname,
-                                         dns_masterformat_raw, NULL);
+               dns_zone_setfile(zone, signedname, dns_masterformat_raw, NULL);
                isc_mem_put(mctx, signedname, signedlen);
-               CHECK(result);
        } else {
-               CHECK(dns_zone_setfile(zone, filename, masterformat,
-                                      masterstyle));
+               dns_zone_setfile(zone, filename, masterformat, masterstyle);
        }
 
        obj = NULL;
        result = cfg_map_get(zoptions, "journal", &obj);
        if (result == ISC_R_SUCCESS) {
-               CHECK(dns_zone_setjournal(mayberaw, cfg_obj_asstring(obj)));
+               dns_zone_setjournal(mayberaw, cfg_obj_asstring(obj));
        }
 
        /*
@@ -1627,7 +1620,7 @@ named_zone_configure(const cfg_obj_t *config, const cfg_obj_t *vconfig,
                result = named_config_get(maps, "key-directory", &obj);
                if (result == ISC_R_SUCCESS) {
                        filename = cfg_obj_asstring(obj);
-                       CHECK(dns_zone_setkeydirectory(zone, filename));
+                       dns_zone_setkeydirectory(zone, filename);
                }
                /* Also save a reference to the keystore list. */
                dns_zone_setkeystores(zone, keystorelist);
index 2491b9b9df30ef22293fb9508b746cb2bc496159..524c9ac3f3ec245f862732dc165ccd90e56f03d5 100644 (file)
@@ -766,14 +766,12 @@ set_source_ports(dns_dispatchmgr_t *manager) {
        in_port_t udpport_low, udpport_high;
        isc_result_t result;
 
-       result = isc_portset_create(gmctx, &v4portset);
-       check_result(result, "isc_portset_create (v4)");
+       isc_portset_create(gmctx, &v4portset);
        result = isc_net_getudpportrange(AF_INET, &udpport_low, &udpport_high);
        check_result(result, "isc_net_getudpportrange (v4)");
        isc_portset_addrange(v4portset, udpport_low, udpport_high);
 
-       result = isc_portset_create(gmctx, &v6portset);
-       check_result(result, "isc_portset_create (v6)");
+       isc_portset_create(gmctx, &v6portset);
        result = isc_net_getudpportrange(AF_INET6, &udpport_low, &udpport_high);
        check_result(result, "isc_net_getudpportrange (v6)");
        isc_portset_addrange(v6portset, udpport_low, udpport_high);
index d8c702236f95085e9f924d63bb84bf6983028038..247caea75eed06505b3aa13655e75bb79f3a9f11 100644 (file)
@@ -68,13 +68,7 @@ create_zone(sample_instance_t *const inst, dns_name_t *const name,
 
        dns_zone_create(&raw, inst->mctx, 0); /* FIXME: all zones are assigned
                                                 to loop 0 */
-       result = dns_zone_setorigin(raw, name);
-       if (result != ISC_R_SUCCESS) {
-               log_write(ISC_LOG_ERROR,
-                         "create_zone: dns_zone_setorigin -> %s\n",
-                         isc_result_totext(result));
-               goto cleanup;
-       }
+       dns_zone_setorigin(raw, name);
        dns_zone_setclass(raw, dns_rdataclass_in);
        dns_zone_settype(raw, dns_zone_primary);
        dns_zone_setdbtype(raw, 1, zone_argv);
index 0119bf718ab136a06cb4fa26519fdcb53f4c9686..c9b7f6a1198bdaf086f8098c47d5a08c0724345c 100644 (file)
@@ -278,7 +278,7 @@ main(int argc, char *argv[]) {
        RUNCHECK(dns_requestmgr_create(mctx, loopmgr, dispatchmgr, dispatchv4,
                                       NULL, &requestmgr));
 
-       RUNCHECK(dns_view_create(mctx, loopmgr, NULL, 0, "_test", &view));
+       dns_view_create(mctx, loopmgr, NULL, 0, "_test", &view);
 
        isc_loopmgr_setup(loopmgr, sendqueries, NULL);
        isc_loopmgr_teardown(loopmgr, teardown_view, view);
index fb8dcf4f6644896f78da260e549a1bbd959bfb3b..c4880041a9034b69311869ff4d94516888dd53ec 100644 (file)
@@ -2044,11 +2044,7 @@ set_source_ports(dns_dispatchmgr_t *manager) {
        in_port_t udpport_low, udpport_high;
        isc_result_t result;
 
-       result = isc_portset_create(mctx, &v4portset);
-       if (result != ISC_R_SUCCESS) {
-               fatal("isc_portset_create (v4) failed");
-       }
-
+       isc_portset_create(mctx, &v4portset);
        result = isc_net_getudpportrange(AF_INET, &udpport_low, &udpport_high);
        if (result != ISC_R_SUCCESS) {
                fatal("isc_net_getudpportrange (v4) failed");
@@ -2056,10 +2052,7 @@ set_source_ports(dns_dispatchmgr_t *manager) {
 
        isc_portset_addrange(v4portset, udpport_low, udpport_high);
 
-       result = isc_portset_create(mctx, &v6portset);
-       if (result != ISC_R_SUCCESS) {
-               fatal("isc_portset_create (v6) failed");
-       }
+       isc_portset_create(mctx, &v6portset);
        result = isc_net_getudpportrange(AF_INET6, &udpport_low, &udpport_high);
        if (result != ISC_R_SUCCESS) {
                fatal("isc_net_getudpportrange (v6) failed");
@@ -2103,7 +2096,7 @@ setup(void *arg ISC_ATTR_UNUSED) {
                mctx, loopmgr, dispatchmgr, have_ipv4 ? dispatchvx : NULL,
                have_ipv6 ? dispatchvx : NULL, &requestmgr));
 
-       RUNCHECK(dns_view_create(mctx, loopmgr, NULL, 0, "_mdig", &view));
+       dns_view_create(mctx, loopmgr, NULL, 0, "_mdig", &view);
 }
 
 /*% Main processing routine for mdig */
index c53d2ba1f80c1537a8505c9784306320d7e2662e..d856756a5c1fa9abc1d7e38463d228441ad814d7 100644 (file)
@@ -182,7 +182,7 @@ main(int argc, char *argv[]) {
        options = ISC_LEXOPT_EOL;
        isc_lex_setcomments(lex, ISC_LEXCOMMENT_DNSMASTERFILE);
 
-       RUNTIME_CHECK(isc_lex_openstream(lex, stdin) == ISC_R_SUCCESS);
+       isc_lex_openstream(lex, stdin);
 
        if (origin != NULL) {
                name = dns_fixedname_initname(&fixed);
index 99cdfe0e2a7a7b8d8ac48a0533655ed9628177e6..e1ec6096712e6d46f05adf6ca675e172390e93be 100644 (file)
@@ -173,13 +173,7 @@ LLVMFuzzerInitialize(int *argc ISC_ATTR_UNUSED, char ***argv ISC_ATTR_UNUSED) {
 
        isc_loopmgr_create(mctx, 1, &loopmgr);
 
-       result = dns_view_create(mctx, loopmgr, NULL, dns_rdataclass_in, "view",
-                                &view);
-       if (result != ISC_R_SUCCESS) {
-               fprintf(stderr, "dns_view_create failed: %s\n",
-                       isc_result_totext(result));
-               return 1;
-       }
+       dns_view_create(mctx, loopmgr, NULL, dns_rdataclass_in, "view", &view);
 
        dns_tsigkeyring_create(mctx, &ring);
        dns_tsigkeyring_create(mctx, &emptyring);
@@ -214,30 +208,12 @@ LLVMFuzzerInitialize(int *argc ISC_ATTR_UNUSED, char ***argv ISC_ATTR_UNUSED) {
 
        dns_zone_create(&zone, mctx, 0);
 
-       result = dns_zone_setorigin(zone, name);
-       if (result != ISC_R_SUCCESS) {
-               fprintf(stderr, "dns_zone_setorigin failed: %s\n",
-                       isc_result_totext(result));
-               return 1;
-       }
-
+       dns_zone_setorigin(zone, name);
        dns_zone_setclass(zone, view->rdclass);
        dns_zone_settype(zone, dns_zone_primary);
-
-       result = dns_zone_setkeydirectory(zone, wd);
-       if (result != ISC_R_SUCCESS) {
-               fprintf(stderr, "dns_zone_setkeydirectory failed: %s\n",
-                       isc_result_totext(result));
-               return 1;
-       }
-
-       result = dns_zone_setfile(zone, pathbuf, dns_masterformat_text,
-                                 &dns_master_style_default);
-       if (result != ISC_R_SUCCESS) {
-               fprintf(stderr, "dns_zone_setfile failed: %s\n",
-                       isc_result_totext(result));
-               return 1;
-       }
+       dns_zone_setkeydirectory(zone, wd);
+       dns_zone_setfile(zone, pathbuf, dns_masterformat_text,
+                        &dns_master_style_default);
 
        result = dns_zone_load(zone, false);
        if (result != ISC_R_SUCCESS) {
index d8d26d2d2e5147cd11448e5be8965cb4e4c428d1..39a81e1ef59f6ddf107855982fbdeee9f53c1542 100644 (file)
@@ -1492,10 +1492,7 @@ catz_process_primaries(dns_catz_zone_t *catz, dns_ipkeylist_t *ipkl,
                                        sizeof(sockaddr));
                        }
                } else {
-                       result = dns_ipkeylist_resize(mctx, ipkl, i + 1);
-                       if (result != ISC_R_SUCCESS) {
-                               return result;
-                       }
+                       dns_ipkeylist_resize(mctx, ipkl, i + 1);
 
                        ipkl->labels[i] = isc_mem_get(mctx,
                                                      sizeof(*ipkl->labels[0]));
@@ -1521,10 +1518,7 @@ catz_process_primaries(dns_catz_zone_t *catz, dns_ipkeylist_t *ipkl,
 
        rcount = dns_rdataset_count(value) + ipkl->count;
 
-       result = dns_ipkeylist_resize(mctx, ipkl, rcount);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_ipkeylist_resize(mctx, ipkl, rcount);
 
        for (result = dns_rdataset_first(value); result == ISC_R_SUCCESS;
             result = dns_rdataset_next(value))
index fa4ba6ee84c0a376a20611b772f62f970b352000..d9526b1bf0f0189da04d8cadb3ae22ee4bf01712 100644 (file)
@@ -149,20 +149,14 @@ setsourceports(isc_mem_t *mctx, dns_dispatchmgr_t *manager) {
        in_port_t udpport_low, udpport_high;
        isc_result_t result;
 
-       result = isc_portset_create(mctx, &v4portset);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
+       isc_portset_create(mctx, &v4portset);
        result = isc_net_getudpportrange(AF_INET, &udpport_low, &udpport_high);
        if (result != ISC_R_SUCCESS) {
                goto cleanup;
        }
        isc_portset_addrange(v4portset, udpport_low, udpport_high);
 
-       result = isc_portset_create(mctx, &v6portset);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
+       isc_portset_create(mctx, &v6portset);
        result = isc_net_getudpportrange(AF_INET6, &udpport_low, &udpport_high);
        if (result != ISC_R_SUCCESS) {
                goto cleanup;
@@ -210,11 +204,8 @@ createview(isc_mem_t *mctx, dns_rdataclass_t rdclass, isc_nm_t *nm,
        isc_result_t result;
        dns_view_t *view = NULL;
 
-       result = dns_view_create(mctx, loopmgr, dispatchmgr, rdclass,
-                                DNS_CLIENTVIEW_NAME, &view);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_view_create(mctx, loopmgr, dispatchmgr, rdclass,
+                       DNS_CLIENTVIEW_NAME, &view);
 
        /* Initialize view security roots */
        dns_view_initsecroots(view);
index 3dbdfb3dc0b721673539b5ff9ceccbd4e3ac457f..1525db56b64476849ad239a126777c5796d6fabc 100644 (file)
@@ -450,10 +450,7 @@ dns_dlz_writeablezone(dns_view_t *view, dns_dlzdb_t *dlzdb,
 
        /* Create it */
        dns_zone_create(&zone, view->mctx, 0);
-       result = dns_zone_setorigin(zone, origin);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
+       dns_zone_setorigin(zone, origin);
        dns_zone_setview(zone, view);
 
        dns_zone_setadded(zone, true);
index 81cc87fdd6139806f489c880d34ac654cd0cf36c..b5332fd42b5113f58e9a4a97a9d28b8203740fc6 100644 (file)
@@ -49,7 +49,7 @@ struct dns_dns64 {
        ISC_LINK(dns_dns64_t) link;
 };
 
-isc_result_t
+void
 dns_dns64_create(isc_mem_t *mctx, const isc_netaddr_t *prefix,
                 unsigned int prefixlen, const isc_netaddr_t *suffix,
                 dns_acl_t *clients, dns_acl_t *mapped, dns_acl_t *excluded,
@@ -100,7 +100,6 @@ dns_dns64_create(isc_mem_t *mctx, const isc_netaddr_t *prefix,
        dns64->mctx = NULL;
        isc_mem_attach(mctx, &dns64->mctx);
        *dns64p = dns64;
-       return ISC_R_SUCCESS;
 }
 
 void
index 7086b5d155324b1fdf7b0aea3ec98381a76c9f57..07879018b7099112e41c21d95b8afd9aad928d85 100644 (file)
@@ -70,9 +70,6 @@ dns_ds_fromkeyrdata(const dns_name_t *owner, dns_rdata_t *key,
        (void)dns_name_downcase(owner, name, NULL);
 
        md = isc_md_new();
-       if (md == NULL) {
-               return ISC_R_NOMEMORY;
-       }
 
        result = isc_md_init(md, md_type);
        if (result != ISC_R_SUCCESS) {
index e862a8555202e9afabcce2b71e2c7fb2ea727a39..6535c1d045b37b1e26bcbc88e9617ee0a7ce56c3 100644 (file)
@@ -265,7 +265,7 @@ dns_dyndb_cleanup(bool exiting) {
        }
 }
 
-isc_result_t
+void
 dns_dyndb_createctx(isc_mem_t *mctx, const void *hashinit, dns_view_t *view,
                    dns_zonemgr_t *zmgr, isc_loopmgr_t *loopmgr,
                    dns_dyndbctx_t **dctxp) {
@@ -290,8 +290,6 @@ dns_dyndb_createctx(isc_mem_t *mctx, const void *hashinit, dns_view_t *view,
        dctx->magic = DNS_DYNDBCTX_MAGIC;
 
        *dctxp = dctx;
-
-       return ISC_R_SUCCESS;
 }
 
 void
index 87902a3277831c616f81ee76ffe0c269331e5366..4cb4e9eae8a3c3f866113429596549ce412b2678 100644 (file)
@@ -342,7 +342,6 @@ dns_adb_createfind(dns_adb_t *adb, isc_loop_t *loop, isc_job_cb cb, void *cbarg,
  *\li  #ISC_R_NOMORE   Addresses might have been returned, but no events
  *                     will ever be posted for this context.  This is only
  *                     returned if task != NULL.
- *\li  #ISC_R_NOMEMORY insufficient resources
  *\li  #DNS_R_ALIAS    'name' is an alias for another name.
  *
  * Notes:
@@ -575,7 +574,6 @@ dns_adb_findaddrinfo(dns_adb_t *adb, const isc_sockaddr_t *sa,
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  *\li  #ISC_R_SHUTTINGDOWN
  */
 
index 2eaa705544adfa2e1d3d56c91525075f76938742..d659eba3808509f187f6f940318a12dc24b5ead3 100644 (file)
@@ -93,7 +93,6 @@ dns_cache_create(isc_loopmgr_t *loopmgr, dns_rdataclass_t rdclass,
  * Returns:
  *
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 void
@@ -190,7 +189,6 @@ dns_cache_flush(dns_cache_t *cache);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 isc_result_t
@@ -205,7 +203,6 @@ dns_cache_flushnode(dns_cache_t *cache, const dns_name_t *name, bool tree);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  *\li  other error returns.
  */
 
@@ -221,7 +218,6 @@ dns_cache_flushname(dns_cache_t *cache, const dns_name_t *name);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  *\li  other error returns.
  */
 
index 32aecc4e14a949b04214b933981431bfad1226db..f4fad39f2ae69385bbc097b76f28e3884fae8a07 100644 (file)
@@ -137,7 +137,6 @@ dns_catz_entry_new(isc_mem_t *mctx, const dns_name_t *domain);
  *
  * Returns:
  * \li ISC_R_SUCCESS on success
- * \li ISC_R_NOMEMORY on allocation failure
  */
 
 dns_catz_entry_t *
@@ -151,7 +150,6 @@ dns_catz_entry_copy(dns_catz_zone_t *catz, const dns_catz_entry_t *entry);
  *
  * Returns:
  * \li ISC_R_SUCCESS on success
- * \li ISC_R_NOMEMORY on allocation failure
  */
 
 void
index c898957b11a3e3f6edee0db95823cdbcab259c8d..fd8ab3323a1ef1823affd9da8fae9c7ca4df550e 100644 (file)
@@ -370,7 +370,6 @@ dns_db_create(isc_mem_t *mctx, const char *db_type, const dns_name_t *origin,
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  * \li #ISC_R_NOTFOUND                         db_type not found
  *
  * \li Many other errors are possible, depending on what db_type was
@@ -501,7 +500,6 @@ dns_db_beginload(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  *
  * \li Other results are possible, depending upon the database
  *     implementation used, syntax errors in the master file, etc.
@@ -527,7 +525,6 @@ dns_db_endload(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  *
  * \li Other results are possible, depending upon the database
  *     implementation used, syntax errors in the master file, etc.
@@ -557,7 +554,6 @@ dns_db_load(dns_db_t *db, const char *filename, dns_masterformat_t format,
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  *
  * \li Other results are possible, depending upon the database
  *     implementation used, syntax errors in the master file, etc.
@@ -577,7 +573,6 @@ dns_db_dump(dns_db_t *db, dns_dbversion_t *version, const char *filename);
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  *
  * \li Other results are possible, depending upon the database
  *     implementation used, OS file errors, etc.
@@ -622,7 +617,6 @@ dns_db_newversion(dns_db_t *db, dns_dbversion_t **versionp);
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  *
  * \li Other results are possible, depending upon the database
  *     implementation used.
@@ -731,7 +725,6 @@ dns__db_findnodeext(dns_db_t *db, const dns_name_t *name, bool create,
  *
  * \li #ISC_R_SUCCESS
  * \li #ISC_R_NOTFOUND                 If !create and name not found.
- * \li #ISC_R_NOMEMORY                 Can only happen if create is true.
  *
  * \li Other results are possible, depending upon the database
  *     implementation used.
@@ -928,8 +921,6 @@ dns__db_findext(dns_db_t *db, const dns_name_t *name, dns_dbversion_t *version,
  *
  *     Error results:
  *
- *     \li     #ISC_R_NOMEMORY
- *
  *     \li     #DNS_R_BADDB                    Data that is required to be
  *                                             present in the DB, e.g. an NSEC
  *                                             record in a secure zone, is not
@@ -1105,7 +1096,6 @@ dns_db_createiterator(dns_db_t *db, unsigned int options,
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  */
 
 /***
@@ -1274,8 +1264,8 @@ dns__db_addrdataset(dns_db_t *db, dns_dbnode_t *node, dns_dbversion_t *version,
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #DNS_R_UNCHANGED                        The operation did not change
- * anything. \li       #ISC_R_NOMEMORY \li     #DNS_R_NOTEXACT
+ * \li #DNS_R_UNCHANGED        The operation did not change anything.
+ * \li #DNS_R_NOTEXACT
  *
  * \li Other results are possible, depending upon the database
  *     implementation used.
@@ -1455,7 +1445,6 @@ dns_db_register(const char *name, dns_dbcreatefunc_t create, void *driverarg,
  *
  * Returns:
  * \li #ISC_R_SUCCESS  The registration succeeded
- * \li #ISC_R_NOMEMORY Out of memory
  * \li #ISC_R_EXISTS   A database implementation with the same name exists
  *
  * Ensures:
@@ -1568,7 +1557,6 @@ dns__db_findnsec3node(dns_db_t *db, const dns_name_t *name, bool create,
  *
  * \li #ISC_R_SUCCESS
  * \li #ISC_R_NOTFOUND                 If !create and name not found.
- * \li #ISC_R_NOMEMORY                 Can only happen if create is true.
  *
  * \li Other results are possible, depending upon the database
  *     implementation used.
@@ -1586,7 +1574,6 @@ dns_db_setsigningtime(dns_db_t *db, dns_rdataset_t *rdataset,
  *
  * Returns:
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  * \li #ISC_R_NOTIMPLEMENTED - Not supported by this DB implementation.
  */
 
index 95a58f990a7aab5c44b18cb29ee24dbb986faca9..86f00b4a6aceaace07e6ab170b8683763b4ad510 100644 (file)
@@ -284,7 +284,5 @@ dns_diff_print(const dns_diff_t *diff, FILE *file);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *\li  #ISC_R_UNEXPECTED
  *\li  any error from dns_rdataset_totext()
  */
index 1e2246b10b9af0e553dedffb1f82aac69cb427f0..acedfb7816919c01f5390e7190b72e8729d807df 100644 (file)
@@ -348,7 +348,6 @@ dns_dispatch_add(dns_dispatch_t *disp, isc_loop_t *loop,
  * Returns:
  *
  *\li  ISC_R_SUCCESS           -- all is well.
- *\li  ISC_R_NOMEMORY          -- memory could not be allocated.
  *\li  ISC_R_NOMORE            -- no more message ids can be allocated
  *                                for this destination.
  */
index 7e10e5bfe0ff5227680bd188498bbe02e568b872..22331bb6dca3a6fb6a049534a03990ffd6e4a8a3 100644 (file)
@@ -38,7 +38,7 @@
 #define DNS_DNS64_RECURSIVE 0x01 /* Recursive query. */
 #define DNS_DNS64_DNSSEC    0x02 /* DNSSEC sensitive query. */
 
-isc_result_t
+void
 dns_dns64_create(isc_mem_t *mctx, const isc_netaddr_t *prefix,
                 unsigned int prefixlen, const isc_netaddr_t *suffix,
                 dns_acl_t *client, dns_acl_t *mapped, dns_acl_t *excluded,
@@ -79,10 +79,6 @@ dns_dns64_create(isc_mem_t *mctx, const isc_netaddr_t *prefix,
  *     'client'        to be NULL or a valid acl.
  *     'mapped'        to be NULL or a valid acl.
  *     'excluded'      to be NULL or a valid acl.
- *
- * Returns:
- *     ISC_R_SUCCESS
- *     ISC_R_NOMEMORY
  */
 
 void
index daacd69a5c368d9bb123871992485a0a352dca75..56f006f66432b290dda91467518a70ea3a0eb59b 100644 (file)
@@ -89,7 +89,6 @@ dns_dnssec_keyfromrdata(const dns_name_t *name, const dns_rdata_t *rdata,
  *
  *     Returns:
  *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
  *\li          DST_R_INVALIDPUBLICKEY
  *\li          various errors from dns_name_totext
  */
@@ -131,7 +130,6 @@ dns_dnssec_sign(const dns_name_t *name, dns_rdataset_t *set, dst_key_t *key,
  *
  *     Returns:
  *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
  *\li          #ISC_R_NOSPACE
  *\li          #DNS_R_INVALIDTIME - the expiration is before the inception
  *\li          #DNS_R_KEYUNAUTHORIZED - the key cannot sign this data (either
@@ -162,7 +160,6 @@ dns_dnssec_verify(const dns_name_t *name, dns_rdataset_t *set, dst_key_t *key,
  *
  *     Returns:
  *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
  *\li          #DNS_R_FROMWILDCARD - the signature is valid and is from
  *                     a wildcard expansion.  dns_dnssec_verify2() only.
  *                     'wild' contains the name of the wildcard if non-NULL.
@@ -201,7 +198,6 @@ dns_dnssec_signmessage(dns_message_t *msg, dst_key_t *key);
  *
  *     Returns:
  *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
  *\li          DST_R_*
  */
 
@@ -223,7 +219,6 @@ dns_dnssec_verifymessage(isc_buffer_t *source, dns_message_t *msg,
  *
  *     Returns:
  *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
  *\li          #ISC_R_NOTFOUND - no SIG(0) was found
  *\li          #DNS_R_SIGINVALID - the SIG record is not well-formed or
  *                                was not generated by the key.
@@ -298,7 +293,6 @@ dns_dnssec_findmatchingkeys(const dns_name_t *origin, dns_kasp_t *kasp,
  *     Returns:
  *\li          #ISC_R_SUCCESS
  *\li          #ISC_R_NOTFOUND
- *\li          #ISC_R_NOMEMORY
  *\li          any error returned by dns_name_totext(), isc_dir_open(), or
  *              dst_key_fromnamedfile()
  *
index acea3f0dcd07be5d536a9050d48c471fb061dd5b..a54f710533c1d80904f3b5ac4da9cd3bcdc52c93 100644 (file)
@@ -70,7 +70,6 @@ dns_dyndb_register_t(isc_mem_t *mctx, const char *name, const char *parameters,
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  *\li  Other errors are possible
  */
 
@@ -114,7 +113,6 @@ dns_dyndb_load(const char *libname, const char *name, const char *parameters,
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  *\li  Other errors are possible
  */
 
@@ -127,7 +125,7 @@ dns_dyndb_cleanup(bool exiting);
  * as opposed to merely being reconfigured.
  */
 
-isc_result_t
+void
 dns_dyndb_createctx(isc_mem_t *mctx, const void *hashinit, dns_view_t *view,
                    dns_zonemgr_t *zmgr, isc_loopmgr_t *loopmgr,
                    dns_dyndbctx_t **dctxp);
@@ -138,11 +136,6 @@ dns_dyndb_createctx(isc_mem_t *mctx, const void *hashinit, dns_view_t *view,
  * etc). This structure is expected to last only until all dyndb
  * modules have been loaded and initialized; after that it will be
  * destroyed with dns_dyndb_destroyctx().
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *\li  Other errors are possible
  */
 
 void
index ca5450f3fdd2f1976e92e1a9d57d6cdba421eec3..1fbb7f9099853dcb8908f43cf9dae099a9e0cdbb 100644 (file)
@@ -73,7 +73,7 @@ dns_fwdtable_add(dns_fwdtable_t *fwdtable, const dns_name_t *name,
  *
  * Returns:
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
+ * \li  other errors from dns_qp_insert()
  */
 
 isc_result_t
index 4087b173810fca849a6e76c663fc059f5e853b4c..dd4245bd4a50609aed0b74f2cb42cb6533e30f41 100644 (file)
@@ -71,7 +71,8 @@ dns_ipkeylist_copy(isc_mem_t *mctx, const dns_ipkeylist_t *src,
  *\li  #ISC_R_SUCCESS  -- success
  *\li  any other value -- failure
  */
-isc_result_t
+
+void
 dns_ipkeylist_resize(isc_mem_t *mctx, dns_ipkeylist_t *ipkl, unsigned int n);
 /*%<
  * Resize ipkl to contain n elements. Size (count) is not changed, and the
@@ -81,8 +82,4 @@ dns_ipkeylist_resize(isc_mem_t *mctx, dns_ipkeylist_t *ipkl, unsigned int n);
  * \li 'mctx' to be a valid memory context.
  * \li 'ipk' to be non NULL
  * \li 'n' >= ipkl->count
- *
- * Returns:
- * \li #ISC_R_SUCCESS if success
- * \li #ISC_R_NOMEMORY if there's no memory, ipkeylist is left untouched
  */
index fd8ec896f273e0e60aacbc5a336bd864d9c586cd..609edbe949e4d6bac856bb9449b39b02071a99f4 100644 (file)
@@ -136,7 +136,7 @@ struct dns_kasp {
 #define DNS_KASP_KEY_ROLE_KSK 0x01
 #define DNS_KASP_KEY_ROLE_ZSK 0x02
 
-isc_result_t
+void
 dns_kasp_create(isc_mem_t *mctx, const char *name, dns_kasp_t **kaspp);
 /*%<
  * Create a KASP.
@@ -150,11 +150,6 @@ dns_kasp_create(isc_mem_t *mctx, const char *name, dns_kasp_t **kaspp);
  *\li  kaspp != NULL && *kaspp == NULL
  *
  * Returns:
- *
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *
- *\li  Other errors are possible.
  */
 
 void
@@ -566,13 +561,6 @@ dns_kasp_keys(dns_kasp_t *kasp);
  * Requires:
  *
  *\li   'kasp' is a valid, frozen kasp.
- *
- * Returns:
- *
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *
- *\li  Other errors are possible.
  */
 
 bool
@@ -600,7 +588,7 @@ dns_kasp_addkey(dns_kasp_t *kasp, dns_kasp_key_t *key);
  *\li   'key' is not NULL.
  */
 
-isc_result_t
+void
 dns_kasp_key_create(dns_kasp_t *kasp, dns_kasp_key_t **keyp);
 /*%<
  * Create a key inside a KASP.
@@ -610,13 +598,6 @@ dns_kasp_key_create(dns_kasp_t *kasp, dns_kasp_key_t **keyp);
  *\li   'kasp' is a valid kasp.
  *
  *\li  keyp != NULL && *keyp == NULL
- *
- * Returns:
- *
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *
- *\li  Other errors are possible.
  */
 
 void
@@ -882,13 +863,6 @@ dns_kasp_digests(dns_kasp_t *kasp);
  * Requires:
  *
  *\li   'kasp' is a valid, frozen kasp.
- *
- * Returns:
- *
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *
- *\li  Other errors are possible.
  */
 
 void
index 2dd27389918e0823c397e336d38fe1b28f8d0a99..fc9898ed8fe4b2bffdaf7ecb36b1fb86078d87a8 100644 (file)
@@ -59,7 +59,7 @@ struct dns_keystore {
 
 #define DNS_KEYSTORE_KEYDIRECTORY "key-directory"
 
-isc_result_t
+void
 dns_keystore_create(isc_mem_t *mctx, const char *name, dns_keystore_t **kspp);
 /*%<
  * Create a key store.
@@ -71,13 +71,6 @@ dns_keystore_create(isc_mem_t *mctx, const char *name, dns_keystore_t **kspp);
  *\li  'name' is a valid C string.
  *
  *\li  kspp != NULL && *kspp == NULL
- *
- * Returns:
- *
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *
- *\li  Other errors are possible.
  */
 
 const char *
index bbcbc613ce6d9f9cb99cb960d6a52f96cd9dfde5..59060d990e680175260f75c4927f195a90b4c8aa 100644 (file)
@@ -177,7 +177,6 @@ dns_master_loadfileasync(const char *master_file, dns_name_t *top,
  *\li  ISC_R_SUCCESS upon successfully loading the master file.
  *\li  DNS_R_SEENINCLUDE upon successfully loading the master file with
  *             a $INCLUDE statement.
- *\li  ISC_R_NOMEMORY out of memory.
  *\li  ISC_R_UNEXPECTEDEND expected to be able to read a input token and
  *             there was not one.
  *\li  ISC_R_UNEXPECTED
index 8d86e9364b466fff0486e5089f32f4c58bc46235..eb6e98ad368c2ba1a4537e5a4667aba6c1ae55c5 100644 (file)
@@ -271,7 +271,6 @@ dns_master_dumptostream(isc_mem_t *mctx, dns_db_t *db, dns_dbversion_t *version,
  *
  * Returns:
  *\li  ISC_R_SUCCESS
- *\li  ISC_R_NOMEMORY
  *\li  Any database or rrset iterator error.
  *\li  Any dns_rdata_totext() error code.
  */
@@ -303,7 +302,6 @@ dns_master_dump(isc_mem_t *mctx, dns_db_t *db, dns_dbversion_t *version,
  *
  * Returns:
  *\li  ISC_R_SUCCESS
- *\li  ISC_R_NOMEMORY
  *\li  Any database or rrset iterator error.
  *\li  Any dns_rdata_totext() error code.
  */
index e8ea327ba0c3253c3088f2a585ed41cbef3d5fc1..283c9b947a2a36a6ada8a481e7746aad75a6c76c 100644 (file)
@@ -584,7 +584,6 @@ dns_message_parse(dns_message_t *msg, isc_buffer_t *source,
  *
  * Returns:
  *\li  #ISC_R_SUCCESS          -- all is well
- *\li  #ISC_R_NOMEMORY         -- no memory
  *\li  #DNS_R_RECOVERABLE      -- the message parsed properly, but contained
  *                                errors.
  *\li  Many other errors possible XXXMLG
@@ -1207,7 +1206,7 @@ dns_message_getquerytsig(dns_message_t *msg, isc_mem_t *mctx,
  * Returns:
  *
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
+ *\li  any return code from dns_rdataset_first()
  *
  * Ensures:
  *\li  'tsig' points to NULL or an allocated buffer which must be freed
@@ -1441,10 +1440,8 @@ dns_message_buildopt(dns_message_t *msg, dns_rdataset_t **opt,
  * \li   opt to be a non NULL and *opt to be NULL.
  *
  * Returns:
- * \li  ISC_R_SUCCESS on success.
- * \li  ISC_R_NOMEMORY
+ * \li  ISC_R_SUCCESS
  * \li  ISC_R_NOSPACE
- * \li  other.
  */
 
 void
index 3e405846e218c3952ed26c8b2b7b85a5d3d5bd5c..68e4bc495e598627ad5b0488d9a4d9a0fa0c0230 100644 (file)
@@ -1217,8 +1217,6 @@ dns_name_tostring(const dns_name_t *source, char **target, isc_mem_t *mctx);
  * Returns:
  *
  *\li  ISC_R_SUCCESS
- *\li  ISC_R_NOMEMORY
- *
  *\li  Any error that dns_name_totext() can return.
  */
 
index 54261257bcd63e2e36bc4e203028f2fb6a56f1ce..f628ad5778775682510c107f466bf82680d86803 100644 (file)
@@ -19,7 +19,7 @@
 
 #include <dns/types.h>
 
-isc_result_t
+void
 dns_order_create(isc_mem_t *mctx, dns_order_t **orderp);
 /*%<
  * Create a order object.
@@ -27,13 +27,9 @@ dns_order_create(isc_mem_t *mctx, dns_order_t **orderp);
  * Requires:
  * \li 'orderp' to be non NULL and '*orderp == NULL'.
  *\li  'mctx' to be valid.
- *
- * Returns:
- *\li  ISC_R_SUCCESS
- *\li  ISC_R_NOMEMORY
  */
 
-isc_result_t
+void
 dns_order_add(dns_order_t *order, const dns_name_t *name,
              dns_rdatatype_t rdtype, dns_rdataclass_t rdclass,
              unsigned int mode);
@@ -45,10 +41,6 @@ dns_order_add(dns_order_t *order, const dns_name_t *name,
  *\li  'name' to be valid.
  *\li  'mode' to be one of #DNS_RDATASETATTR_RANDOMIZE,
  *             #DNS_RDATASETATTR_FIXEDORDER or zero (#DNS_RDATASETATTR_CYCLIC).
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 unsigned int
index 1e676113004500272666bd055f65359311f4a62d..e2bcc667257c321005619558b109c96a4406fed9 100644 (file)
@@ -45,7 +45,7 @@
  *** Functions
  ***/
 
-isc_result_t
+void
 dns_peerlist_new(isc_mem_t *mem, dns_peerlist_t **list);
 
 void
index 00ed2f68956313296a7294502a762665773ea3c8..82660b5d9d628cbd5141c4f4b28704043a2aef11 100644 (file)
@@ -182,8 +182,9 @@ dns_rdataslab_fromrdataset(dns_rdataset_t *rdataset, isc_mem_t *mctx,
  *
  * Returns:
  *\li  ISC_R_SUCCESS           - successful completion
- *\li  ISC_R_NOMEMORY          - no memory.
- *\li  XXX others
+ *\li  ISC_R_NOSPACE           - more than 64k RRs
+ *\li  DNS_R_TOOMANYRECORDS    - more than max-records-per-rrset RRs
+ *\li  DNS_R_SINGLETON         - singleton type has more than one RR
  */
 
 unsigned int
index 62ffb9885ff1adea4dc53b776e07a242ac73d5f5..6a8f715afeafc5c74feb13d901b711b8adecb805 100644 (file)
@@ -408,7 +408,6 @@ dns_resolver_disable_algorithm(dns_resolver_t *resolver, const dns_name_t *name,
  * Returns:
  *\li  #ISC_R_SUCCESS
  *\li  #ISC_R_RANGE
- *\li  #ISC_R_NOMEMORY
  */
 
 isc_result_t
@@ -421,7 +420,6 @@ dns_resolver_disable_ds_digest(dns_resolver_t *resolver, const dns_name_t *name,
  * Returns:
  *\li  #ISC_R_SUCCESS
  *\li  #ISC_R_RANGE
- *\li  #ISC_R_NOMEMORY
  */
 
 bool
index 606dd31af91efc4d75fcfd11a7d4a9bd3183f252..be90ff1acddcf4f88168c613c0ec3dc54a22fdf8 100644 (file)
@@ -80,7 +80,7 @@ dns_rriterator_init(dns_rriterator_t *it, dns_db_t *db, dns_dbversion_t *ver,
  * Returns:
  *
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
+ * \li  any error from dns_db_createiterator()
  */
 
 isc_result_t
index d71067bbf1a7e98d513dee06d5dbea6865fcce13..e39104361667000285b421617a44f89557e064a8 100644 (file)
@@ -60,10 +60,6 @@ dns_ssutable_create(isc_mem_t *mctx, dns_ssutable_t **table);
  *     Requires:
  *\li          'mctx' is a valid memory context
  *\li          'table' is not NULL, and '*table' is NULL
- *
- *     Returns:
- *\li          ISC_R_SUCCESS
- *\li          ISC_R_NOMEMORY
  */
 
 void
@@ -131,10 +127,6 @@ dns_ssutable_addrule(dns_ssutable_t *table, bool grant,
  *\li          'name' is a valid absolute name
  *\li          If 'ntypes' > 0, 'types' must not be NULL
  *\li          'debug' must not be NULL
- *
- *     Returns:
- *\li          ISC_R_SUCCESS
- *\li          ISC_R_NOMEMORY
  */
 
 bool
index 5c58231c64de542bc297a3c9268f3123e2f2ca36..8b9c76f28d7ea2bbb6365bded07694cfb7d80611 100644 (file)
@@ -37,20 +37,15 @@ struct dns_tkeyctx {
        char             *gssapi_keytab;
 };
 
-isc_result_t
+void
 dns_tkeyctx_create(isc_mem_t *mctx, dns_tkeyctx_t **tctxp);
 /*%<
  *     Create an empty TKEY context.
  *
  *     Requires:
  *\li          'mctx' is not NULL
- *\li          'tctx' is not NULL
- *\li          '*tctx' is NULL
- *
- *     Returns
- *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
- *\li          return codes from dns_name_fromtext()
+ *\li          'tctxp' is not NULL
+ *\li          '*tctxp' is NULL
  */
 
 void
index 7fe3dcefd559d1f70b9499be7dd445fa78a4aec9..5b4e17d90ed1a31463f178454620ceeaf01500c1 100644 (file)
@@ -158,7 +158,6 @@ dns_tsigkey_createfromkey(const dns_name_t *name, dst_algorithm_t algorithm,
  *\li          #ISC_R_SUCCESS
  *\li          #ISC_R_EXISTS - a key with this name already exists
  *\li          #ISC_R_NOTIMPLEMENTED - algorithm is not implemented
- *\li          #ISC_R_NOMEMORY
  */
 
 void
@@ -183,7 +182,6 @@ dns_tsig_sign(dns_message_t *msg);
  *
  *     Returns:
  *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
  *\li          #ISC_R_NOSPACE
  *\li          #DNS_R_EXPECTEDTSIG
  *                     - this is a response & msg->querytsig is NULL
@@ -205,7 +203,6 @@ dns_tsig_verify(isc_buffer_t *source, dns_message_t *msg,
  *
  *     Returns:
  *\li          #ISC_R_SUCCESS
- *\li          #ISC_R_NOMEMORY
  *\li          #DNS_R_EXPECTEDTSIG - A TSIG was expected but not seen
  *\li          #DNS_R_UNEXPECTEDTSIG - A TSIG was seen but not expected
  *\li          #DNS_R_TSIGERRORSET - the TSIG verified but ->error was set
index edab9d2be1dd637bc349d00c4ca5b37c05f216c0..aa87fa6c9a3385fd1bb7bdba7cae65b75e08681c 100644 (file)
@@ -259,7 +259,7 @@ struct dns_view {
 #endif /* __OpenBSD__ */
 #endif /* HAVE_LMDB */
 
-isc_result_t
+void
 dns_view_create(isc_mem_t *mctx, isc_loopmgr_t *loopmgr,
                dns_dispatchmgr_t *dispmgr, dns_rdataclass_t rdclass,
                const char *name, dns_view_t **viewp);
@@ -280,13 +280,6 @@ dns_view_create(isc_mem_t *mctx, isc_loopmgr_t *loopmgr,
  *\li  'name' is a valid C string.
  *
  *\li  viewp != NULL && *viewp == NULL
- *
- * Returns:
- *
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *
- *\li  Other errors are possible.
  */
 
 void
@@ -869,7 +862,7 @@ dns_view_flushcache(dns_view_t *view, bool fixuponly);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
+ *     other returns are failures.
  */
 
 isc_result_t
index 7866ac43a064955a91f047c118a92543ff5b6762..4524142525fe41dec93c93cc82d4eed2261bcf2c 100644 (file)
@@ -266,7 +266,7 @@ dns_zone_setviewrevert(dns_zone_t *zone);
  *\li  'zone' to be a valid zone.
  */
 
-isc_result_t
+void
 dns_zone_setorigin(dns_zone_t *zone, const dns_name_t *origin);
 /*%<
  *     Sets the zones origin to 'origin'.
@@ -277,7 +277,6 @@ dns_zone_setorigin(dns_zone_t *zone, const dns_name_t *origin);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 dns_name_t *
@@ -289,7 +288,7 @@ dns_zone_getorigin(dns_zone_t *zone);
  *\li  'zone' to be a valid zone.
  */
 
-isc_result_t
+void
 dns_zone_setfile(dns_zone_t *zone, const char *file, dns_masterformat_t format,
                 const dns_master_style_t *style);
 /*%<
@@ -303,10 +302,6 @@ dns_zone_setfile(dns_zone_t *zone, const char *file, dns_masterformat_t format,
  *
  * Require:
  *\li  'zone' to be a valid zone.
- *
- * Returns:
- *\li  #ISC_R_NOMEMORY
- *\li  #ISC_R_SUCCESS
  */
 
 const char *
@@ -321,7 +316,7 @@ dns_zone_getfile(dns_zone_t *zone);
  *\li  Pointer to null-terminated file name, or NULL.
  */
 
-isc_result_t
+void
 dns_zone_setstream(dns_zone_t *zone, const FILE *stream,
                   dns_masterformat_t format, const dns_master_style_t *style);
 /*%<
@@ -332,10 +327,6 @@ dns_zone_setstream(dns_zone_t *zone, const FILE *stream,
  *\li  'stream' to be a valid and open FILE *.
  *\li  'zone->masterfile' to be NULL, since we should load data either from
  *     'stream' or from a master file, but not both.
- *
- * Returns:
- *\li  #ISC_R_NOMEMORY
- *\li  #ISC_R_SUCCESS
  */
 
 void
@@ -495,7 +486,6 @@ dns_zone_asyncload(dns_zone_t *zone, bool newonly, dns_zt_callback_t done,
  *\li  #ISC_R_ALREADYRUNNING
  *\li  #ISC_R_SUCCESS
  *\li  #ISC_R_FAILURE
- *\li  #ISC_R_NOMEMORY
  */
 
 bool
@@ -578,7 +568,7 @@ dns_zone_setdbtype(dns_zone_t *zone, unsigned int dbargc,
  *\li  'dbargv' to point to dbargc NULL-terminated strings
  */
 
-isc_result_t
+void
 dns_zone_getdbtype(dns_zone_t *zone, char ***argv, isc_mem_t *mctx);
 /*%<
  *     Returns the current dbtype.  isc_mem_free() should be used
@@ -588,10 +578,6 @@ dns_zone_getdbtype(dns_zone_t *zone, char ***argv, isc_mem_t *mctx);
  *\li  'zone' to be a valid zone.
  *\li  'argv' to be non NULL and *argv to be NULL.
  *\li  'mctx' to be valid.
- *
- * Returns:
- *\li  #ISC_R_NOMEMORY
- *\li  #ISC_R_SUCCESS
  */
 
 void
@@ -681,11 +667,6 @@ dns_zone_setprimaries(dns_zone_t *zone, isc_sockaddr_t *addresses,
  *\li  'keynames' array of dns_name_t's for tsig keys or NULL.
  *
  *\li  If 'addresses' is NULL then 'count' must be zero.
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *\li      Any result dns_name_dup() can return, if keynames!=NULL
  */
 
 void
@@ -702,11 +683,6 @@ dns_zone_setparentals(dns_zone_t *zone, isc_sockaddr_t *addresses,
  *\li  'keynames' array of dns_name_t's for tsig keys or NULL.
  *
  *\li  If 'addresses' is NULL then 'count' must be zero.
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
- *\li      Any result dns_name_dup() can return, if keynames!=NULL
  */
 
 void
@@ -724,10 +700,6 @@ dns_zone_setalsonotify(dns_zone_t *zone, isc_sockaddr_t *addresses,
  *\li  'zone' to be a valid zone.
  *\li  'addresses' to be non-NULL if count != 0.
  *\li  'count' to be the number of notifiees.
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 void
@@ -1303,7 +1275,7 @@ dns_zone_getmaxxfrout(dns_zone_t *zone);
  *\li  'zone' to be valid initialised zone.
  */
 
-isc_result_t
+void
 dns_zone_setjournal(dns_zone_t *zone, const char *myjournal);
 /*%<
  * Sets the filename used for journaling updates / IXFR transfers.
@@ -1313,10 +1285,6 @@ dns_zone_setjournal(dns_zone_t *zone, const char *myjournal);
  *
  * Requires:
  *\li  'zone' to be a valid zone.
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 char *
@@ -1607,7 +1575,6 @@ dns_zone_forwardupdate(dns_zone_t *zone, dns_message_t *msg,
  *\li  'callback' to be non NULL.
  * Returns:
  *\li  #ISC_R_SUCCESS if the message has been forwarded,
- *\li  #ISC_R_NOMEMORY
  *\li  Others
  */
 
@@ -1643,7 +1610,7 @@ dns_zone_first(dns_zonemgr_t *zmgr, dns_zone_t **first);
  *     (result ISC_R_NOMORE).
  */
 
-isc_result_t
+void
 dns_zone_setkeydirectory(dns_zone_t *zone, const char *directory);
 /*%<
  *     Sets the name of the directory where private keys used for
@@ -1651,10 +1618,6 @@ dns_zone_setkeydirectory(dns_zone_t *zone, const char *directory);
  *
  * Require:
  *\li  'zone' to be a valid zone.
- *
- * Returns:
- *\li  #ISC_R_NOMEMORY
- *\li  #ISC_R_SUCCESS
  */
 
 const char *
index 33866d66821be1189a1d87b89683192ee056d484..5eb60380595582a4844087a59dd95c3b25582915 100644 (file)
@@ -229,7 +229,9 @@ dst_context_create(dst_key_t *key, isc_mem_t *mctx, isc_logcategory_t category,
  *
  * Returns:
  * \li ISC_R_SUCCESS
- * \li ISC_R_NOMEMORY
+ * \li DST_R_UNSUPPORTEDALG
+ * \li DST_R_NULLKEY
+ * \li Other errors are possible.
  *
  * Ensures:
  * \li *dctxp will contain a usable context.
@@ -1053,7 +1055,6 @@ dst_key_dump(dst_key_t *key, isc_mem_t *mctx, char **buffer, int *length);
  *
  * Returns:
  *     ISC_R_SUCCESS
- *     ISC_R_NOMEMORY
  *     ISC_R_NOTIMPLEMENTED
  *     others.
  */
index b24634ef75fd04e7041d10f0a91468828c27b4b0..7b1dc7ac61ea25fa392d222238216612b0754a04 100644 (file)
@@ -98,7 +98,6 @@ dns_ipkeylist_clear(isc_mem_t *mctx, dns_ipkeylist_t *ipkl) {
 isc_result_t
 dns_ipkeylist_copy(isc_mem_t *mctx, const dns_ipkeylist_t *src,
                   dns_ipkeylist_t *dst) {
-       isc_result_t result = ISC_R_SUCCESS;
        uint32_t i;
 
        REQUIRE(dst != NULL);
@@ -109,10 +108,7 @@ dns_ipkeylist_copy(isc_mem_t *mctx, const dns_ipkeylist_t *src,
                return ISC_R_SUCCESS;
        }
 
-       result = dns_ipkeylist_resize(mctx, dst, src->count);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_ipkeylist_resize(mctx, dst, src->count);
 
        memmove(dst->addrs, src->addrs, src->count * sizeof(isc_sockaddr_t));
 
@@ -164,13 +160,13 @@ dns_ipkeylist_copy(isc_mem_t *mctx, const dns_ipkeylist_t *src,
        return ISC_R_SUCCESS;
 }
 
-isc_result_t
+void
 dns_ipkeylist_resize(isc_mem_t *mctx, dns_ipkeylist_t *ipkl, unsigned int n) {
        REQUIRE(ipkl != NULL);
        REQUIRE(n > ipkl->count);
 
        if (n <= ipkl->allocated) {
-               return ISC_R_SUCCESS;
+               return;
        }
 
        ipkl->addrs = isc_mem_creget(mctx, ipkl->addrs, ipkl->allocated, n,
@@ -185,5 +181,4 @@ dns_ipkeylist_resize(isc_mem_t *mctx, dns_ipkeylist_t *ipkl, unsigned int n) {
                                      sizeof(ipkl->labels[0]));
 
        ipkl->allocated = n;
-       return ISC_R_SUCCESS;
 }
index b6f4d1901bd349acf93706984b93aa6cd50bf2e3..ebf7f8e53d8b60dfdfa92434784625b8598c0bc9 100644 (file)
@@ -1854,13 +1854,10 @@ read_one_rr(dns_journal_t *j);
  *     previously allocated by isc_mem_get().
  */
 
-static isc_result_t
+static void
 size_buffer(isc_mem_t *mctx, isc_buffer_t *b, unsigned int size) {
        if (b->length < size) {
                void *mem = isc_mem_get(mctx, size);
-               if (mem == NULL) {
-                       return ISC_R_NOMEMORY;
-               }
                if (b->base != NULL) {
                        isc_mem_put(mctx, b->base, b->length);
                }
@@ -1868,7 +1865,6 @@ size_buffer(isc_mem_t *mctx, isc_buffer_t *b, unsigned int size) {
                b->length = size;
        }
        isc_buffer_clear(b);
-       return ISC_R_SUCCESS;
 }
 
 isc_result_t
@@ -2033,7 +2029,7 @@ read_one_rr(dns_journal_t *j) {
                FAIL(ISC_R_UNEXPECTED);
        }
 
-       CHECK(size_buffer(j->mctx, &j->it.source, rrhdr.size));
+       size_buffer(j->mctx, &j->it.source, rrhdr.size);
        CHECK(journal_read(j, j->it.source.base, rrhdr.size));
        isc_buffer_add(&j->it.source, rrhdr.size);
 
@@ -2043,7 +2039,7 @@ read_one_rr(dns_journal_t *j) {
         * no compression in present, the output of dns_*_fromwire()
         * is no larger than the input.
         */
-       CHECK(size_buffer(j->mctx, &j->it.target, rrhdr.size));
+       size_buffer(j->mctx, &j->it.target, rrhdr.size);
 
        /*
         * Parse the owner name.  We don't know where it
index cc2290ec492b729e8d4aeea6d8bab9dbf09150d5..7deac18bdace57f0ff4e79ba706adf073c10cfb6 100644 (file)
@@ -31,7 +31,7 @@
 /* Default TTLsig (maximum zone ttl) */
 #define DEFAULT_TTLSIG 604800 /* one week */
 
-isc_result_t
+void
 dns_kasp_create(isc_mem_t *mctx, const char *name, dns_kasp_t **kaspp) {
        dns_kasp_t *kasp;
        dns_kasp_t k = {
@@ -54,7 +54,6 @@ dns_kasp_create(isc_mem_t *mctx, const char *name, dns_kasp_t **kaspp) {
        isc_refcount_init(&kasp->references, 1);
 
        *kaspp = kasp;
-       return ISC_R_SUCCESS;
 }
 
 void
@@ -398,7 +397,7 @@ dns_kasp_addkey(dns_kasp_t *kasp, dns_kasp_key_t *key) {
        ISC_LIST_APPEND(kasp->keys, key, link);
 }
 
-isc_result_t
+void
 dns_kasp_key_create(dns_kasp_t *kasp, dns_kasp_key_t **keyp) {
        dns_kasp_key_t *key = NULL;
        dns_kasp_key_t k = { .tag_max = 0xffff, .length = -1 };
@@ -415,7 +414,6 @@ dns_kasp_key_create(dns_kasp_t *kasp, dns_kasp_key_t **keyp) {
        ISC_LINK_INIT(key, link);
 
        *keyp = key;
-       return ISC_R_SUCCESS;
 }
 
 void
index e35df663cd1ff284417ccdbb7bc5ec39005904bd..75a7528d638136e7115a262bc2c05237391929e7 100644 (file)
@@ -25,7 +25,7 @@
 #include <dns/keystore.h>
 #include <dns/keyvalues.h>
 
-isc_result_t
+void
 dns_keystore_create(isc_mem_t *mctx, const char *name, dns_keystore_t **kspp) {
        dns_keystore_t *keystore;
 
@@ -48,8 +48,6 @@ dns_keystore_create(isc_mem_t *mctx, const char *name, dns_keystore_t **kspp) {
 
        keystore->magic = DNS_KEYSTORE_MAGIC;
        *kspp = keystore;
-
-       return ISC_R_SUCCESS;
 }
 
 static inline void
index ec5e7c41e3e9f074242d9848f1105d92c465d7a5..e5982f14e4c6bf81655262a8340f4354484934b9 100644 (file)
@@ -319,14 +319,9 @@ loadctx_destroy(dns_loadctx_t *lctx);
        else                                                                   \
                LOGIT(result)
 
-#define LOGIT(result)                                                 \
-       if (result == ISC_R_NOMEMORY)                                 \
-               (*callbacks->error)(callbacks, "dns_master_load: %s", \
-                                   isc_result_totext(result));       \
-       else                                                          \
-               (*callbacks->error)(callbacks, "%s: %s:%lu: %s",      \
-                                   "dns_master_load", source, line,  \
-                                   isc_result_totext(result))
+#define LOGIT(result)                                                       \
+       (*callbacks->error)(callbacks, "%s: %s:%lu: %s", "dns_master_load", \
+                           source, line, isc_result_totext(result))
 
 static unsigned char in_addr_arpa_data[] = "\007IN-ADDR\004ARPA";
 static unsigned char in_addr_arpa_offsets[] = { 0, 8, 13 };
@@ -360,8 +355,6 @@ gettoken(isc_lex_t *lex, unsigned int options, isc_token_t *token, bool eol,
        result = isc_lex_gettoken(lex, options, token);
        if (result != ISC_R_SUCCESS) {
                switch (result) {
-               case ISC_R_NOMEMORY:
-                       return ISC_R_NOMEMORY;
                default:
                        (*callbacks->error)(callbacks,
                                            "dns_master_load: %s:%lu:"
@@ -796,10 +789,6 @@ generate(dns_loadctx_t *lctx, char *range, char *lhs, char *gtype, char *rhs,
        target_mem = isc_mem_get(lctx->mctx, target_size);
        rhsbuf = isc_mem_get(lctx->mctx, DNS_MASTER_RHS);
        lhsbuf = isc_mem_get(lctx->mctx, DNS_MASTER_LHS);
-       if (target_mem == NULL || rhsbuf == NULL || lhsbuf == NULL) {
-               result = ISC_R_NOMEMORY;
-               goto error_cleanup;
-       }
        isc_buffer_init(&target, target_mem, target_size);
 
        n = sscanf(range, "%d-%d%1[/]%d", &start, &stop, dummy, &step);
@@ -908,13 +897,8 @@ generate(dns_loadctx_t *lctx, char *range, char *lhs, char *gtype, char *rhs,
        goto cleanup;
 
 error_cleanup:
-       if (result == ISC_R_NOMEMORY) {
-               (*callbacks->error)(callbacks, "$GENERATE: %s",
-                                   isc_result_totext(result));
-       } else {
-               (*callbacks->error)(callbacks, "$GENERATE: %s:%lu: %s", source,
-                                   line, isc_result_totext(result));
-       }
+       (*callbacks->error)(callbacks, "$GENERATE: %s:%lu: %s", source, line,
+                           isc_result_totext(result));
 
 insist_cleanup:
        INSIST(result != ISC_R_SUCCESS);
@@ -1781,10 +1765,6 @@ load_text(dns_loadctx_t *lctx) {
                        new_rdata = grow_rdata(rdata_size + RDSZ, rdata,
                                               rdata_size, &current_list,
                                               &glue_list, mctx);
-                       if (new_rdata == NULL) {
-                               result = ISC_R_NOMEMORY;
-                               goto log_and_cleanup;
-                       }
                        rdata_size += RDSZ;
                        rdata = new_rdata;
                }
@@ -2052,10 +2032,6 @@ load_text(dns_loadctx_t *lctx) {
                                        rdatalist_size + RDLSZ, rdatalist,
                                        rdatalist_size, &current_list,
                                        &glue_list, mctx);
-                               if (new_rdatalist == NULL) {
-                                       result = ISC_R_NOMEMORY;
-                                       goto log_and_cleanup;
-                               }
                                rdatalist = new_rdatalist;
                                rdatalist_size += RDLSZ;
                        }
@@ -2524,10 +2500,6 @@ load_raw(dns_loadctx_t *lctx) {
 
                        new_rdata = grow_rdata(rdcount + RDSZ, rdata,
                                               rdata_size, &head, &dummy, mctx);
-                       if (new_rdata == NULL) {
-                               result = ISC_R_NOMEMORY;
-                               goto cleanup;
-                       }
                        rdata_size = rdcount + RDSZ;
                        rdata = new_rdata;
                }
@@ -2742,15 +2714,11 @@ dns_master_loadstream(FILE *stream, dns_name_t *top, dns_name_t *origin,
        loadctx_create(dns_masterformat_text, mctx, options, 0, top, zclass,
                       origin, callbacks, NULL, NULL, NULL, NULL, NULL, &lctx);
 
-       result = isc_lex_openstream(lctx->lex, stream);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
+       isc_lex_openstream(lctx->lex, stream);
 
        result = (lctx->load)(lctx);
        INSIST(result != DNS_R_CONTINUE);
 
-cleanup:
        dns_loadctx_detach(&lctx);
        return result;
 }
@@ -2950,10 +2918,7 @@ commit(dns_rdatacallbacks_t *callbacks, dns_loadctx_t *lctx,
                }
                result = callbacks->add(callbacks->add_private, owner,
                                        &dataset DNS__DB_FILELINE);
-               if (result == ISC_R_NOMEMORY) {
-                       (*error)(callbacks, "dns_master_load: %s",
-                                isc_result_totext(result));
-               } else if (result != ISC_R_SUCCESS) {
+               if (result != ISC_R_SUCCESS) {
                        dns_name_format(owner, namebuf, sizeof(namebuf));
                        if (source != NULL) {
                                (*error)(callbacks, "%s: %s:%lu: %s: %s",
index 1a012679a1b6f5ac4a280f2e3bc29b895863ea8d..ac827a9849ecfba43cf8734cd70cf70b47a55520 100644 (file)
@@ -46,31 +46,28 @@ struct dns_order {
 #define DNS_ORDER_MAGIC               ISC_MAGIC('O', 'r', 'd', 'r')
 #define DNS_ORDER_VALID(order) ISC_MAGIC_VALID(order, DNS_ORDER_MAGIC)
 
-isc_result_t
+void
 dns_order_create(isc_mem_t *mctx, dns_order_t **orderp) {
-       dns_order_t *order;
+       dns_order_t *order = NULL;
 
        REQUIRE(orderp != NULL && *orderp == NULL);
 
        order = isc_mem_get(mctx, sizeof(*order));
+       *order = (dns_order_t){
+               .ents = ISC_LIST_INITIALIZER,
+               .references = ISC_REFCOUNT_INITIALIZER(1),
+               .magic = DNS_ORDER_MAGIC,
+       };
 
-       ISC_LIST_INIT(order->ents);
-
-       /* Implicit attach. */
-       isc_refcount_init(&order->references, 1);
-
-       order->mctx = NULL;
        isc_mem_attach(mctx, &order->mctx);
-       order->magic = DNS_ORDER_MAGIC;
        *orderp = order;
-       return ISC_R_SUCCESS;
 }
 
-isc_result_t
+void
 dns_order_add(dns_order_t *order, const dns_name_t *name,
              dns_rdatatype_t rdtype, dns_rdataclass_t rdclass,
              unsigned int mode) {
-       dns_order_ent_t *ent;
+       dns_order_ent_t *ent = NULL;
 
        REQUIRE(DNS_ORDER_VALID(order));
        REQUIRE(mode == DNS_RDATASETATTR_RANDOMIZE ||
@@ -79,15 +76,17 @@ dns_order_add(dns_order_t *order, const dns_name_t *name,
                mode == DNS_RDATASETATTR_NONE);
 
        ent = isc_mem_get(order->mctx, sizeof(*ent));
+       *ent = (dns_order_ent_t){
+               .rdtype = rdtype,
+               .rdclass = rdclass,
+               .mode = mode,
+               .link = ISC_LINK_INITIALIZER,
+       };
 
        dns_fixedname_init(&ent->name);
        dns_name_copy(name, dns_fixedname_name(&ent->name));
-       ent->rdtype = rdtype;
-       ent->rdclass = rdclass;
-       ent->mode = mode;
-       ISC_LINK_INIT(ent, link);
+
        ISC_LIST_INITANDAPPEND(order->ents, ent, link);
-       return ISC_R_SUCCESS;
 }
 
 static bool
index a7b56258b9a374ccc9d6680e84e4dc702640fdf0..2b6d87bdd26edbffe9c7c01fee163f57537ee87f 100644 (file)
@@ -110,7 +110,7 @@ peerlist_delete(dns_peerlist_t **list);
 static void
 peer_delete(dns_peer_t **peer);
 
-isc_result_t
+void
 dns_peerlist_new(isc_mem_t *mem, dns_peerlist_t **list) {
        dns_peerlist_t *l;
 
@@ -124,8 +124,6 @@ dns_peerlist_new(isc_mem_t *mem, dns_peerlist_t **list) {
        l->magic = DNS_PEERLIST_MAGIC;
 
        *list = l;
-
-       return ISC_R_SUCCESS;
 }
 
 void
index 7e3d3f193070511f92b2168918b9fb24a625ffaa..83d8764b84eb87ec439f0b657e6d9f75ec9bbdef 100644 (file)
@@ -1024,10 +1024,6 @@ modrdataset(dns_db_t *db, dns_dbnode_t *node, dns_dbversion_t *version,
        }
 
        rdatastr = isc_buffer_base(buffer);
-       if (rdatastr == NULL) {
-               result = ISC_R_NOMEMORY;
-               goto cleanup;
-       }
        rdatastr[isc_buffer_usedlength(buffer) - 1] = 0;
 
        MAYBE_LOCK(sdlz->dlzimp);
index f2cb3989a76eb538f856a439982265b433f1b8d0..c34d8f8f3f187d8350de3cbdd185a7500edace93 100644 (file)
@@ -75,7 +75,7 @@ tkey_log(const char *fmt, ...) {
        va_end(ap);
 }
 
-isc_result_t
+void
 dns_tkeyctx_create(isc_mem_t *mctx, dns_tkeyctx_t **tctxp) {
        REQUIRE(mctx != NULL);
        REQUIRE(tctxp != NULL && *tctxp == NULL);
@@ -87,7 +87,6 @@ dns_tkeyctx_create(isc_mem_t *mctx, dns_tkeyctx_t **tctxp) {
        isc_mem_attach(mctx, &tctx->mctx);
 
        *tctxp = tctx;
-       return ISC_R_SUCCESS;
 }
 
 void
index d2f21ee8baa8ddc9f77a8c50ef6a2feeb2e4222d..be7db571c858f9d689a4d80d0d65934d166b3a79 100644 (file)
@@ -84,7 +84,7 @@
  */
 #define DEFAULT_EDNS_BUFSIZE 1232
 
-isc_result_t
+void
 dns_view_create(isc_mem_t *mctx, isc_loopmgr_t *loopmgr,
                dns_dispatchmgr_t *dispatchmgr, dns_rdataclass_t rdclass,
                const char *name, dns_view_t **viewp) {
@@ -96,9 +96,7 @@ dns_view_create(isc_mem_t *mctx, isc_loopmgr_t *loopmgr,
        REQUIRE(viewp != NULL && *viewp == NULL);
 
        result = isc_file_sanitize(NULL, name, "nta", buffer, sizeof(buffer));
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       RUNTIME_CHECK(result == ISC_R_SUCCESS);
 
        view = isc_mem_get(mctx, sizeof(*view));
        *view = (dns_view_t){
@@ -153,15 +151,9 @@ dns_view_create(isc_mem_t *mctx, isc_loopmgr_t *loopmgr,
 
        isc_mutex_init(&view->new_zone_lock);
 
-       result = dns_order_create(view->mctx, &view->order);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup_new_zone_lock;
-       }
+       dns_order_create(view->mctx, &view->order);
 
-       result = dns_peerlist_new(view->mctx, &view->peers);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup_order;
-       }
+       dns_peerlist_new(view->mctx, &view->peers);
 
        dns_aclenv_create(view->mctx, &view->aclenv);
 
@@ -169,41 +161,6 @@ dns_view_create(isc_mem_t *mctx, isc_loopmgr_t *loopmgr,
 
        view->magic = DNS_VIEW_MAGIC;
        *viewp = view;
-
-       return ISC_R_SUCCESS;
-
-cleanup_order:
-       if (view->order != NULL) {
-               dns_order_detach(&view->order);
-       }
-
-cleanup_new_zone_lock:
-       isc_mutex_destroy(&view->new_zone_lock);
-       dns_badcache_destroy(&view->failcache);
-
-       if (view->dynamickeys != NULL) {
-               dns_tsigkeyring_detach(&view->dynamickeys);
-       }
-
-       isc_refcount_decrementz(&view->weakrefs);
-       isc_refcount_destroy(&view->weakrefs);
-
-       isc_refcount_decrementz(&view->references);
-       isc_refcount_destroy(&view->references);
-
-       dns_fwdtable_destroy(&view->fwdtable);
-       dns_zt_detach(&view->zonetable);
-
-       isc_mutex_destroy(&view->lock);
-
-       if (view->nta_file != NULL) {
-               isc_mem_free(mctx, view->nta_file);
-       }
-
-       isc_mem_free(mctx, view->name);
-       isc_mem_putanddetach(&view->mctx, view, sizeof(*view));
-
-       return result;
 }
 
 static void
index cb2798a3fc3752871850abea7734e88696a0292b..89122af6e4b4ecd18f34c612ec9a42593a3a7c6f 100644 (file)
@@ -885,7 +885,7 @@ static void
 zone_catz_enable(dns_zone_t *zone, dns_catz_zones_t *catzs);
 static void
 zone_catz_disable(dns_zone_t *zone);
-static isc_result_t
+static void
 default_journal(dns_zone_t *zone);
 static void
 zone_xfrdone(dns_zone_t *zone, uint32_t *expireopt, isc_result_t result);
@@ -1537,11 +1537,10 @@ zone_freedbargs(dns_zone_t *zone) {
        zone->db_argv = NULL;
 }
 
-isc_result_t
+void
 dns_zone_getdbtype(dns_zone_t *zone, char ***argv, isc_mem_t *mctx) {
        size_t size = 0;
        unsigned int i;
-       isc_result_t result = ISC_R_SUCCESS;
        void *mem;
        char **tmp, *tmp2, *base;
 
@@ -1568,7 +1567,6 @@ dns_zone_getdbtype(dns_zone_t *zone, char ***argv, isc_mem_t *mctx) {
        }
        UNLOCK_ZONE(zone);
        *argv = mem;
-       return result;
 }
 
 void
@@ -1682,9 +1680,8 @@ dns_zone_setviewrevert(dns_zone_t *zone) {
        UNLOCK_ZONE(zone);
 }
 
-isc_result_t
+void
 dns_zone_setorigin(dns_zone_t *zone, const dns_name_t *origin) {
-       isc_result_t result = ISC_R_SUCCESS;
        char namebuf[1024];
 
        REQUIRE(DNS_ZONE_VALID(zone));
@@ -1711,14 +1708,13 @@ dns_zone_setorigin(dns_zone_t *zone, const dns_name_t *origin) {
        zone->strname = isc_mem_strdup(zone->mctx, namebuf);
 
        if (inline_secure(zone)) {
-               result = dns_zone_setorigin(zone->raw, origin);
+               dns_zone_setorigin(zone->raw, origin);
        }
        UNLOCK_ZONE(zone);
-       return result;
 }
 
-static isc_result_t
-dns_zone_setstring(dns_zone_t *zone, char **field, const char *value) {
+static void
+setstring(dns_zone_t *zone, char **field, const char *value) {
        char *copy;
 
        if (value != NULL) {
@@ -1732,29 +1728,22 @@ dns_zone_setstring(dns_zone_t *zone, char **field, const char *value) {
        }
 
        *field = copy;
-       return ISC_R_SUCCESS;
 }
 
-isc_result_t
+void
 dns_zone_setfile(dns_zone_t *zone, const char *file, dns_masterformat_t format,
                 const dns_master_style_t *style) {
-       isc_result_t result = ISC_R_SUCCESS;
-
        REQUIRE(DNS_ZONE_VALID(zone));
        REQUIRE(zone->stream == NULL);
 
        LOCK_ZONE(zone);
-       result = dns_zone_setstring(zone, &zone->masterfile, file);
-       if (result == ISC_R_SUCCESS) {
-               zone->masterformat = format;
-               if (format == dns_masterformat_text) {
-                       zone->masterstyle = style;
-               }
-               result = default_journal(zone);
+       setstring(zone, &zone->masterfile, file);
+       zone->masterformat = format;
+       if (format == dns_masterformat_text) {
+               zone->masterstyle = style;
        }
+       default_journal(zone);
        UNLOCK_ZONE(zone);
-
-       return result;
 }
 
 const char *
@@ -1764,11 +1753,9 @@ dns_zone_getfile(dns_zone_t *zone) {
        return zone->masterfile;
 }
 
-isc_result_t
+void
 dns_zone_setstream(dns_zone_t *zone, const FILE *stream,
                   dns_masterformat_t format, const dns_master_style_t *style) {
-       isc_result_t result = ISC_R_SUCCESS;
-
        REQUIRE(DNS_ZONE_VALID(zone));
        REQUIRE(stream != NULL);
        REQUIRE(zone->masterfile == NULL);
@@ -1779,10 +1766,8 @@ dns_zone_setstream(dns_zone_t *zone, const FILE *stream,
        if (format == dns_masterformat_text) {
                zone->masterstyle = style;
        }
-       result = default_journal(zone);
+       default_journal(zone);
        UNLOCK_ZONE(zone);
-
-       return result;
 }
 
 dns_ttl_t
@@ -1808,9 +1793,8 @@ dns_zone_setmaxttl(dns_zone_t *zone, dns_ttl_t maxttl) {
        return;
 }
 
-static isc_result_t
+static void
 default_journal(dns_zone_t *zone) {
-       isc_result_t result;
        char *journal;
 
        REQUIRE(DNS_ZONE_VALID(zone));
@@ -1825,24 +1809,19 @@ default_journal(dns_zone_t *zone) {
        } else {
                journal = NULL;
        }
-       result = dns_zone_setstring(zone, &zone->journal, journal);
+       setstring(zone, &zone->journal, journal);
        if (journal != NULL) {
                isc_mem_free(zone->mctx, journal);
        }
-       return result;
 }
 
-isc_result_t
+void
 dns_zone_setjournal(dns_zone_t *zone, const char *myjournal) {
-       isc_result_t result = ISC_R_SUCCESS;
-
        REQUIRE(DNS_ZONE_VALID(zone));
 
        LOCK_ZONE(zone);
-       result = dns_zone_setstring(zone, &zone->journal, myjournal);
+       setstring(zone, &zone->journal, myjournal);
        UNLOCK_ZONE(zone);
-
-       return result;
 }
 
 char *
@@ -5474,15 +5453,11 @@ cleanup:
        {
                DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_FIRSTREFRESH);
 
-               if (result != ISC_R_NOMEMORY) {
-                       if (zone->journal != NULL) {
-                               zone_saveunique(zone, zone->journal,
-                                               "jn-XXXXXXXX");
-                       }
-                       if (zone->masterfile != NULL) {
-                               zone_saveunique(zone, zone->masterfile,
-                                               "db-XXXXXXXX");
-                       }
+               if (zone->journal != NULL) {
+                       zone_saveunique(zone, zone->journal, "jn-XXXXXXXX");
+               }
+               if (zone->masterfile != NULL) {
+                       zone_saveunique(zone, zone->masterfile, "db-XXXXXXXX");
                }
 
                /* Mark the zone for immediate refresh. */
@@ -19962,17 +19937,13 @@ dns_zone_getrcvquerystats(dns_zone_t *zone) {
        }
 }
 
-isc_result_t
+void
 dns_zone_setkeydirectory(dns_zone_t *zone, const char *directory) {
-       isc_result_t result = ISC_R_SUCCESS;
-
        REQUIRE(DNS_ZONE_VALID(zone));
 
        LOCK_ZONE(zone);
-       result = dns_zone_setstring(zone, &zone->keydirectory, directory);
+       setstring(zone, &zone->keydirectory, directory);
        UNLOCK_ZONE(zone);
-
-       return result;
 }
 
 const char *
index 59b1b464310a70f856dc20ff32168a8f6e8c73d8..c476cf6ef8610e18e9982af5d01d6a505cdc0110 100644 (file)
@@ -634,8 +634,8 @@ isc_file_safecreate(const char *filename, FILE **fp) {
 isc_result_t
 isc_file_splitpath(isc_mem_t *mctx, const char *path, char **dirname,
                   char const **bname) {
-       char *dir;
-       const char *file, *slash;
+       char *dir = NULL;
+       const char *file = NULL, *slash = NULL;
 
        if (path == NULL) {
                return ISC_R_INVALIDFILE;
@@ -655,10 +655,6 @@ isc_file_splitpath(isc_mem_t *mctx, const char *path, char **dirname,
                dir = isc_mem_strdup(mctx, ".");
        }
 
-       if (dir == NULL) {
-               return ISC_R_NOMEMORY;
-       }
-
        if (*file == '\0') {
                isc_mem_free(mctx, dir);
                return ISC_R_INVALIDFILE;
index 2d67ecc28d6764cf360bd97085360739238a62ea..320d72e6503b9db7733039ed190ff5951ab5acd3 100644 (file)
@@ -713,7 +713,6 @@ render_500(const isc_httpd_t *httpd, const isc_httpdurl_t *urlinfo, void *arg,
  *
  * Returns:
  *\li  #ISC_R_SUCCESS    -- all is well.
- *\li  #ISC_R_NOMEMORY   -- not enough memory to compress data
  *\li  #ISC_R_FAILURE    -- error during compression or compressed
  *                          data would be larger than input data
  */
index 9cd508bfd31717ad4e0c45689f67c0406da933ab..75754bf08e5365e3b82fadd388eeade4bb5b6529 100644 (file)
@@ -485,7 +485,6 @@ isc_buffer_printf(isc_buffer_t *restrict b, const char *restrict format, ...)
  *\li  #ISC_R_SUCCESS  Operation succeeded.
  *\li  #ISC_R_NOSPACE  'b' does not allow reallocation and appending the
  *                     formatted string to it would cause it to overflow.
- *\li  #ISC_R_NOMEMORY Reallocation failed.
  *\li  #ISC_R_FAILURE  Other error occurred.
  */
 
index 5db3a9cb4414b4aaf5e0896b941bbd509d1c75a9..200b2d6fb6e5c47a00f519d385f95c37b4e8f82f 100644 (file)
@@ -309,7 +309,6 @@ isc_file_splitpath(isc_mem_t *mctx, const char *path, char **dirname,
  * Returns:
  * - ISC_R_SUCCESS on success
  * - ISC_R_INVALIDFILE if 'path' is empty or ends with '/'
- * - ISC_R_NOMEMORY if unable to allocate memory
  */
 
 isc_result_t
index 163fbefb793381c651de2d46f07f514e7227a0e8..54c6f39e58a903d2c54460b1f6938c1356e73025 100644 (file)
@@ -61,7 +61,6 @@ isc_ht_destroy(isc_ht_t **htp);
  *\li   write-lock
  *
  * Returns:
- *\li  #ISC_R_NOMEMORY         -- not enough memory to create pool
  *\li  #ISC_R_EXISTS           -- node of the same key already exists
  *\li  #ISC_R_SUCCESS          -- all is well.
  */
index 9aad357402cc0cd0479fc82a6401cd7e7127a621..d04903b007286bf05a3d50642da89358b67bfc52 100644 (file)
@@ -73,7 +73,6 @@ isc_interfaceiter_create(isc_mem_t *mctx, isc_interfaceiter_t **iterp);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  *\li  Various network-related errors
  */
 
index 9ce051c6f3f98f7e4a0e218833ea0cd9bb67e857..a4faf25667a8a17cd108d3fbe054195db74b22b5 100644 (file)
@@ -151,10 +151,6 @@ isc_lex_create(isc_mem_t *mctx, size_t max_token, isc_lex_t **lexp);
  *
  * Ensures:
  *\li  On success, *lexp is attached to the newly created lexer.
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 void
@@ -227,14 +223,13 @@ isc_lex_openfile(isc_lex_t *lex, const char *filename);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY                 Out of memory
  *\li  #ISC_R_NOTFOUND                 File not found
  *\li  #ISC_R_NOPERM                   No permission to open file
  *\li  #ISC_R_FAILURE                  Couldn't open file, not sure why
  *\li  #ISC_R_UNEXPECTED
  */
 
-isc_result_t
+void
 isc_lex_openstream(isc_lex_t *lex, FILE *stream);
 /*%<
  * Make 'stream' the current input source for 'lex'.
@@ -243,10 +238,6 @@ isc_lex_openstream(isc_lex_t *lex, FILE *stream);
  *\li  'lex' is a valid lexer.
  *
  *\li  'stream' is a valid C stream.
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY                 Out of memory
  */
 
 isc_result_t
@@ -261,7 +252,6 @@ isc_lex_openbuffer(isc_lex_t *lex, isc_buffer_t *buffer);
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY                 Out of memory
  */
 
 isc_result_t
@@ -291,7 +281,6 @@ isc_lex_gettoken(isc_lex_t *lex, unsigned int options, isc_token_t *tokenp);
  * Returns:
  *\li  #ISC_R_SUCCESS
  *\li  #ISC_R_UNEXPECTEDEND
- *\li  #ISC_R_NOMEMORY
  *
  *     These two results are returned only if their corresponding lexer
  *     options are not set.
@@ -404,7 +393,6 @@ isc_lex_setsourcename(isc_lex_t *lex, const char *name);
  *
  * Returns:
  * \li #ISC_R_SUCCESS
- * \li #ISC_R_NOMEMORY
  * \li #ISC_R_NOTFOUND - there are no sources.
  */
 
index c076aff901a194702b8d183c424b2dcabba058ee..ff954be1bac37bfbbfa94c537881f0378c042f63 100644 (file)
@@ -428,14 +428,7 @@ isc_log_createchannel(isc_logconfig_t *lcfg, const char *name,
  *     #ISC_LOG_DEBUGONLY or #ISC_LOG_BUFFERED.
  *
  * Ensures:
- *\li  #ISC_R_SUCCESS
- *             A channel with the given name is usable with
- *             isc_log_usechannel().
- *
- *\li  #ISC_R_NOMEMORY or #ISC_R_UNEXPECTED
- *             No additional memory is being used by the logging context.
- *             Any channel that previously existed with the given name
- *             is not redefined.
+ *     A channel with the given name is usable with isc_log_usechannel().
  */
 
 isc_result_t
index 789efee2cbe93cc8ed1de9c1825ea76b23e3aa81..28bdd64041e410a9a55e9e69a6c62118144b1c9c 100644 (file)
@@ -405,10 +405,6 @@ isc__mempool_create(isc_mem_t *restrict mctx, const size_t element_size,
  * Defaults:
  *\li  freemax = 1
  *\li  fillcount = 1
- *
- * Returns:
- *\li  #ISC_R_NOMEMORY         -- not enough memory to create pool
- *\li  #ISC_R_SUCCESS          -- all is well.
  */
 
 #define isc_mempool_destroy(mp) isc__mempool_destroy((mp)_ISC_MEM_FILELINE)
index 30aa11f8383ec29ec9aa5d1bf4a73c1296a021fd..900b6a50f9399d48714806cddd4d26bfd6c7a651 100644 (file)
@@ -37,7 +37,7 @@
  *** Functions
  ***/
 
-isc_result_t
+void
 isc_portset_create(isc_mem_t *mctx, isc_portset_t **portsetp);
 /*%<
  * Create a port set and initialize it as an empty set.
@@ -45,10 +45,6 @@ isc_portset_create(isc_mem_t *mctx, isc_portset_t **portsetp);
  * Requires:
  *\li  'mctx' to be valid.
  *\li  'portsetp' to be non NULL and '*portsetp' to be NULL;
- *
- * Returns:
- *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  */
 
 void
index cdedb687647fa0b201a3a058c7e7be626a7f1da7..22d05af116a2a1b3ab36bd99d0166bbbc8eec3b9 100644 (file)
@@ -143,7 +143,6 @@ isc_radix_insert(isc_radix_tree_t *radix, isc_radix_node_t **target,
  *     a valid prefix.
  *
  * Returns:
- * \li ISC_R_NOMEMORY
  * \li ISC_R_SUCCESS
  */
 
@@ -166,10 +165,6 @@ isc_radix_create(isc_mem_t *mctx, isc_radix_tree_t **target, int maxbits);
  * \li 'mctx' to be valid.
  * \li 'target' to be non NULL and '*target' to be NULL.
  * \li 'maxbits' to be less than or equal to RADIX_MAXBITS.
- *
- * Returns:
- * \li ISC_R_NOMEMORY
- * \li ISC_R_SUCCESS
  */
 
 void
index 3cc9b9b0cc08a0f13362ce9ef4c8229d9d54e2e9..9dda1c1047162df5921076abe63fd01874640699 100644 (file)
@@ -103,7 +103,7 @@ typedef enum {
 } isc_symexists_t;
 
 /*% Create a symbol table. */
-isc_result_t
+void
 isc_symtab_create(isc_mem_t *mctx, unsigned int size,
                  isc_symtabaction_t undefine_action, void *undefine_arg,
                  bool case_sensitive, isc_symtab_t **symtabp);
index aa3beaa976566712c7e65125afe95f615f5aab0c..b1ac9b2cbd576bf0accffe9083f59238151e89f8 100644 (file)
@@ -64,7 +64,7 @@ struct isc_lex {
        LIST(struct inputsource) sources;
 };
 
-static isc_result_t
+static void
 grow_data(isc_lex_t *lex, size_t *remainingp, char **currp, char **prevp) {
        char *tmp;
 
@@ -78,7 +78,6 @@ grow_data(isc_lex_t *lex, size_t *remainingp, char **currp, char **prevp) {
        lex->data = tmp;
        *remainingp += lex->max_token;
        lex->max_token *= 2;
-       return ISC_R_SUCCESS;
 }
 
 void
@@ -179,27 +178,24 @@ isc_lex_setspecials(isc_lex_t *lex, isc_lexspecials_t specials) {
        memmove(lex->specials, specials, 256);
 }
 
-static isc_result_t
+static void
 new_source(isc_lex_t *lex, bool is_file, bool need_close, void *input,
           const char *name) {
        inputsource *source;
 
        source = isc_mem_get(lex->mctx, sizeof(*source));
-       source->result = ISC_R_SUCCESS;
-       source->is_file = is_file;
-       source->need_close = need_close;
-       source->at_eof = false;
-       source->last_was_eol = lex->last_was_eol;
-       source->input = input;
-       source->name = isc_mem_strdup(lex->mctx, name);
-       source->pushback = NULL;
+       *source = (inputsource){
+               .is_file = is_file,
+               .need_close = need_close,
+               .last_was_eol = lex->last_was_eol,
+               .input = input,
+               .name = isc_mem_strdup(lex->mctx, name),
+               .line = 1,
+               .link = ISC_LINK_INITIALIZER,
+       };
        isc_buffer_allocate(lex->mctx, &source->pushback,
                            (unsigned int)lex->max_token);
-       source->ignored = 0;
-       source->line = 1;
-       ISC_LIST_INITANDPREPEND(lex->sources, source, link);
-
-       return ISC_R_SUCCESS;
+       ISC_LIST_PREPEND(lex->sources, source, link);
 }
 
 isc_result_t
@@ -218,14 +214,11 @@ isc_lex_openfile(isc_lex_t *lex, const char *filename) {
                return result;
        }
 
-       result = new_source(lex, true, true, stream, filename);
-       if (result != ISC_R_SUCCESS) {
-               (void)fclose(stream);
-       }
-       return result;
+       new_source(lex, true, true, stream, filename);
+       return ISC_R_SUCCESS;
 }
 
-isc_result_t
+void
 isc_lex_openstream(isc_lex_t *lex, FILE *stream) {
        char name[128];
 
@@ -237,7 +230,7 @@ isc_lex_openstream(isc_lex_t *lex, FILE *stream) {
 
        snprintf(name, sizeof(name), "stream-%p", stream);
 
-       return new_source(lex, true, false, stream, name);
+       new_source(lex, true, false, stream, name);
 }
 
 isc_result_t
@@ -252,7 +245,8 @@ isc_lex_openbuffer(isc_lex_t *lex, isc_buffer_t *buffer) {
 
        snprintf(name, sizeof(name), "buffer-%p", buffer);
 
-       return new_source(lex, false, false, buffer, name);
+       new_source(lex, false, false, buffer, name);
+       return ISC_R_SUCCESS;
 }
 
 isc_result_t
@@ -669,11 +663,7 @@ isc_lex_gettoken(isc_lex_t *lex, unsigned int options, isc_token_t *tokenp) {
                                state = lexstate_string;
                        }
                        if (remaining == 0U) {
-                               result = grow_data(lex, &remaining, &curr,
-                                                  &prev);
-                               if (result != ISC_R_SUCCESS) {
-                                       goto done;
-                               }
+                               grow_data(lex, &remaining, &curr, &prev);
                        }
                        INSIST(remaining > 0U);
                        *curr++ = c;
@@ -685,11 +675,8 @@ isc_lex_gettoken(isc_lex_t *lex, unsigned int options, isc_token_t *tokenp) {
                            (options & ISC_LEXOPT_VPAIR) != 0)
                        {
                                if (remaining == 0U) {
-                                       result = grow_data(lex, &remaining,
-                                                          &curr, &prev);
-                                       if (result != ISC_R_SUCCESS) {
-                                               goto done;
-                                       }
+                                       grow_data(lex, &remaining, &curr,
+                                                 &prev);
                                }
                                INSIST(remaining > 0U);
                                *curr++ = c;
@@ -744,11 +731,7 @@ isc_lex_gettoken(isc_lex_t *lex, unsigned int options, isc_token_t *tokenp) {
                                                                  : false;
                        }
                        if (remaining == 0U) {
-                               result = grow_data(lex, &remaining, &curr,
-                                                  &prev);
-                               if (result != ISC_R_SUCCESS) {
-                                       goto done;
-                               }
+                               grow_data(lex, &remaining, &curr, &prev);
                        }
                        INSIST(remaining > 0U);
                        *curr++ = c;
@@ -849,11 +832,8 @@ isc_lex_gettoken(isc_lex_t *lex, unsigned int options, isc_token_t *tokenp) {
                                        escaped = false;
                                }
                                if (remaining == 0U) {
-                                       result = grow_data(lex, &remaining,
-                                                          &curr, &prev);
-                                       if (result != ISC_R_SUCCESS) {
-                                               goto done;
-                                       }
+                                       grow_data(lex, &remaining, &curr,
+                                                 &prev);
                                }
                                INSIST(remaining > 0U);
                                prev = curr;
@@ -901,11 +881,7 @@ isc_lex_gettoken(isc_lex_t *lex, unsigned int options, isc_token_t *tokenp) {
                        }
 
                        if (remaining == 0U) {
-                               result = grow_data(lex, &remaining, &curr,
-                                                  &prev);
-                               if (result != ISC_R_SUCCESS) {
-                                       goto done;
-                               }
+                               grow_data(lex, &remaining, &curr, &prev);
                        }
                        INSIST(remaining > 0U);
                        prev = curr;
index f3e515eaad50399b9c12013033fe7741c4d4f975..55d9b034171d5f82069dc6b49dca482c9d21868e 100644 (file)
@@ -55,7 +55,7 @@ portset_remove(isc_portset_t *portset, in_port_t port) {
        }
 }
 
-isc_result_t
+void
 isc_portset_create(isc_mem_t *mctx, isc_portset_t **portsetp) {
        isc_portset_t *portset;
 
@@ -64,8 +64,6 @@ isc_portset_create(isc_mem_t *mctx, isc_portset_t **portsetp) {
        portset = isc_mem_get(mctx, sizeof(*portset));
        *portset = (isc_portset_t){ 0 };
        *portsetp = portset;
-
-       return ISC_R_SUCCESS;
 }
 
 void
index 0d4da2fe1980df304fe075112ee098072ae58aab..bdd6a9d13c8b1e3c3c180aef3370bfe9008d7a09 100644 (file)
@@ -47,7 +47,7 @@ struct isc_symtab {
        bool case_sensitive;
 };
 
-isc_result_t
+void
 isc_symtab_create(isc_mem_t *mctx, unsigned int size,
                  isc_symtabaction_t undefine_action, void *undefine_arg,
                  bool case_sensitive, isc_symtab_t **symtabp) {
@@ -75,8 +75,6 @@ isc_symtab_create(isc_mem_t *mctx, unsigned int size,
        symtab->magic = SYMTAB_MAGIC;
 
        *symtabp = symtab;
-
-       return ISC_R_SUCCESS;
 }
 
 void
index 78b51c1c5e48b736cf3405376ddc661ceba3dbe5..68dd6dd6cd2809f337ae147d285a56c211f5889d 100644 (file)
@@ -2102,13 +2102,11 @@ check_remoteserverlist(const cfg_obj_t *cctx, const char *list,
  */
 static isc_result_t
 check_remoteserverlists(const cfg_obj_t *cctx, isc_mem_t *mctx) {
-       isc_result_t result, tresult;
+       isc_result_t result = ISC_R_SUCCESS, tresult;
        isc_symtab_t *symtab = NULL;
 
-       result = isc_symtab_create(mctx, 100, freekey, mctx, false, &symtab);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       isc_symtab_create(mctx, 100, freekey, mctx, false, &symtab);
+
        tresult = check_remoteserverlist(cctx, "remote-servers", symtab, mctx);
        if (tresult != ISC_R_SUCCESS) {
                result = tresult;
@@ -2200,10 +2198,7 @@ check_httpservers(const cfg_obj_t *config, isc_mem_t *mctx) {
        const cfg_listelt_t *elt = NULL;
        isc_symtab_t *symtab = NULL;
 
-       result = isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
 
        result = cfg_map_get(config, "http", &obj);
        if (result != ISC_R_SUCCESS) {
@@ -2373,10 +2368,7 @@ check_tls_definitions(const cfg_obj_t *config, isc_mem_t *mctx) {
                return result;
        }
 
-       result = isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
 
        for (elt = cfg_list_first(obj); elt != NULL; elt = cfg_list_next(elt)) {
                const char *name;
@@ -2457,11 +2449,7 @@ validate_remotes(const cfg_obj_t *obj, const cfg_obj_t *config,
        const cfg_obj_t *listobj;
 
        REQUIRE(countp != NULL);
-       result = isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
-       if (result != ISC_R_SUCCESS) {
-               *countp = count;
-               return result;
-       }
+       isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
 
 newlist:
        listobj = cfg_tuple_get(obj, "addresses");
@@ -4895,20 +4883,13 @@ record_ds_keys(isc_symtab_t *symtab, isc_mem_t *mctx,
 static isc_result_t
 check_ta_conflicts(const cfg_obj_t *global_ta, const cfg_obj_t *view_ta,
                   bool autovalidation, isc_mem_t *mctx) {
-       isc_result_t result, tresult;
+       isc_result_t result = ISC_R_SUCCESS, tresult;
        const cfg_listelt_t *elt = NULL;
        const cfg_obj_t *keylist = NULL;
        isc_symtab_t *statictab = NULL, *dstab = NULL;
 
-       result = isc_symtab_create(mctx, 100, freekey, mctx, false, &statictab);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
-
-       result = isc_symtab_create(mctx, 100, freekey, mctx, false, &dstab);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
+       isc_symtab_create(mctx, 100, freekey, mctx, false, &statictab);
+       isc_symtab_create(mctx, 100, freekey, mctx, false, &dstab);
 
        /*
         * First we record all the static keys (trust-anchors configured with
@@ -4970,7 +4951,6 @@ check_ta_conflicts(const cfg_obj_t *global_ta, const cfg_obj_t *view_ta,
                }
        }
 
-cleanup:
        if (statictab != NULL) {
                isc_symtab_destroy(&statictab);
        }
@@ -5116,7 +5096,7 @@ check_catz(const cfg_obj_t *catz_obj, const char *viewname, isc_mem_t *mctx) {
        const cfg_obj_t *obj, *nameobj, *primariesobj;
        const char *zonename;
        const char *forview = " for view ";
-       isc_result_t result, tresult;
+       isc_result_t result = ISC_R_SUCCESS, tresult;
        isc_symtab_t *symtab = NULL;
        dns_fixedname_t fixed;
        dns_name_t *name = dns_fixedname_initname(&fixed);
@@ -5126,10 +5106,7 @@ check_catz(const cfg_obj_t *catz_obj, const char *viewname, isc_mem_t *mctx) {
                forview = "";
        }
 
-       result = isc_symtab_create(mctx, 100, freekey, mctx, false, &symtab);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       isc_symtab_create(mctx, 100, freekey, mctx, false, &symtab);
 
        obj = cfg_tuple_get(catz_obj, "zone list");
 
@@ -5310,10 +5287,7 @@ check_viewconf(const cfg_obj_t *config, const cfg_obj_t *voptions,
         * Check that all zone statements are syntactically correct and
         * there are no duplicate zones.
         */
-       tresult = isc_symtab_create(mctx, 1000, freekey, mctx, false, &symtab);
-       if (tresult != ISC_R_SUCCESS) {
-               return ISC_R_NOMEMORY;
-       }
+       isc_symtab_create(mctx, 1000, freekey, mctx, false, &symtab);
 
        cfg_aclconfctx_create(mctx, &actx);
 
@@ -5426,10 +5400,7 @@ check_viewconf(const cfg_obj_t *config, const cfg_obj_t *voptions,
         * Check that all key statements are syntactically correct and
         * there are no duplicate keys.
         */
-       tresult = isc_symtab_create(mctx, 1000, freekey, mctx, false, &symtab);
-       if (tresult != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
+       isc_symtab_create(mctx, 1000, freekey, mctx, false, &symtab);
 
        (void)cfg_map_get(config, "key", &keys);
        tresult = check_keylist(keys, symtab, mctx);
@@ -5659,10 +5630,7 @@ check_logging(const cfg_obj_t *config, isc_mem_t *mctx) {
                return ISC_R_SUCCESS;
        }
 
-       result = isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       isc_symtab_create(mctx, 100, NULL, NULL, false, &symtab);
 
        symvalue.as_cpointer = NULL;
        for (i = 0; default_channels[i] != NULL; i++) {
@@ -5796,10 +5764,7 @@ check_controls(const cfg_obj_t *config, isc_mem_t *mctx) {
 
        cfg_aclconfctx_create(mctx, &actx);
 
-       result = isc_symtab_create(mctx, 100, freekey, mctx, true, &symtab);
-       if (result != ISC_R_SUCCESS) {
-               goto cleanup;
-       }
+       isc_symtab_create(mctx, 100, freekey, mctx, true, &symtab);
 
        /*
         * INET: Check allow clause.
@@ -5858,7 +5823,7 @@ check_controls(const cfg_obj_t *config, isc_mem_t *mctx) {
                        result = ISC_R_FAMILYNOSUPPORT;
                }
        }
-cleanup:
+
        cfg_aclconfctx_detach(&actx);
        if (symtab != NULL) {
                isc_symtab_destroy(&symtab);
@@ -5935,23 +5900,9 @@ isccfg_check_namedconf(const cfg_obj_t *config, unsigned int flags,
         * case sensitive. This will prevent people using FOO.DB and foo.db
         * on case sensitive file systems but that shouldn't be a major issue.
         */
-       tresult = isc_symtab_create(mctx, 100, NULL, NULL, false, &files);
-       if (tresult != ISC_R_SUCCESS) {
-               result = tresult;
-               goto cleanup;
-       }
-
-       tresult = isc_symtab_create(mctx, 100, freekey, mctx, false, &keydirs);
-       if (tresult != ISC_R_SUCCESS) {
-               result = tresult;
-               goto cleanup;
-       }
-
-       tresult = isc_symtab_create(mctx, 100, freekey, mctx, true, &inview);
-       if (tresult != ISC_R_SUCCESS) {
-               result = tresult;
-               goto cleanup;
-       }
+       isc_symtab_create(mctx, 100, NULL, NULL, false, &files);
+       isc_symtab_create(mctx, 100, freekey, mctx, false, &keydirs);
+       isc_symtab_create(mctx, 100, freekey, mctx, true, &inview);
 
        if (views == NULL) {
                tresult = check_viewconf(config, NULL, NULL, dns_rdataclass_in,
@@ -5980,11 +5931,8 @@ isccfg_check_namedconf(const cfg_obj_t *config, unsigned int flags,
                }
        }
 
-       tresult = isc_symtab_create(mctx, 100, NULL, NULL, true, &symtab);
-       if (tresult != ISC_R_SUCCESS) {
-               result = tresult;
-               goto cleanup;
-       }
+       isc_symtab_create(mctx, 100, NULL, NULL, true, &symtab);
+
        for (velement = cfg_list_first(views); velement != NULL;
             velement = cfg_list_next(velement))
        {
@@ -6108,7 +6056,6 @@ isccfg_check_namedconf(const cfg_obj_t *config, unsigned int flags,
                }
        }
 
-cleanup:
        if (symtab != NULL) {
                isc_symtab_destroy(&symtab);
        }
index 5a43e36e31875b57d50c5f5810e2780ed951d4f1..8489a63e71e38cdb6c4cd5273c641af4e85fa3f8 100644 (file)
@@ -152,7 +152,6 @@ cfg_parse_buffer(cfg_parser_t *pctx, isc_buffer_t *buffer, const char *file,
  *
  * Returns:
  *     \li #ISC_R_SUCCESS                 - success
- *\li      #ISC_R_NOMEMORY                - no memory available
  *\li      #ISC_R_INVALIDFILE             - file doesn't exist or is unreadable
  *\li      others                            - file contains errors
  */
index 1818ef319924021a356f421476f77acfe2f97d45..0a2e501bc760512655e223b6f7ff51e30d117de9 100644 (file)
@@ -51,7 +51,6 @@ cfg_kasp_fromconfig(const cfg_obj_t *config, dns_kasp_t *default_kasp,
  *
  *\li  #ISC_R_SUCCESS  If creating and configuring the KASP succeeds.
  *\li  #ISC_R_EXISTS   If 'kasplist' already has a kasp structure with 'name'.
- *\li  #ISC_R_NOMEMORY
  *
  *\li  Other errors are possible.
  */
@@ -79,7 +78,6 @@ cfg_keystore_fromconfig(const cfg_obj_t *config, isc_mem_t *mctx,
  *
  *\li  #ISC_R_SUCCESS  If creating and configuring the keystore succeeds.
  *\li  #ISC_R_EXISTS   If 'keystorelist' already has a keystore with 'name'.
- *\li  #ISC_R_NOMEMORY
  *
  *\li  Other errors are possible.
  */
index e5fd22083447feadeda37357ed95579340afcb00..9ab717a66f0c78d110bbd54c70b6fa393da616af 100644 (file)
@@ -120,10 +120,7 @@ cfg_kaspkey_fromconfig(const cfg_obj_t *config, dns_kasp_t *kasp,
        const cfg_obj_t *tagrange = NULL;
 
        /* Create a new key reference. */
-       result = dns_kasp_key_create(kasp, &key);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_kasp_key_create(kasp, &key);
 
        if (config == NULL) {
                /* We are creating a key reference for the default kasp. */
@@ -484,12 +481,11 @@ cfg_kasp_fromconfig(const cfg_obj_t *config, dns_kasp_t *default_kasp,
                return result;
        }
 
+       result = ISC_R_SUCCESS;
+
        /* No kasp with configured name was found in list, create new one. */
        INSIST(kasp == NULL);
-       result = dns_kasp_create(mctx, kaspname, &kasp);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_kasp_create(mctx, kaspname, &kasp);
        INSIST(kasp != NULL);
 
        /* Now configure. */
@@ -719,14 +715,7 @@ cfg_kasp_fromconfig(const cfg_obj_t *config, dns_kasp_t *default_kasp,
                {
                        /* Create a new key reference. */
                        new_key = NULL;
-                       result = dns_kasp_key_create(kasp, &new_key);
-                       if (result != ISC_R_SUCCESS) {
-                               cfg_obj_log(config, ISC_LOG_ERROR,
-                                           "dnssec-policy: failed to "
-                                           "configure keys (%s)",
-                                           isc_result_totext(result));
-                               goto cleanup;
-                       }
+                       dns_kasp_key_create(kasp, &new_key);
                        if (dns_kasp_key_ksk(key)) {
                                new_key->role |= DNS_KASP_KEY_ROLE_KSK;
                        }
@@ -832,10 +821,7 @@ cfg_keystore_fromconfig(const cfg_obj_t *config, isc_mem_t *mctx,
         * No key-store with configured name was found in list, create new one.
         */
        INSIST(keystore == NULL);
-       result = dns_keystore_create(mctx, name, &keystore);
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
+       dns_keystore_create(mctx, name, &keystore);
        INSIST(keystore != NULL);
 
        /* Now configure. */
index bef0acdb292aa33db5656ac02e19c70a5bbd3e2a..7571d8cb3467aa1a451336fb49018652e2c5eb1d 100644 (file)
@@ -3797,8 +3797,8 @@ create_map(cfg_parser_t *pctx, const cfg_type_t *type, cfg_obj_t **ret) {
        cfg_obj_t *obj = NULL;
 
        CHECK(cfg_create_obj(pctx, type, &obj));
-       CHECK(isc_symtab_create(pctx->mctx, 5, /* XXX */
-                               map_symtabitem_destroy, pctx, false, &symtab));
+       isc_symtab_create(pctx->mctx, 5, /* XXX */
+                         map_symtabitem_destroy, pctx, false, &symtab);
        obj->value.map.symtab = symtab;
        obj->value.map.id = NULL;
 
index a330f663cb46a026a10b04214b93ca1514bb4e24..4ff16aa6a4f7b6a20ae044fb65fe06584bc93d6c 100644 (file)
@@ -3181,9 +3181,6 @@ ns_client_findversion(ns_client_t *client, dns_db_t *db) {
                 * the active list.
                 */
                dbversion = client_getdbversion(client);
-               if (dbversion == NULL) {
-                       return NULL;
-               }
                dns_db_attach(db, &dbversion->db);
                dns_db_currentversion(db, &dbversion->version);
                dbversion->acl_checked = false;
index e5bf4176b9575c77b8552d1aea00285e2e4eb795..c5addf2c829ff3423bb6ad07d660b48aa6492524 100644 (file)
@@ -474,7 +474,6 @@ ns_plugin_register_t(const char *parameters, const void *cfg, const char *file,
  *
  * Returns:
  *\li  #ISC_R_SUCCESS
- *\li  #ISC_R_NOMEMORY
  *\li  Other errors are possible
  */
 
index ff75978b13e43fbda4521547d346da8c0184a4fd..4584be6e26732a9792d02ddd974225aab3e470b6 100644 (file)
@@ -1507,15 +1507,12 @@ query_getdb(ns_client_t *client, dns_name_t *name, dns_rdatatype_t qtype,
                        *versionp = NULL;
 
                        dbversion = ns_client_findversion(client, tdbp);
-                       if (dbversion == NULL) {
-                               tresult = ISC_R_NOMEMORY;
-                       } else {
-                               /*
-                                * Be sure to return our database.
-                                */
-                               *dbp = tdbp;
-                               *versionp = dbversion->version;
-                       }
+
+                       /*
+                        * Be sure to return our database.
+                        */
+                       *dbp = tdbp;
+                       *versionp = dbversion->version;
 
                        /*
                         * We return a null zone, No stats for DLZ zones.
@@ -7619,7 +7616,7 @@ query_addnoqnameproof(query_ctx_t *qctx) {
        isc_buffer_t *dbuf, b;
        dns_name_t *fname = NULL;
        dns_rdataset_t *neg = NULL, *negsig = NULL;
-       isc_result_t result = ISC_R_NOMEMORY;
+       isc_result_t result;
 
        CTRACE(ISC_LOG_DEBUG(3), "query_addnoqnameproof");
 
@@ -9179,23 +9176,15 @@ query_sign_nodata(query_ctx_t *qctx) {
                                dns_name_getlabelsequence(qname, skip, count,
                                                          found);
 
+                               /*
+                                * These ensure fname, rdataset, and
+                                * sigrdataset are non-NULL
+                                */
                                fixfname(qctx->client, &qctx->fname,
                                         &qctx->dbuf, &b);
                                fixrdataset(qctx->client, &qctx->rdataset);
                                fixrdataset(qctx->client, &qctx->sigrdataset);
-                               if (qctx->fname == NULL ||
-                                   qctx->rdataset == NULL ||
-                                   qctx->sigrdataset == NULL)
-                               {
-                                       CCTRACE(ISC_LOG_ERROR, "query_sign_"
-                                                              "nodata: "
-                                                              "failure "
-                                                              "getting "
-                                                              "closest "
-                                                              "encloser");
-                                       QUERY_ERROR(qctx, ISC_R_NOMEMORY);
-                                       return ns_query_done(qctx);
-                               }
+
                                /*
                                 * 'nearest' doesn't exist so
                                 * 'exist' is set to false.
index e907d72dfbec464c2c606ce769c851a7967b34d2..0892d931a362199b1ee91ea775795a85bebd309d 100644 (file)
@@ -117,10 +117,8 @@ nsec3param_change_test(const nsec3param_change_test_params_t *test) {
        result = dns_test_makezone("nsec3", &zone, NULL, false);
        assert_int_equal(result, ISC_R_SUCCESS);
 
-       result = dns_zone_setfile(
-               zone, TESTS_DIR "/testdata/nsec3param/nsec3.db.signed",
-               dns_masterformat_text, &dns_master_style_default);
-       assert_int_equal(result, ISC_R_SUCCESS);
+       dns_zone_setfile(zone, TESTS_DIR "/testdata/nsec3param/nsec3.db.signed",
+                        dns_masterformat_text, &dns_master_style_default);
 
        result = dns_zone_load(zone, false);
        assert_int_equal(result, ISC_R_SUCCESS);
index 30b1fe20081e4134a439e6de19e10774caa88406..f5679d983203da070deac3a0ac6630d0a012f595 100644 (file)
@@ -285,8 +285,7 @@ ISC_RUN_TEST_IMPL(updatesigs_next) {
                                 "testdata/master/master18.data");
        assert_int_equal(result, DNS_R_SEENINCLUDE);
 
-       result = dns_zone_setkeydirectory(zone, TESTS_DIR "/testkeys");
-       assert_int_equal(result, ISC_R_SUCCESS);
+       dns_zone_setkeydirectory(zone, TESTS_DIR "/testkeys");
 
        result = dns_zone_findkeys(zone, db, NULL, now, mctx, DNS_MAXZONEKEYS,
                                   zone_keys, &nkeys);
index 2b4f30d9f2c675b99bbdbe601604452059aecaa5..b7a15400752bc25ee511e713bc4ee7614010b3a6 100644 (file)
@@ -47,8 +47,7 @@ ISC_RUN_TEST_IMPL(symtab_grow) {
 
        UNUSED(state);
 
-       result = isc_symtab_create(mctx, 3, undefine, NULL, false, &st);
-       assert_int_equal(result, ISC_R_SUCCESS);
+       isc_symtab_create(mctx, 3, undefine, NULL, false, &st);
        assert_non_null(st);
 
        /* Nothing should be in the table yet */
index 7d2df9efd943c3a0f6f7bc5ee084f2576b788966..2ded8b3e3bea3c72acd6f9d9b915e2449cebfbbd 100644 (file)
@@ -72,17 +72,13 @@ dns_test_makeview(const char *name, bool with_dispatchmgr, bool with_cache,
                }
        }
 
-       result = dns_view_create(mctx, loopmgr, dispatchmgr, dns_rdataclass_in,
-                                name, &view);
+       dns_view_create(mctx, loopmgr, dispatchmgr, dns_rdataclass_in, name,
+                       &view);
 
        if (dispatchmgr != NULL) {
                dns_dispatchmgr_detach(&dispatchmgr);
        }
 
-       if (result != ISC_R_SUCCESS) {
-               return result;
-       }
-
        if (with_cache) {
                result = dns_cache_create(loopmgr, dns_rdataclass_in, "", mctx,
                                          &cache);
@@ -129,10 +125,7 @@ dns_test_makezone(const char *name, dns_zone_t **zonep, dns_view_t *view,
        if (result != ISC_R_SUCCESS) {
                goto detach_zone;
        }
-       result = dns_zone_setorigin(zone, origin);
-       if (result != ISC_R_SUCCESS) {
-               goto detach_zone;
-       }
+       dns_zone_setorigin(zone, origin);
 
        /*
         * If requested, create a view.
index 93384803f7a1968d7d8a058ab598636782b39b48..64c94d6f0563cb851cfd030e96ca52558a3607c0 100644 (file)
@@ -396,11 +396,7 @@ create_qctx_for_client(ns_client_t *client, query_ctx_t **qctxp) {
 
        isc_nmhandle_detach(&client->reqhandle);
 
-       if (*qctxp == NULL) {
-               return ISC_R_NOMEMORY;
-       } else {
-               return ISC_R_SUCCESS;
-       }
+       return ISC_R_SUCCESS;
 }
 
 isc_result_t