]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
[v9_10] clean up gcc -Wshadow warnings
authorEvan Hunt <each@isc.org>
Tue, 20 Jan 2015 21:43:35 +0000 (13:43 -0800)
committerEvan Hunt <each@isc.org>
Tue, 20 Jan 2015 21:43:35 +0000 (13:43 -0800)
4039. [cleanup] Cleaned up warnings from gcc -Wshadow. [RT #37381]

52 files changed:
CHANGES
bin/delv/delv.c
bin/dig/dighost.c
bin/dig/host.c
bin/dig/include/dig/dig.h
bin/dig/nslookup.c
bin/dnssec/dnssec-dsfromkey.c
bin/dnssec/dnssec-importkey.c
bin/dnssec/dnssec-keyfromlabel.c
bin/dnssec/dnssec-keygen.c
bin/dnssec/dnssec-settime.c
bin/dnssec/dnssec-signzone.c
bin/dnssec/dnssec-verify.c
bin/dnssec/dnssectool.c
bin/dnssec/dnssectool.h
bin/named/client.c
bin/named/query.c
bin/named/server.c
bin/named/zoneconf.c
bin/nsupdate/nsupdate.c
bin/rndc/rndc.c
configure
configure.in
lib/bind9/check.c
lib/bind9/getaddresses.c
lib/dns/diff.c
lib/dns/masterdump.c
lib/dns/message.c
lib/dns/nsec3.c
lib/dns/private.c
lib/dns/rbtdb.c
lib/dns/rdata/generic/nsec3_50.c
lib/dns/rdata/generic/rrsig_46.c
lib/dns/rdata/generic/sig_24.c
lib/dns/rdataset.c
lib/dns/resolver.c
lib/dns/spnego_asn1.c
lib/dns/tests/db_test.c
lib/dns/tests/dbversion_test.c
lib/dns/tests/gost_test.c
lib/dns/tests/master_test.c
lib/dns/tests/rbt_serialize_test.c
lib/dns/tests/zonemgr_test.c
lib/dns/tkey.c
lib/dns/zone.c
lib/isc/mem.c
lib/isc/ratelimiter.c
lib/isc/result.c
lib/isc/tests/hash_test.c
lib/isccfg/parser.c
lib/lwres/gethost.c
lib/samples/nsprobe.c

diff --git a/CHANGES b/CHANGES
index 514a04f8d18f4b78bdbc1389468d06d39b6d10d7..f036f900eeee03156b6d0e93665c8332e78427d6 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,3 +1,5 @@
+4039.  [cleanup]       Cleaned up warnings from gcc -Wshadow. [RT #37381]
+
 4038.  [bug]           Add 'rpz' flag to node and use it to determine whether
                        to call dns_rpz_delete.  This should prevent unbalanced
                        add / delete calls. [RT #36888]
index 955169167d66a64303dd5a1af0eaa2035d6a5c2c..69d49c35c8494bda9a03800cabb77bab982ca117 100644 (file)
@@ -444,8 +444,6 @@ printdata(dns_rdataset_t *rdataset, dns_name_t *owner,
                             result == ISC_R_SUCCESS;
                             result = dns_rdataset_next(rdataset))
                        {
-                               isc_region_t r;
-
                                if ((rdataset->attributes &
                                     DNS_RDATASETATTR_NEGATIVE) != 0)
                                        continue;
@@ -459,14 +457,12 @@ printdata(dns_rdataset_t *rdataset, dns_name_t *owner,
                                if (result != ISC_R_SUCCESS)
                                        break;
 
-                               isc_buffer_availableregion(&target, &r);
-                               if (r.length < 1) {
+                               if (isc_buffer_availablelength(&target) < 1) {
                                        result = ISC_R_NOSPACE;
                                        break;
                                }
 
-                               r.base[0] = '\n';
-                               isc_buffer_add(&target, 1);
+                               isc_buffer_putstr(&target, "\n");
 
                                dns_rdata_reset(&rdata);
                        }
index 5bc301e32a5a88b833e6b42082618ef5c256c829..97ba2f2109d9a173b5f639d8a72ef4e300668863 100644 (file)
@@ -1155,10 +1155,9 @@ parse_hmac(const char *hmac) {
  */
 static isc_result_t
 read_confkey(void) {
-       isc_log_t *lctx = NULL;
        cfg_parser_t *pctx = NULL;
        cfg_obj_t *file = NULL;
-       const cfg_obj_t *key = NULL;
+       const cfg_obj_t *keyobj = NULL;
        const cfg_obj_t *secretobj = NULL;
        const cfg_obj_t *algorithmobj = NULL;
        const char *keyname;
@@ -1169,7 +1168,7 @@ read_confkey(void) {
        if (! isc_file_exists(keyfile))
                return (ISC_R_FILENOTFOUND);
 
-       result = cfg_parser_create(mctx, lctx, &pctx);
+       result = cfg_parser_create(mctx, NULL, &pctx);
        if (result != ISC_R_SUCCESS)
                goto cleanup;
 
@@ -1178,16 +1177,16 @@ read_confkey(void) {
        if (result != ISC_R_SUCCESS)
                goto cleanup;
 
-       result = cfg_map_get(file, "key", &key);
+       result = cfg_map_get(file, "key", &keyobj);
        if (result != ISC_R_SUCCESS)
                goto cleanup;
 
-       (void) cfg_map_get(key, "secret", &secretobj);
-       (void) cfg_map_get(key, "algorithm", &algorithmobj);
+       (void) cfg_map_get(keyobj, "secret", &secretobj);
+       (void) cfg_map_get(keyobj, "algorithm", &algorithmobj);
        if (secretobj == NULL || algorithmobj == NULL)
                fatal("key must have algorithm and secret");
 
-       keyname = cfg_obj_asstring(cfg_map_getname(key));
+       keyname = cfg_obj_asstring(cfg_map_getname(keyobj));
        secretstr = cfg_obj_asstring(secretobj);
        algorithm = cfg_obj_asstring(algorithmobj);
 
@@ -1492,7 +1491,7 @@ setup_libs(void) {
  */
 static void
 add_opt(dns_message_t *msg, isc_uint16_t udpsize, isc_uint16_t edns,
-       isc_boolean_t dnssec, dns_ednsopt_t *ednsopts, size_t count)
+       isc_boolean_t dnssec, dns_ednsopt_t *opts, size_t count)
 {
        dns_rdataset_t *rdataset = NULL;
        isc_result_t result;
@@ -1502,7 +1501,7 @@ add_opt(dns_message_t *msg, isc_uint16_t udpsize, isc_uint16_t edns,
        if (dnssec)
                flags |= DNS_MESSAGEEXTFLAG_DO;
        result = dns_message_buildopt(msg, &rdataset, edns, udpsize, flags,
-                                     ednsopts, count);
+                                     opts, count);
        check_result(result, "dns_message_buildopt");
        result = dns_message_setopt(msg, rdataset);
        check_result(result, "dns_message_setopt");
@@ -2097,10 +2096,10 @@ insert_soa(dig_lookup_t *lookup) {
 
 #ifdef ISC_PLATFORM_USESIT
 static void
-compute_cookie(unsigned char *cookie, size_t len) {
+compute_cookie(unsigned char *clientcookie, size_t len) {
        /* XXXMPA need to fix, should be per server. */
        INSIST(len >= 8U);
-       memmove(cookie, cookie_secret, 8);
+       memmove(clientcookie, cookie_secret, 8);
 }
 #endif
 
@@ -2297,7 +2296,6 @@ setup_lookup(dig_lookup_t *lookup) {
                if (result != ISC_R_SUCCESS) {
                        dns_message_puttempname(lookup->sendmsg,
                                                &lookup->name);
-                       isc_buffer_init(&b, store, MXNAME);
                        fatal("'%s' is not a legal name "
                              "(%s)", lookup->textname,
                              isc_result_totext(result));
@@ -2415,7 +2413,6 @@ setup_lookup(dig_lookup_t *lookup) {
                        struct sockaddr_in *sin;
                        struct sockaddr_in6 *sin6;
                        size_t addrl;
-                       isc_buffer_t b;
 
                        sa = &lookup->ecs_addr->type.sa;
                        prefixlen = lookup->ecs_addr->length;
@@ -2455,8 +2452,6 @@ setup_lookup(dig_lookup_t *lookup) {
                        INSIST(i < DNS_EDNSOPTIONS);
                        opts[i].code = DNS_OPT_SIT;
                        if (lookup->sitvalue != NULL) {
-                               isc_buffer_t b;
-
                                isc_buffer_init(&b, sitbuf, sizeof(sitbuf));
                                result = isc_hex_decodestring(lookup->sitvalue,
                                                              &b);
@@ -3765,7 +3760,7 @@ recv_done(isc_task_t *task, isc_event_t *event) {
 #endif
                                printmessage(query, msg, ISC_TRUE);
                } else if (l->trace) {
-                       int n = 0;
+                       int nl = 0;
                        int count = msg->counts[DNS_SECTION_ANSWER];
 
                        debug("in TRACE code");
@@ -3776,13 +3771,13 @@ recv_done(isc_task_t *task, isc_event_t *event) {
                        if (l->trace_root || (l->ns_search_only && count > 0)) {
                                if (!l->trace_root)
                                        l->rdtype = dns_rdatatype_soa;
-                               n = followup_lookup(msg, query,
-                                                   DNS_SECTION_ANSWER);
+                               nl = followup_lookup(msg, query,
+                                                    DNS_SECTION_ANSWER);
                                l->trace_root = ISC_FALSE;
                        } else if (count == 0)
-                               n = followup_lookup(msg, query,
-                                                   DNS_SECTION_AUTHORITY);
-                       if (n == 0)
+                               nl = followup_lookup(msg, query,
+                                                    DNS_SECTION_AUTHORITY);
+                       if (nl == 0)
                                docancel = ISC_TRUE;
                } else {
                        debug("in NSSEARCH code");
@@ -3791,12 +3786,12 @@ recv_done(isc_task_t *task, isc_event_t *event) {
                                /*
                                 * This is the initial NS query.
                                 */
-                               int n;
+                               int nl;
 
                                l->rdtype = dns_rdatatype_soa;
-                               n = followup_lookup(msg, query,
-                                                   DNS_SECTION_ANSWER);
-                               if (n == 0)
+                               nl = followup_lookup(msg, query,
+                                                    DNS_SECTION_ANSWER);
+                               if (nl == 0)
                                        docancel = ISC_TRUE;
                                l->trace_root = ISC_FALSE;
                                usesearch = ISC_FALSE;
@@ -3926,12 +3921,12 @@ recv_done(isc_task_t *task, isc_event_t *event) {
  * routines, since they may be using a non-DNS system for these lookups.
  */
 isc_result_t
-get_address(char *host, in_port_t port, isc_sockaddr_t *sockaddr) {
+get_address(char *host, in_port_t myport, isc_sockaddr_t *sockaddr) {
        int count;
        isc_result_t result;
 
        isc_app_block();
-       result = bind9_getaddresses(host, port, sockaddr, 1, &count);
+       result = bind9_getaddresses(host, myport, sockaddr, 1, &count);
        isc_app_unblock();
        if (result != ISC_R_SUCCESS)
                return (result);
index a308824c045585b6f75746fad21ae92d168c5514..ef62f20efea21cef35888bfd2ce6c17f2cc236ce 100644 (file)
@@ -255,7 +255,7 @@ printsection(dns_message_t *msg, dns_section_t sectionid,
        isc_result_t result, loopresult;
        isc_region_t r;
        dns_name_t empty_name;
-       char t[4096];
+       char tbuf[4096];
        isc_boolean_t first;
        isc_boolean_t no_rdata;
 
@@ -279,7 +279,7 @@ printsection(dns_message_t *msg, dns_section_t sectionid,
                name = NULL;
                dns_message_currentname(msg, sectionid, &name);
 
-               isc_buffer_init(&target, t, sizeof(t));
+               isc_buffer_init(&target, tbuf, sizeof(tbuf));
                first = ISC_TRUE;
                print_name = name;
 
@@ -370,13 +370,13 @@ printrdata(dns_message_t *msg, dns_rdataset_t *rdataset, dns_name_t *owner,
        isc_buffer_t target;
        isc_result_t result;
        isc_region_t r;
-       char t[4096];
+       char tbuf[4096];
 
        UNUSED(msg);
        if (headers)
                printf(";; %s SECTION:\n", set_name);
 
-       isc_buffer_init(&target, t, sizeof(t));
+       isc_buffer_init(&target, tbuf, sizeof(tbuf));
 
        result = dns_rdataset_totext(rdataset, owner, ISC_FALSE, ISC_FALSE,
                                     &target);
index a52972322ca92df271343be3c1b5ba371a8a1bc6..7a0f91d653ff234bd533bb35c45ea8c24b0a1cb1 100644 (file)
@@ -15,8 +15,6 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-/* $Id: dig.h,v 1.114 2011/12/07 17:23:28 each Exp $ */
-
 #ifndef DIG_H
 #define DIG_H
 
@@ -267,7 +265,6 @@ extern isc_boolean_t check_ra, have_ipv4, have_ipv6, specified_source,
 extern in_port_t port;
 extern unsigned int timeout;
 extern isc_mem_t *mctx;
-extern dns_messageid_t id;
 extern int sendcount;
 extern int ndots;
 extern int lookup_counter;
index 051ed34b9415ac5d5620fc5286025e6a13795f5c..9745cd7160e5d0be7b8b10bd0f5f991246661a63 100644 (file)
@@ -425,13 +425,12 @@ printmessage(dig_query_t *query, dns_message_t *msg, isc_boolean_t headers) {
        puts("");
 
        if (!short_form) {
-               isc_boolean_t headers = ISC_TRUE;
                puts("------------");
                /*              detailheader(query, msg);*/
-               detailsection(query, msg, headers, DNS_SECTION_QUESTION);
-               detailsection(query, msg, headers, DNS_SECTION_ANSWER);
-               detailsection(query, msg, headers, DNS_SECTION_AUTHORITY);
-               detailsection(query, msg, headers, DNS_SECTION_ADDITIONAL);
+               detailsection(query, msg, ISC_TRUE, DNS_SECTION_QUESTION);
+               detailsection(query, msg, ISC_TRUE, DNS_SECTION_ANSWER);
+               detailsection(query, msg, ISC_TRUE, DNS_SECTION_AUTHORITY);
+               detailsection(query, msg, ISC_TRUE, DNS_SECTION_ADDITIONAL);
                puts("------------");
        }
 
index 77a5da9f2a3837867f8f1754e578c2f2897b7d62..310b745985cd5da7e41e6616a824da3f306a793b 100644 (file)
@@ -496,7 +496,7 @@ main(int argc, char **argv) {
                      isc_result_totext(result));
        isc_entropy_stopcallbacksources(ectx);
 
-       setup_logging(verbose, mctx, &log);
+       setup_logging(mctx, &log);
 
        dns_rdataset_init(&rdataset);
 
index a6ba566548519ee65eb09a0022415ad6c7d89882..251d3b2c10523cea915bb12683ecce9d9440e540 100644 (file)
@@ -387,7 +387,7 @@ main(int argc, char **argv) {
                      isc_result_totext(result));
        isc_entropy_stopcallbacksources(ectx);
 
-       setup_logging(verbose, mctx, &log);
+       setup_logging(mctx, &log);
 
        dns_rdataset_init(&rdataset);
 
index a805e5a5b25eb4e177b1debcbc419d9f454c5605..0a71d8e2de65739fffe4284cfc1e0ad81646ff4f 100644 (file)
@@ -331,7 +331,7 @@ main(int argc, char **argv) {
                fatal("could not initialize dst: %s",
                      isc_result_totext(ret));
 
-       setup_logging(verbose, mctx, &log);
+       setup_logging(mctx, &log);
 
        if (predecessor == NULL) {
                if (label == NULL)
index 167dc45f5d706101a72be6d865c2faf96c7c62dc..c2ae1e8fe215fee9b3d3b1977374ac6dbec9b0f8 100644 (file)
@@ -487,7 +487,7 @@ main(int argc, char **argv) {
                fatal("could not initialize dst: %s",
                      isc_result_totext(ret));
 
-       setup_logging(verbose, mctx, &log);
+       setup_logging(mctx, &log);
 
        if (predecessor == NULL) {
                if (prepub == -1)
index eac54c66382dfe3fe8f911d1610c3f2c4d74c67a..9658b4cc7ca4c5548a9cec542891b3b1a797f4e9 100644 (file)
@@ -169,7 +169,7 @@ main(int argc, char **argv) {
        if (result != ISC_R_SUCCESS)
                fatal("Out of memory");
 
-       setup_logging(verbose, mctx, &log);
+       setup_logging(mctx, &log);
 
 #ifdef PKCS11CRYPTO
        pk11_result_register();
@@ -343,7 +343,6 @@ main(int argc, char **argv) {
        isc_entropy_stopcallbacksources(ectx);
 
        if (predecessor != NULL) {
-               char keystr[DST_KEY_FORMATSIZE];
                int major, minor;
 
                if (prepub == -1)
index 03d6755376c7a889baf0df4236e7bfab805380fb..8dc112cccdfb888effea22f17f38cf3d27d0cdfc 100644 (file)
@@ -136,7 +136,7 @@ static isc_mem_t *mctx = NULL;
 static isc_entropy_t *ectx = NULL;
 static dns_ttl_t zone_soa_min_ttl;
 static dns_ttl_t soa_ttl;
-static FILE *fp = NULL;
+static FILE *outfp = NULL;
 static char *tempfile = NULL;
 static const dns_master_style_t *masterstyle;
 static dns_masterformat_t inputformat = dns_masterformat_text;
@@ -156,7 +156,7 @@ static dns_name_t *gorigin;         /* The database origin */
 static int nsec3flags = 0;
 static dns_iterations_t nsec3iter = 10U;
 static unsigned char saltbuf[255];
-static unsigned char *salt = saltbuf;
+static unsigned char *gsalt = saltbuf;
 static size_t salt_length = 0;
 static isc_task_t *master = NULL;
 static unsigned int ntasks = 0;
@@ -208,7 +208,7 @@ dumpnode(dns_name_t *name, dns_dbnode_t *node) {
 
        if (!output_dnssec_only) {
                result = dns_master_dumpnodetostream(mctx, gdb, gversion, node,
-                                                    name, masterstyle, fp);
+                                                    name, masterstyle, outfp);
                check_result(result, "dns_master_dumpnodetostream");
                return;
        }
@@ -250,7 +250,7 @@ dumpnode(dns_name_t *name, dns_dbnode_t *node) {
                check_result(result, "dns_master_rdatasettotext");
 
                isc_buffer_usedregion(buffer, &r);
-               result = isc_stdio_write(r.base, 1, r.length, fp, NULL);
+               result = isc_stdio_write(r.base, 1, r.length, outfp, NULL);
                check_result(result, "isc_stdio_write");
                isc_buffer_clear(buffer);
 
@@ -291,8 +291,6 @@ signwithkey(dns_name_t *name, dns_rdataset_t *rdataset, dst_key_t *key,
                                 mctx, &b, &trdata);
        isc_entropy_stopcallbacksources(ectx);
        if (result != ISC_R_SUCCESS) {
-               char keystr[DST_KEY_FORMATSIZE];
-               dst_key_format(key, keystr, sizeof(keystr));
                fatal("dnskey '%s' failed to sign data: %s",
                      keystr, isc_result_totext(result));
        }
@@ -742,7 +740,7 @@ hashlist_add(hashlist_t *l, const unsigned char *hash, size_t len)
 static void
 hashlist_add_dns_name(hashlist_t *l, /*const*/ dns_name_t *name,
                      unsigned int hashalg, unsigned int iterations,
-                     const unsigned char *salt, size_t salt_length,
+                     const unsigned char *salt, size_t salt_len,
                      isc_boolean_t speculative)
 {
        char nametext[DNS_NAME_FORMATSIZE];
@@ -751,7 +749,7 @@ hashlist_add_dns_name(hashlist_t *l, /*const*/ dns_name_t *name,
        size_t i;
 
        len = isc_iterated_hash(hash, hashalg, iterations,
-                               salt, (int)salt_length,
+                               salt, (int)salt_len,
                                name->ndata, name->length);
        if (verbose) {
                dns_name_format(name, nametext, sizeof nametext);
@@ -833,7 +831,7 @@ hashlist_exists(const hashlist_t *l,
 static void
 addnowildcardhash(hashlist_t *l, /*const*/ dns_name_t *name,
                  unsigned int hashalg, unsigned int iterations,
-                 const unsigned char *salt, size_t salt_length)
+                 const unsigned char *salt, size_t salt_len)
 {
        dns_fixedname_t fixed;
        dns_name_t *wild;
@@ -860,7 +858,7 @@ addnowildcardhash(hashlist_t *l, /*const*/ dns_name_t *name,
                fprintf(stderr, "adding no-wildcardhash for %s\n", namestr);
        }
 
-       hashlist_add_dns_name(l, wild, hashalg, iterations, salt, salt_length,
+       hashlist_add_dns_name(l, wild, hashalg, iterations, salt, salt_len,
                              ISC_TRUE);
 }
 
@@ -1835,7 +1833,7 @@ nsecify(void) {
 }
 
 static void
-addnsec3param(const unsigned char *salt, size_t salt_length,
+addnsec3param(const unsigned char *salt, size_t salt_len,
              dns_iterations_t iterations)
 {
        dns_dbnode_t *node = NULL;
@@ -1856,7 +1854,7 @@ addnsec3param(const unsigned char *salt, size_t salt_length,
        nsec3param.flags = 0;
        nsec3param.hash = unknownalg ? DNS_NSEC3_UNKNOWNALG : dns_hash_sha1;
        nsec3param.iterations = iterations;
-       nsec3param.salt_length = (unsigned char)salt_length;
+       nsec3param.salt_length = (unsigned char)salt_len;
        DE_CONST(salt, nsec3param.salt);
 
        isc_buffer_init(&b, nsec3parambuf, sizeof(nsec3parambuf));
@@ -1895,7 +1893,7 @@ addnsec3param(const unsigned char *salt, size_t salt_length,
 
 static void
 addnsec3(dns_name_t *name, dns_dbnode_t *node,
-        const unsigned char *salt, size_t salt_length,
+        const unsigned char *salt, size_t salt_len,
         unsigned int iterations, hashlist_t *hashlist,
         dns_ttl_t ttl)
 {
@@ -1909,7 +1907,7 @@ addnsec3(dns_name_t *name, dns_dbnode_t *node,
        isc_result_t result;
        dns_dbnode_t *nsec3node = NULL;
        char namebuf[DNS_NAME_FORMATSIZE];
-       size_t hash_length;
+       size_t hash_len;
 
        dns_name_format(name, namebuf, sizeof(namebuf));
 
@@ -1917,16 +1915,16 @@ addnsec3(dns_name_t *name, dns_dbnode_t *node,
        dns_rdataset_init(&rdataset);
 
        dns_name_downcase(name, name, NULL);
-       result = dns_nsec3_hashname(&hashname, hash, &hash_length,
+       result = dns_nsec3_hashname(&hashname, hash, &hash_len,
                                    name, gorigin, dns_hash_sha1, iterations,
-                                   salt, salt_length);
+                                   salt, salt_len);
        check_result(result, "addnsec3: dns_nsec3_hashname()");
        nexthash = hashlist_findnext(hashlist, hash);
        result = dns_nsec3_buildrdata(gdb, gversion, node,
                                      unknownalg ?
                                          DNS_NSEC3_UNKNOWNALG : dns_hash_sha1,
                                      nsec3flags, iterations,
-                                     salt, salt_length,
+                                     salt, salt_len,
                                      nexthash, ISC_SHA1_DIGESTLENGTH,
                                      nsec3buffer, &rdata);
        check_result(result, "addnsec3: dns_nsec3_buildrdata()");
@@ -1962,7 +1960,7 @@ addnsec3(dns_name_t *name, dns_dbnode_t *node,
 static void
 nsec3clean(dns_name_t *name, dns_dbnode_t *node,
           unsigned int hashalg, unsigned int iterations,
-          const unsigned char *salt, size_t salt_length, hashlist_t *hashlist)
+          const unsigned char *salt, size_t salt_len, hashlist_t *hashlist)
 {
        dns_label_t label;
        dns_rdata_nsec3_t nsec3;
@@ -2022,8 +2020,8 @@ nsec3clean(dns_name_t *name, dns_dbnode_t *node,
                check_result(result, "dns_rdata_tostruct");
                if (exists && nsec3.hash == hashalg &&
                    nsec3.iterations == iterations &&
-                   nsec3.salt_length == salt_length &&
-                   !memcmp(nsec3.salt, salt, salt_length))
+                   nsec3.salt_length == salt_len &&
+                   !memcmp(nsec3.salt, salt, salt_len))
                        continue;
                rdatalist.rdclass = rdata.rdclass;
                rdatalist.type = rdata.type;
@@ -2183,7 +2181,7 @@ cleanup_zone(void) {
  */
 static void
 nsec3ify(unsigned int hashalg, dns_iterations_t iterations,
-        const unsigned char *salt, size_t salt_length, hashlist_t *hashlist)
+        const unsigned char *salt, size_t salt_len, hashlist_t *hashlist)
 {
        dns_dbiterator_t *dbiter = NULL;
        dns_dbnode_t *node = NULL, *nextnode = NULL;
@@ -2279,7 +2277,7 @@ nsec3ify(unsigned int hashalg, dns_iterations_t iterations,
                              isc_result_totext(result));
                dns_name_downcase(name, name, NULL);
                hashlist_add_dns_name(hashlist, name, hashalg, iterations,
-                                     salt, salt_length, ISC_FALSE);
+                                     salt, salt_len, ISC_FALSE);
                dns_db_detachnode(gdb, &node);
                /*
                 * Add hashs for empty nodes.  Use closest encloser logic.
@@ -2290,16 +2288,16 @@ nsec3ify(unsigned int hashalg, dns_iterations_t iterations,
                dns_name_downcase(nextname, nextname, NULL);
                dns_name_fullcompare(name, nextname, &order, &nlabels);
                addnowildcardhash(hashlist, name, hashalg, iterations,
-                                 salt, salt_length);
+                                 salt, salt_len);
                count = dns_name_countlabels(nextname);
                while (count > nlabels + 1) {
                        count--;
                        dns_name_split(nextname, count, NULL, nextname);
                        hashlist_add_dns_name(hashlist, nextname, hashalg,
-                                             iterations, salt, salt_length,
+                                             iterations, salt, salt_len,
                                              ISC_FALSE);
                        addnowildcardhash(hashlist, nextname, hashalg,
-                                         iterations, salt, salt_length);
+                                         iterations, salt, salt_len);
                }
        }
        dns_dbiterator_destroy(&dbiter);
@@ -2322,7 +2320,7 @@ nsec3ify(unsigned int hashalg, dns_iterations_t iterations,
        zonecut = NULL;
        done = ISC_FALSE;
 
-       addnsec3param(salt, salt_length, iterations);
+       addnsec3param(salt, salt_len, iterations);
 
        /*
         * Clean out NSEC3 records which don't match this chain.
@@ -2335,7 +2333,7 @@ nsec3ify(unsigned int hashalg, dns_iterations_t iterations,
             result = dns_dbiterator_next(dbiter)) {
                result = dns_dbiterator_current(dbiter, &node, name);
                check_dns_dbiterator_current(result);
-               nsec3clean(name, node, hashalg, iterations, salt, salt_length,
+               nsec3clean(name, node, hashalg, iterations, salt, salt_len,
                           hashlist);
                dns_db_detachnode(gdb, &node);
        }
@@ -2409,7 +2407,7 @@ nsec3ify(unsigned int hashalg, dns_iterations_t iterations,
                 * We need to pause here to release the lock on the database.
                 */
                dns_dbiterator_pause(dbiter);
-               addnsec3(name, node, salt, salt_length, iterations,
+               addnsec3(name, node, salt, salt_len, iterations,
                         hashlist, zone_soa_min_ttl);
                dns_db_detachnode(gdb, &node);
                /*
@@ -2420,7 +2418,7 @@ nsec3ify(unsigned int hashalg, dns_iterations_t iterations,
                while (count > nlabels + 1) {
                        count--;
                        dns_name_split(nextname, count, NULL, nextname);
-                       addnsec3(nextname, NULL, salt, salt_length,
+                       addnsec3(nextname, NULL, salt, salt_len,
                                 iterations, hashlist, zone_soa_min_ttl);
                }
        }
@@ -2682,7 +2680,7 @@ warnifallksk(dns_db_t *db) {
 }
 
 static void
-set_nsec3params(isc_boolean_t update_chain, isc_boolean_t set_salt,
+set_nsec3params(isc_boolean_t update, isc_boolean_t set_salt,
                isc_boolean_t set_optout, isc_boolean_t set_iter)
 {
        isc_result_t result;
@@ -2710,7 +2708,7 @@ set_nsec3params(isc_boolean_t update_chain, isc_boolean_t set_salt,
 
        nsec_datatype = dns_rdatatype_nsec3;
 
-       if (!update_chain && set_salt) {
+       if (!update && set_salt) {
                if (salt_length != orig_saltlen ||
                    memcmp(saltbuf, orig_salt, salt_length) != 0)
                        fatal("An NSEC3 chain exists with a different salt. "
@@ -2718,10 +2716,10 @@ set_nsec3params(isc_boolean_t update_chain, isc_boolean_t set_salt,
        } else if (!set_salt) {
                salt_length = orig_saltlen;
                memmove(saltbuf, orig_salt, orig_saltlen);
-               salt = saltbuf;
+               gsalt = saltbuf;
        }
 
-       if (!update_chain && set_iter) {
+       if (!update && set_iter) {
                if (nsec3iter != orig_iter)
                        fatal("An NSEC3 chain exists with different "
                              "iterations. Use -u to update it.");
@@ -2755,7 +2753,7 @@ set_nsec3params(isc_boolean_t update_chain, isc_boolean_t set_salt,
        result = dns_rdata_tostruct(&rdata, &nsec3, NULL);
        check_result(result, "dns_rdata_tostruct");
 
-       if (!update_chain && set_optout) {
+       if (!update && set_optout) {
                if (nsec3flags != nsec3.flags)
                        fatal("An NSEC3 chain exists with%s OPTOUT. "
                              "Use -u -%s to %s it.",
@@ -3465,7 +3463,7 @@ main(int argc, char *argv[]) {
        if (directory == NULL)
                directory = ".";
 
-       setup_logging(verbose, mctx, &log);
+       setup_logging(mctx, &log);
 
        argc -= isc_commandline_index;
        argv += isc_commandline_index;
@@ -3681,7 +3679,7 @@ main(int argc, char *argv[]) {
 
        if (!nonsecify) {
                if (IS_NSEC3)
-                       nsec3ify(dns_hash_sha1, nsec3iter, salt, salt_length,
+                       nsec3ify(dns_hash_sha1, nsec3iter, gsalt, salt_length,
                                 &hashlist);
                else
                        nsecify();
@@ -3697,7 +3695,7 @@ main(int argc, char *argv[]) {
        }
 
        if (output_stdout) {
-               fp = stdout;
+               outfp = stdout;
                if (outputformatstr == NULL)
                        masterstyle = &dns_master_style_full;
        } else {
@@ -3710,9 +3708,9 @@ main(int argc, char *argv[]) {
                check_result(result, "isc_file_mktemplate");
 
                if (outputformat == dns_masterformat_text)
-                       result = isc_file_openunique(tempfile, &fp);
+                       result = isc_file_openunique(tempfile, &outfp);
                else
-                       result = isc_file_bopenunique(tempfile, &fp);
+                       result = isc_file_bopenunique(tempfile, &outfp);
                if (result != ISC_R_SUCCESS)
                        fatal("failed to open temporary output file: %s",
                              isc_result_totext(result));
@@ -3720,8 +3718,8 @@ main(int argc, char *argv[]) {
                setfatalcallback(&removetempfile);
        }
 
-       print_time(fp);
-       print_version(fp);
+       print_time(outfp);
+       print_version(outfp);
 
        result = isc_taskmgr_create(mctx, ntasks, 0, &taskmgr);
        if (result != ISC_R_SUCCESS)
@@ -3791,7 +3789,7 @@ main(int argc, char *argv[]) {
                }
                result = dns_master_dumptostream3(mctx, gdb, gversion,
                                                  masterstyle, outputformat,
-                                                 &header, fp);
+                                                 &header, outfp);
                check_result(result, "dns_master_dumptostream3");
        }
 
@@ -3800,7 +3798,7 @@ main(int argc, char *argv[]) {
                DESTROYLOCK(&statslock);
 
        if (!output_stdout) {
-               result = isc_stdio_close(fp);
+               result = isc_stdio_close(outfp);
                check_result(result, "isc_stdio_close");
                removefile = ISC_FALSE;
 
index 6b0dc9fba9c91ff8db2a5bd8f149cba8223c240a..7faa2326d3b4f2843e64bc549d449f44cfe0ede8 100644 (file)
@@ -282,7 +282,7 @@ main(int argc, char *argv[]) {
 
        rdclass = strtoclass(classname);
 
-       setup_logging(verbose, mctx, &log);
+       setup_logging(mctx, &log);
 
        argc -= isc_commandline_index;
        argv += isc_commandline_index;
index b1d1ed664bf2b77cb7204c6014ec21a44ff1e972..bc0ccd67f67a1b880bb54750c8d19068644468af 100644 (file)
@@ -120,8 +120,8 @@ vbprintf(int level, const char *fmt, ...) {
 }
 
 void
-version(const char *program) {
-       fprintf(stderr, "%s %s\n", program, VERSION);
+version(const char *name) {
+       fprintf(stderr, "%s %s\n", name, VERSION);
        exit(0);
 }
 
@@ -149,7 +149,7 @@ sig_format(dns_rdata_rrsig_t *sig, char *cp, unsigned int size) {
 }
 
 void
-setup_logging(int verbose, isc_mem_t *mctx, isc_log_t **logp) {
+setup_logging(isc_mem_t *mctx, isc_log_t **logp) {
        isc_result_t result;
        isc_logdestination_t destination;
        isc_logconfig_t *logconfig = NULL;
index f51bd0001a7b39edc255851ce4e3127b15e2f16f..3df2c69333c210b6500c959081cef20eb12c6ab4 100644 (file)
@@ -57,7 +57,7 @@ sig_format(dns_rdata_rrsig_t *sig, char *cp, unsigned int size);
 #define SIG_FORMATSIZE (DNS_NAME_FORMATSIZE + DNS_SECALG_FORMATSIZE + sizeof("65535"))
 
 void
-setup_logging(int verbose, isc_mem_t *mctx, isc_log_t **logp);
+setup_logging(isc_mem_t *mctx, isc_log_t **logp);
 
 void
 cleanup_logging(isc_log_t **logp);
index 68d14a942d91f991bbe2bf74ee8b87d9bebfdbc5..5d8a5790db22eb99978f70a2bbc65a70b4c91fec 100644 (file)
@@ -1412,7 +1412,6 @@ ns_client_addopt(ns_client_t *client, dns_message_t *message,
            (ns_g_server->server_id != NULL ||
             ns_g_server->server_usehostname)) {
                if (ns_g_server->server_usehostname) {
-                       isc_result_t result;
                        result = ns_os_gethostname(nsid, sizeof(nsid));
                        if (result != ISC_R_SUCCESS) {
                                goto no_nsid;
index 092212ed15a4e10d92995ab268770c91f285b5f1..f9d16d6b5411320c6b5ab3fb50a5ecd56bbda675 100644 (file)
@@ -6806,8 +6806,6 @@ query_find(ns_client_t *client, dns_fetchevent_t *event, dns_rdatatype_t qtype)
                         * We're authoritative for an ancestor of QNAME.
                         */
                        if (!USECACHE(client) || !RECURSIONOK(client)) {
-                               dns_fixedname_t fixed;
-
                                dns_fixedname_init(&fixed);
                                dns_name_copy(fname,
                                              dns_fixedname_name(&fixed), NULL);
@@ -6953,8 +6951,6 @@ query_find(ns_client_t *client, dns_fetchevent_t *event, dns_rdatatype_t qtype)
                                else
                                        RECURSE_ERROR(result);
                        } else {
-                               dns_fixedname_t fixed;
-
                                dns_fixedname_init(&fixed);
                                dns_name_copy(fname,
                                              dns_fixedname_name(&fixed), NULL);
index f5060d7f4ae1da25f5bd8975a77de39a63013752..e7cc327cef2a5d972f94396d2de81741ae07c87c 100644 (file)
@@ -2664,7 +2664,6 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
        result = ns_config_get(maps, "dns64", &obj);
        if (result == ISC_R_SUCCESS && strcmp(view->name, "_bind") &&
            strcmp(view->name, "_meta")) {
-               const cfg_listelt_t *element;
                isc_netaddr_t na, suffix, *sp;
                unsigned int prefixlen;
                const char *server, *contact;
@@ -3173,7 +3172,6 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
         */
        {
                const cfg_obj_t *peers = NULL;
-               const cfg_listelt_t *element;
                dns_peerlist_t *newpeers = NULL;
 
                (void)ns_config_get(cfgmaps, "server", &peers);
@@ -3198,7 +3196,6 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
         */
        {
                const cfg_obj_t *rrsetorder = NULL;
-               const cfg_listelt_t *element;
 
                (void)ns_config_get(maps, "rrset-order", &rrsetorder);
                CHECK(dns_order_create(mctx, &order));
@@ -3501,10 +3498,10 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
                        view->prefetch_trigger = 10;
                eligible = cfg_tuple_get(obj, "eligible");
                if (cfg_obj_isvoid(eligible)) {
-                       int i;
-                       for (i = 1; maps[i] != NULL; i++) {
+                       int m;
+                       for (m = 1; maps[m] != NULL; m++) {
                                obj = NULL;
-                               result = ns_config_get(&maps[i],
+                               result = ns_config_get(&maps[m],
                                                       "prefetch", &obj);
                                INSIST(result == ISC_R_SUCCESS);
                                eligible = cfg_tuple_get(obj, "eligible");
@@ -3549,18 +3546,13 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
                     element != NULL;
                     element = cfg_list_next(element))
                {
-                       const char *str;
-                       isc_buffer_t b;
                        dns_name_t *dlv;
 
                        obj = cfg_listelt_value(element);
-                       str = cfg_obj_asstring(cfg_tuple_get(obj,
-                                                            "trust-anchor"));
-                       isc_buffer_constinit(&b, str, strlen(str));
-                       isc_buffer_add(&b, strlen(str));
+                       obj = cfg_tuple_get(obj, "trust-anchor");
                        dlv = dns_fixedname_name(&view->dlv_fixed);
-                       CHECK(dns_name_fromtext(dlv, &b, dns_rootname,
-                                               DNS_NAME_DOWNCASE, NULL));
+                       CHECK(dns_name_fromstring(dlv, cfg_obj_asstring(obj),
+                                                 DNS_NAME_DOWNCASE, NULL));
                        view->dlv = dns_fixedname_name(&view->dlv_fixed);
                }
        } else
@@ -3594,28 +3586,22 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
        obj = NULL;
        result = ns_config_get(maps, "root-delegation-only", &obj);
        if (result == ISC_R_SUCCESS) {
+               dns_fixedname_t fixed;
+               dns_name_t *name;
+               const cfg_obj_t *exclude;
+
                dns_view_setrootdelonly(view, ISC_TRUE);
-               if (!cfg_obj_isvoid(obj)) {
-                       dns_fixedname_t fixed;
-                       dns_name_t *name;
-                       isc_buffer_t b;
-                       const char *str;
-                       const cfg_obj_t *exclude;
 
-                       dns_fixedname_init(&fixed);
-                       name = dns_fixedname_name(&fixed);
-                       for (element = cfg_list_first(obj);
-                            element != NULL;
-                            element = cfg_list_next(element)) {
-                               exclude = cfg_listelt_value(element);
-                               str = cfg_obj_asstring(exclude);
-                               isc_buffer_constinit(&b, str, strlen(str));
-                               isc_buffer_add(&b, strlen(str));
-                               CHECK(dns_name_fromtext(name, &b, dns_rootname,
-                                                       0, NULL));
-                               CHECK(dns_view_excludedelegationonly(view,
-                                                                    name));
-                       }
+               dns_fixedname_init(&fixed);
+               name = dns_fixedname_name(&fixed);
+               for (element = cfg_list_first(obj);
+                    element != NULL;
+                    element = cfg_list_next(element)) {
+                       exclude = cfg_listelt_value(element);
+                       CHECK(dns_name_fromstring(name,
+                                                 cfg_obj_asstring(exclude),
+                                                 0, NULL));
+                       CHECK(dns_view_excludedelegationonly(view, name));
                }
        } else
                dns_view_setrootdelonly(view, ISC_FALSE);
@@ -3644,7 +3630,6 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
                dns_fixedname_t fixed;
                dns_name_t *name;
                isc_buffer_t buffer;
-               const char *str;
                char server[DNS_NAME_FORMATSIZE + 1];
                char contact[DNS_NAME_FORMATSIZE + 1];
                const char *empty_dbtype[4] =
@@ -3658,11 +3643,8 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
                obj = NULL;
                result = ns_config_get(maps, "empty-server", &obj);
                if (result == ISC_R_SUCCESS) {
-                       str = cfg_obj_asstring(obj);
-                       isc_buffer_constinit(&buffer, str, strlen(str));
-                       isc_buffer_add(&buffer, strlen(str));
-                       CHECK(dns_name_fromtext(name, &buffer, dns_rootname, 0,
-                                               NULL));
+                       CHECK(dns_name_fromstring(name, cfg_obj_asstring(obj),
+                                                 0, NULL));
                        isc_buffer_init(&buffer, server, sizeof(server) - 1);
                        CHECK(dns_name_totext(name, ISC_FALSE, &buffer));
                        server[isc_buffer_usedlength(&buffer)] = 0;
@@ -3673,11 +3655,8 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
                obj = NULL;
                result = ns_config_get(maps, "empty-contact", &obj);
                if (result == ISC_R_SUCCESS) {
-                       str = cfg_obj_asstring(obj);
-                       isc_buffer_constinit(&buffer, str, strlen(str));
-                       isc_buffer_add(&buffer, strlen(str));
-                       CHECK(dns_name_fromtext(name, &buffer, dns_rootname, 0,
-                                               NULL));
+                       CHECK(dns_name_fromstring(name, cfg_obj_asstring(obj),
+                                                0, NULL));
                        isc_buffer_init(&buffer, contact, sizeof(contact) - 1);
                        CHECK(dns_name_totext(name, ISC_FALSE, &buffer));
                        contact[isc_buffer_usedlength(&buffer)] = 0;
@@ -3709,16 +3688,12 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
                     empty != NULL;
                     empty = empty_zones[++empty_zone])
                {
-                       dns_forwarders_t *forwarders = NULL;
-                       dns_view_t *pview = NULL;
+                       dns_forwarders_t *dnsforwarders = NULL;
 
-                       isc_buffer_constinit(&buffer, empty, strlen(empty));
-                       isc_buffer_add(&buffer, strlen(empty));
                        /*
                         * Look for zone on drop list.
                         */
-                       CHECK(dns_name_fromtext(name, &buffer, dns_rootname, 0,
-                                               NULL));
+                       CHECK(dns_name_fromstring(name, empty, 0, NULL));
                        if (disablelist != NULL &&
                            on_disable_list(disablelist, name))
                                continue;
@@ -3737,9 +3712,9 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist,
                         * empty zone for it.
                         */
                        result = dns_fwdtable_find(view->fwdtable, name,
-                                                  &forwarders);
+                                                  &dnsforwarders);
                        if (result == ISC_R_SUCCESS &&
-                           forwarders->fwdpolicy == dns_fwdpolicy_only)
+                           dnsforwarders->fwdpolicy == dns_fwdpolicy_only)
                                continue;
 
                        /*
@@ -6130,7 +6105,6 @@ load_configuration(const char *filename, ns_server_t *server,
                                (void)cfg_map_get(logobj, "category",
                                                  &categories);
                        if (categories != NULL) {
-                               const cfg_listelt_t *element;
                                for (element = cfg_list_first(categories);
                                     element != NULL;
                                     element = cfg_list_next(element))
index 45f17b229423daf09d7dde6b5b2042134b52d415..4ac0edc813f712a6c013789a64949a87680933cc 100644 (file)
@@ -1513,7 +1513,7 @@ ns_zone_configure(const cfg_obj_t *config, const cfg_obj_t *vconfig,
                        dns_zone_setoption(mayberaw, DNS_ZONEOPT_CHECKINTEGRITY,
                                           cfg_obj_asboolean(obj));
                } else {
-                       isc_boolean_t check = ISC_FALSE;
+                       check = ISC_FALSE;
                        result = ns_config_get(nodefault, "check-integrity",
                                               &obj);
                        if (result == ISC_R_SUCCESS)
index 37dae5535cb53ed3a4ac7879aed1b444e088b4fb..6649bebc6f07cd7739192f08623123dc6a8b3e0f 100644 (file)
@@ -15,8 +15,6 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-/* $Id: nsupdate.c,v 1.198 2011/12/16 23:01:16 each Exp $ */
-
 /*! \file */
 
 #include <config.h>
@@ -140,8 +138,8 @@ static isc_boolean_t local_only = ISC_FALSE;
 static isc_taskmgr_t *taskmgr = NULL;
 static isc_task_t *global_task = NULL;
 static isc_event_t *global_event = NULL;
-static isc_log_t *lctx = NULL;
-static isc_mem_t *mctx = NULL;
+static isc_log_t *glctx = NULL;
+static isc_mem_t *gmctx = NULL;
 static dns_dispatchmgr_t *dispatchmgr = NULL;
 static dns_requestmgr_t *requestmgr = NULL;
 static isc_socketmgr_t *socketmgr = NULL;
@@ -151,7 +149,7 @@ static dns_dispatch_t *dispatchv6 = NULL;
 static dns_message_t *updatemsg = NULL;
 static dns_fixedname_t fuserzone;
 static dns_name_t *userzone = NULL;
-static dns_name_t *zonename = NULL;
+static dns_name_t *zname = NULL;
 static dns_name_t tmpzonename;
 static dns_name_t restart_master;
 static dns_tsig_keyring_t *gssring = NULL;
@@ -248,8 +246,7 @@ struct entropysource {
 static ISC_LIST(entropysource_t) sources;
 
 static void
-setup_entropy(isc_mem_t *mctx, const char *randomfile, isc_entropy_t **ectx)
-{
+setup_entropy(isc_mem_t *mctx, const char *randomfile, isc_entropy_t **ectx) {
        isc_result_t result;
        isc_entropysource_t *source = NULL;
        entropysource_t *elt;
@@ -303,7 +300,7 @@ static void
 master_from_servers(void) {
 
        if (master_servers != NULL && master_servers != servers)
-               isc_mem_put(mctx, master_servers,
+               isc_mem_put(gmctx, master_servers,
                            master_total * sizeof(isc_sockaddr_t));
        master_servers = servers;
        master_total = ns_total;
@@ -431,7 +428,7 @@ reset_system(void) {
        if (updatemsg != NULL)
                dns_message_reset(updatemsg, DNS_MESSAGE_INTENTRENDER);
        else {
-               result = dns_message_create(mctx, DNS_MESSAGE_INTENTRENDER,
+               result = dns_message_create(gmctx, DNS_MESSAGE_INTENTRENDER,
                                            &updatemsg);
                check_result(result, "dns_message_create");
        }
@@ -536,13 +533,13 @@ setup_keystr(void) {
        char *secretstr;
        char *s, *n;
        dns_fixedname_t fkeyname;
-       dns_name_t *keyname;
+       dns_name_t *mykeyname;
        char *name;
        dns_name_t *hmacname = NULL;
        isc_uint16_t digestbits = 0;
 
        dns_fixedname_init(&fkeyname);
-       keyname = dns_fixedname_name(&fkeyname);
+       mykeyname = dns_fixedname_name(&fkeyname);
 
        debug("Creating key...");
 
@@ -567,11 +564,12 @@ setup_keystr(void) {
        isc_buffer_add(&keynamesrc, (unsigned int)(n - name));
 
        debug("namefromtext");
-       result = dns_name_fromtext(keyname, &keynamesrc, dns_rootname, 0, NULL);
+       result = dns_name_fromtext(mykeyname, &keynamesrc, dns_rootname, 0,
+                                  NULL);
        check_result(result, "dns_name_fromtext");
 
        secretlen = strlen(secretstr) * 3 / 4;
-       secret = isc_mem_allocate(mctx, secretlen);
+       secret = isc_mem_allocate(gmctx, secretlen);
        if (secret == NULL)
                fatal("out of memory");
 
@@ -586,8 +584,8 @@ setup_keystr(void) {
        secretlen = isc_buffer_usedlength(&secretbuf);
 
        debug("keycreate");
-       result = dns_tsigkey_create(keyname, hmacname, secret, secretlen,
-                                   ISC_FALSE, NULL, 0, 0, mctx, NULL,
+       result = dns_tsigkey_create(mykeyname, hmacname, secret, secretlen,
+                                   ISC_FALSE, NULL, 0, 0, gmctx, NULL,
                                    &tsigkey);
        if (result != ISC_R_SUCCESS)
                fprintf(stderr, "could not create key from %s: %s\n",
@@ -596,7 +594,7 @@ setup_keystr(void) {
                dst_key_setbits(tsigkey->key, digestbits);
  failure:
        if (secret != NULL)
-               isc_mem_free(mctx, secret);
+               isc_mem_free(gmctx, secret);
 }
 
 /*
@@ -609,7 +607,7 @@ read_sessionkey(isc_mem_t *mctx, isc_log_t *lctx) {
        const cfg_obj_t *key = NULL;
        const cfg_obj_t *secretobj = NULL;
        const cfg_obj_t *algorithmobj = NULL;
-       const char *keyname;
+       const char *mykeyname;
        const char *secretstr;
        const char *algorithm;
        isc_result_t result;
@@ -636,13 +634,13 @@ read_sessionkey(isc_mem_t *mctx, isc_log_t *lctx) {
        if (secretobj == NULL || algorithmobj == NULL)
                fatal("key must have algorithm and secret");
 
-       keyname = cfg_obj_asstring(cfg_map_getname(key));
+       mykeyname = cfg_obj_asstring(cfg_map_getname(key));
        secretstr = cfg_obj_asstring(secretobj);
        algorithm = cfg_obj_asstring(algorithmobj);
 
-       len = strlen(algorithm) + strlen(keyname) + strlen(secretstr) + 3;
+       len = strlen(algorithm) + strlen(mykeyname) + strlen(secretstr) + 3;
        keystr = isc_mem_allocate(mctx, len);
-       snprintf(keystr, len, "%s:%s:%s", algorithm, keyname, secretstr);
+       snprintf(keystr, len, "%s:%s:%s", algorithm, mykeyname, secretstr);
        setup_keystr();
 
  cleanup:
@@ -735,17 +733,17 @@ doshutdown(void) {
         * to NULL.
         */
        if (master_servers != NULL && master_servers != servers)
-               isc_mem_put(mctx, master_servers,
+               isc_mem_put(gmctx, master_servers,
                            master_total * sizeof(isc_sockaddr_t));
 
        if (servers != NULL)
-               isc_mem_put(mctx, servers, ns_total * sizeof(isc_sockaddr_t));
+               isc_mem_put(gmctx, servers, ns_total * sizeof(isc_sockaddr_t));
 
        if (localaddr4 != NULL)
-               isc_mem_put(mctx, localaddr4, sizeof(isc_sockaddr_t));
+               isc_mem_put(gmctx, localaddr4, sizeof(isc_sockaddr_t));
 
        if (localaddr6 != NULL)
-               isc_mem_put(mctx, localaddr6, sizeof(isc_sockaddr_t));
+               isc_mem_put(gmctx, localaddr6, sizeof(isc_sockaddr_t));
 
        if (tsigkey != NULL) {
                ddebug("Freeing TSIG key");
@@ -832,19 +830,19 @@ setup_system(void) {
        if (!have_ipv4 && !have_ipv6)
                fatal("could not find either IPv4 or IPv6");
 
-       result = isc_log_create(mctx, &lctx, &logconfig);
+       result = isc_log_create(gmctx, &glctx, &logconfig);
        check_result(result, "isc_log_create");
 
-       isc_log_setcontext(lctx);
-       dns_log_init(lctx);
-       dns_log_setcontext(lctx);
+       isc_log_setcontext(glctx);
+       dns_log_init(glctx);
+       dns_log_setcontext(glctx);
 
        result = isc_log_usechannel(logconfig, "default_debug", NULL, NULL);
        check_result(result, "isc_log_usechannel");
 
-       isc_log_setdebuglevel(lctx, logdebuglevel);
+       isc_log_setdebuglevel(glctx, logdebuglevel);
 
-       lwresult = lwres_context_create(&lwctx, mctx, mem_alloc, mem_free, 1);
+       lwresult = lwres_context_create(&lwctx, gmctx, mem_alloc, mem_free, 1);
        if (lwresult != LWRES_R_SUCCESS)
                fatal("lwres_context_create failed");
 
@@ -854,7 +852,7 @@ setup_system(void) {
        if (servers != NULL) {
                if (master_servers == servers)
                        master_servers = NULL;
-               isc_mem_put(mctx, servers, ns_total * sizeof(isc_sockaddr_t));
+               isc_mem_put(gmctx, servers, ns_total * sizeof(isc_sockaddr_t));
        }
 
        ns_inuse = 0;
@@ -868,7 +866,7 @@ setup_system(void) {
                default_servers = !local_only;
 
                ns_total = (have_ipv4 ? 1 : 0) + (have_ipv6 ? 1 : 0);
-               servers = isc_mem_get(mctx, ns_total * sizeof(isc_sockaddr_t));
+               servers = isc_mem_get(gmctx, ns_total * sizeof(isc_sockaddr_t));
                if (servers == NULL)
                        fatal("out of memory");
 
@@ -884,7 +882,7 @@ setup_system(void) {
                }
        } else {
                ns_total = lwconf->nsnext;
-               servers = isc_mem_get(mctx, ns_total * sizeof(isc_sockaddr_t));
+               servers = isc_mem_get(gmctx, ns_total * sizeof(isc_sockaddr_t));
                if (servers == NULL)
                        fatal("out of memory");
                for (i = 0; i < ns_total; i++) {
@@ -905,22 +903,22 @@ setup_system(void) {
                }
        }
 
-       setup_entropy(mctx, NULL, &entropy);
+       setup_entropy(gmctx, NULL, &entropy);
 
-       result = isc_hash_create(mctx, entropy, DNS_NAME_MAXWIRE);
+       result = isc_hash_create(gmctx, entropy, DNS_NAME_MAXWIRE);
        check_result(result, "isc_hash_create");
        isc_hash_init();
 
-       result = dns_dispatchmgr_create(mctx, entropy, &dispatchmgr);
+       result = dns_dispatchmgr_create(gmctx, entropy, &dispatchmgr);
        check_result(result, "dns_dispatchmgr_create");
 
-       result = isc_socketmgr_create(mctx, &socketmgr);
+       result = isc_socketmgr_create(gmctx, &socketmgr);
        check_result(result, "dns_socketmgr_create");
 
-       result = isc_timermgr_create(mctx, &timermgr);
+       result = isc_timermgr_create(gmctx, &timermgr);
        check_result(result, "dns_timermgr_create");
 
-       result = isc_taskmgr_create(mctx, 1, 0, &taskmgr);
+       result = isc_taskmgr_create(gmctx, 1, 0, &taskmgr);
        check_result(result, "isc_taskmgr_create");
 
        result = isc_task_create(taskmgr, 0, &global_task);
@@ -929,7 +927,7 @@ setup_system(void) {
        result = isc_task_onshutdown(global_task, shutdown_program, NULL);
        check_result(result, "isc_task_onshutdown");
 
-       result = dst_lib_init(mctx, entropy, 0);
+       result = dst_lib_init(gmctx, entropy, 0);
        check_result(result, "dst_lib_init");
        is_dst_up = ISC_TRUE;
 
@@ -960,7 +958,7 @@ setup_system(void) {
                check_result(result, "dns_dispatch_getudp (v4)");
        }
 
-       result = dns_requestmgr_create(mctx, timermgr,
+       result = dns_requestmgr_create(gmctx, timermgr,
                                       socketmgr, taskmgr, dispatchmgr,
                                       dispatchv4, dispatchv6, &requestmgr);
        check_result(result, "dns_requestmgr_create");
@@ -968,12 +966,12 @@ setup_system(void) {
        if (keystr != NULL)
                setup_keystr();
        else if (local_only) {
-               result = read_sessionkey(mctx, lctx);
+               result = read_sessionkey(gmctx, glctx);
                if (result != ISC_R_SUCCESS)
                        fatal("can't read key from %s: %s\n",
                              keyfile, isc_result_totext(result));
        } else if (keyfile != NULL)
-               setup_keyfile(mctx, lctx);
+               setup_keyfile(gmctx, glctx);
 }
 
 static void
@@ -1211,7 +1209,7 @@ parse_name(char **cmdlinep, dns_message_t *msg, dns_name_t **namep) {
 
        result = dns_message_gettempname(msg, namep);
        check_result(result, "dns_message_gettempname");
-       result = isc_buffer_allocate(mctx, &namebuf, DNS_NAME_MAXWIRE);
+       result = isc_buffer_allocate(gmctx, &namebuf, DNS_NAME_MAXWIRE);
        check_result(result, "isc_buffer_allocate");
        dns_name_init(*namep, NULL);
        dns_name_setbuffer(*namep, namebuf);
@@ -1246,21 +1244,21 @@ parse_rdata(char **cmdlinep, dns_rdataclass_t rdataclass,
 
        if (*cmdline != 0) {
                dns_rdatacallbacks_init(&callbacks);
-               result = isc_lex_create(mctx, strlen(cmdline), &lex);
+               result = isc_lex_create(gmctx, strlen(cmdline), &lex);
                check_result(result, "isc_lex_create");
                isc_buffer_init(&source, cmdline, strlen(cmdline));
                isc_buffer_add(&source, strlen(cmdline));
                result = isc_lex_openbuffer(lex, &source);
                check_result(result, "isc_lex_openbuffer");
-               result = isc_buffer_allocate(mctx, &buf, MAXWIRE);
+               result = isc_buffer_allocate(gmctx, &buf, MAXWIRE);
                check_result(result, "isc_buffer_allocate");
                result = dns_rdata_fromtext(NULL, rdataclass, rdatatype, lex,
-                                           dns_rootname, 0, mctx, buf,
+                                           dns_rootname, 0, gmctx, buf,
                                            &callbacks);
                isc_lex_destroy(&lex);
                if (result == ISC_R_SUCCESS) {
                        isc_buffer_usedregion(buf, &r);
-                       result = isc_buffer_allocate(mctx, &newbuf, r.length);
+                       result = isc_buffer_allocate(gmctx, &newbuf, r.length);
                        check_result(result, "isc_buffer_allocate");
                        isc_buffer_putmem(newbuf, r.base, r.length);
                        isc_buffer_usedregion(newbuf, &r);
@@ -1456,14 +1454,14 @@ evaluate_server(char *cmdline) {
        if (servers != NULL) {
                if (master_servers == servers)
                        master_servers = NULL;
-               isc_mem_put(mctx, servers, ns_total * sizeof(isc_sockaddr_t));
+               isc_mem_put(gmctx, servers, ns_total * sizeof(isc_sockaddr_t));
        }
 
        default_servers = ISC_FALSE;
 
        ns_total = MAX_SERVERADDRS;
        ns_inuse = 0;
-       servers = isc_mem_get(mctx, ns_total * sizeof(isc_sockaddr_t));
+       servers = isc_mem_get(gmctx, ns_total * sizeof(isc_sockaddr_t));
        if (servers == NULL)
                fatal("out of memory");
 
@@ -1505,13 +1503,13 @@ evaluate_local(char *cmdline) {
 
        if (have_ipv6 && inet_pton(AF_INET6, local, &in6) == 1) {
                if (localaddr6 == NULL)
-                       localaddr6 = isc_mem_get(mctx, sizeof(isc_sockaddr_t));
+                       localaddr6 = isc_mem_get(gmctx, sizeof(isc_sockaddr_t));
                if (localaddr6 == NULL)
                        fatal("out of memory");
                isc_sockaddr_fromin6(localaddr6, &in6, (in_port_t)port);
        } else if (have_ipv4 && inet_pton(AF_INET, local, &in4) == 1) {
                if (localaddr4 == NULL)
-                       localaddr4 = isc_mem_get(mctx, sizeof(isc_sockaddr_t));
+                       localaddr4 = isc_mem_get(gmctx, sizeof(isc_sockaddr_t));
                if (localaddr4 == NULL)
                        fatal("out of memory");
                isc_sockaddr_fromin(localaddr4, &in4, (in_port_t)port);
@@ -1530,7 +1528,7 @@ evaluate_key(char *cmdline) {
        isc_buffer_t b;
        isc_result_t result;
        dns_fixedname_t fkeyname;
-       dns_name_t *keyname;
+       dns_name_t *mykeyname;
        int secretlen;
        unsigned char *secret = NULL;
        isc_buffer_t secretbuf;
@@ -1545,7 +1543,7 @@ evaluate_key(char *cmdline) {
        }
 
        dns_fixedname_init(&fkeyname);
-       keyname = dns_fixedname_name(&fkeyname);
+       mykeyname = dns_fixedname_name(&fkeyname);
 
        n = strchr(namestr, ':');
        if (n != NULL) {
@@ -1556,7 +1554,7 @@ evaluate_key(char *cmdline) {
 
        isc_buffer_init(&b, namestr, strlen(namestr));
        isc_buffer_add(&b, strlen(namestr));
-       result = dns_name_fromtext(keyname, &b, dns_rootname, 0, NULL);
+       result = dns_name_fromtext(mykeyname, &b, dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS) {
                fprintf(stderr, "could not parse key name\n");
                return (STATUS_SYNTAX);
@@ -1568,7 +1566,7 @@ evaluate_key(char *cmdline) {
                return (STATUS_SYNTAX);
        }
        secretlen = strlen(secretstr) * 3 / 4;
-       secret = isc_mem_allocate(mctx, secretlen);
+       secret = isc_mem_allocate(gmctx, secretlen);
        if (secret == NULL)
                fatal("out of memory");
 
@@ -1577,17 +1575,17 @@ evaluate_key(char *cmdline) {
        if (result != ISC_R_SUCCESS) {
                fprintf(stderr, "could not create key from %s: %s\n",
                        secretstr, isc_result_totext(result));
-               isc_mem_free(mctx, secret);
+               isc_mem_free(gmctx, secret);
                return (STATUS_SYNTAX);
        }
        secretlen = isc_buffer_usedlength(&secretbuf);
 
        if (tsigkey != NULL)
                dns_tsigkey_detach(&tsigkey);
-       result = dns_tsigkey_create(keyname, hmacname, secret, secretlen,
-                                   ISC_FALSE, NULL, 0, 0, mctx, NULL,
+       result = dns_tsigkey_create(mykeyname, hmacname, secret, secretlen,
+                                   ISC_FALSE, NULL, 0, 0, gmctx, NULL,
                                    &tsigkey);
-       isc_mem_free(mctx, secret);
+       isc_mem_free(gmctx, secret);
        if (result != ISC_R_SUCCESS) {
                fprintf(stderr, "could not create key from %s %s: %s\n",
                        namestr, secretstr, dns_result_totext(result));
@@ -1631,7 +1629,7 @@ evaluate_realm(char *cmdline) {
        int n;
 
        if (realm != NULL) {
-               isc_mem_free(mctx, realm);
+               isc_mem_free(gmctx, realm);
                realm = NULL;
        }
 
@@ -1642,7 +1640,7 @@ evaluate_realm(char *cmdline) {
        n = snprintf(buf, sizeof(buf), "@%s", word);
        if (n < 0 || (size_t)n >= sizeof(buf))
                fatal("realm is too long");
-       realm = isc_mem_strdup(mctx, buf);
+       realm = isc_mem_strdup(gmctx, buf);
        if (realm == NULL)
                fatal("out of memory");
        return (STATUS_MORE);
@@ -1967,7 +1965,7 @@ show_message(FILE *stream, dns_message_t *msg, const char *description) {
                }
                if (buf != NULL)
                        isc_buffer_free(&buf);
-               result = isc_buffer_allocate(mctx, &buf, bufsz);
+               result = isc_buffer_allocate(gmctx, &buf, bufsz);
                check_result(result, "isc_buffer_allocate");
                result = dns_message_totext(msg, style, 0, buf);
                bufsz *= 2;
@@ -2227,12 +2225,12 @@ update_completed(isc_task_t *task, isc_event_t *event) {
                dns_request_destroy(&request);
                dns_message_renderreset(updatemsg);
                dns_message_settsigkey(updatemsg, NULL);
-               send_update(zonename, &master_servers[master_inuse]);
+               send_update(zname, &master_servers[master_inuse]);
                isc_event_free(&event);
                return;
        }
 
-       result = dns_message_create(mctx, DNS_MESSAGE_INTENTPARSE, &answer);
+       result = dns_message_create(gmctx, DNS_MESSAGE_INTENTPARSE, &answer);
        check_result(result, "dns_message_create");
        result = dns_request_getresponse(request, answer,
                                         DNS_MESSAGEPARSE_PRESERVEORDER);
@@ -2286,15 +2284,15 @@ update_completed(isc_task_t *task, isc_event_t *event) {
  done:
        dns_request_destroy(&request);
        if (usegsstsig) {
-               dns_name_free(&tmpzonename, mctx);
-               dns_name_free(&restart_master, mctx);
+               dns_name_free(&tmpzonename, gmctx);
+               dns_name_free(&restart_master, gmctx);
        }
        isc_event_free(&event);
        done_update();
 }
 
 static void
-send_update(dns_name_t *zonename, isc_sockaddr_t *master) {
+send_update(dns_name_t *zone, isc_sockaddr_t *master) {
        isc_result_t result;
        dns_request_t *request = NULL;
        unsigned int options = DNS_REQUESTOPT_CASE;
@@ -2302,7 +2300,7 @@ send_update(dns_name_t *zonename, isc_sockaddr_t *master) {
 
        ddebug("send_update()");
 
-       setzone(zonename);
+       setzone(zone);
 
        if (usevc)
                options |= DNS_REQUESTOPT_TCP;
@@ -2389,7 +2387,7 @@ recvsoa(isc_task_t *task, isc_event_t *event) {
        if (shuttingdown) {
                dns_request_destroy(&request);
                dns_message_destroy(&soaquery);
-               isc_mem_put(mctx, reqinfo, sizeof(nsu_requestinfo_t));
+               isc_mem_put(gmctx, reqinfo, sizeof(nsu_requestinfo_t));
                isc_event_free(&event);
                maybeshutdown();
                return;
@@ -2402,19 +2400,19 @@ recvsoa(isc_task_t *task, isc_event_t *event) {
                dns_message_renderreset(soaquery);
                dns_message_settsigkey(soaquery, NULL);
                sendrequest(&servers[ns_inuse], soaquery, &request);
-               isc_mem_put(mctx, reqinfo, sizeof(nsu_requestinfo_t));
+               isc_mem_put(gmctx, reqinfo, sizeof(nsu_requestinfo_t));
                isc_event_free(&event);
                setzoneclass(dns_rdataclass_none);
                return;
        }
 
-       isc_mem_put(mctx, reqinfo, sizeof(nsu_requestinfo_t));
+       isc_mem_put(gmctx, reqinfo, sizeof(nsu_requestinfo_t));
        reqinfo = NULL;
        isc_event_free(&event);
        reqev = NULL;
 
        ddebug("About to create rcvmsg");
-       result = dns_message_create(mctx, DNS_MESSAGE_INTENTPARSE, &rcvmsg);
+       result = dns_message_create(gmctx, DNS_MESSAGE_INTENTPARSE, &rcvmsg);
        check_result(result, "dns_message_create");
        result = dns_request_getresponse(request, rcvmsg,
                                         DNS_MESSAGEPARSE_PRESERVEORDER);
@@ -2422,7 +2420,7 @@ recvsoa(isc_task_t *task, isc_event_t *event) {
                dns_message_destroy(&rcvmsg);
                ddebug("Destroying request [%p]", request);
                dns_request_destroy(&request);
-               reqinfo = isc_mem_get(mctx, sizeof(nsu_requestinfo_t));
+               reqinfo = isc_mem_get(gmctx, sizeof(nsu_requestinfo_t));
                if (reqinfo == NULL)
                        fatal("out of memory");
                reqinfo->msg = soaquery;
@@ -2530,9 +2528,9 @@ recvsoa(isc_task_t *task, isc_event_t *event) {
        dns_name_clone(&soa.origin, &master);
 
        if (userzone != NULL)
-               zonename = userzone;
+               zname = userzone;
        else
-               zonename = name;
+               zname = name;
 
        if (debugging) {
                char namestr[DNS_NAME_FORMATSIZE];
@@ -2551,11 +2549,11 @@ recvsoa(isc_task_t *task, isc_event_t *event) {
                serverstr[isc_buffer_usedlength(&buf)] = 0;
 
                if (master_servers != NULL && master_servers != servers)
-                       isc_mem_put(mctx, master_servers,
+                       isc_mem_put(gmctx, master_servers,
                                    master_total * sizeof(isc_sockaddr_t));
                master_total = MAX_SERVERADDRS;
                size = master_total * sizeof(isc_sockaddr_t);
-               master_servers = isc_mem_get(mctx, size);
+               master_servers = isc_mem_get(gmctx, size);
                if (master_servers == NULL)
                        fatal("out of memory");
 
@@ -2569,16 +2567,16 @@ recvsoa(isc_task_t *task, isc_event_t *event) {
 #ifdef GSSAPI
        if (usegsstsig) {
                dns_name_init(&tmpzonename, NULL);
-               dns_name_dup(zonename, mctx, &tmpzonename);
+               dns_name_dup(zname, gmctx, &tmpzonename);
                dns_name_init(&restart_master, NULL);
-               dns_name_dup(&master, mctx, &restart_master);
+               dns_name_dup(&master, gmctx, &restart_master);
                start_gssrequest(&master);
        } else {
-               send_update(zonename, &master_servers[master_inuse]);
+               send_update(zname, &master_servers[master_inuse]);
                setzoneclass(dns_rdataclass_none);
        }
 #else
-       send_update(zonename, &master_servers[master_inuse]);
+       send_update(zname, &master_servers[master_inuse]);
        setzoneclass(dns_rdataclass_none);
 #endif
 
@@ -2616,7 +2614,7 @@ sendrequest(isc_sockaddr_t *destaddr, dns_message_t *msg,
        nsu_requestinfo_t *reqinfo;
        isc_sockaddr_t *srcaddr;
 
-       reqinfo = isc_mem_get(mctx, sizeof(nsu_requestinfo_t));
+       reqinfo = isc_mem_get(gmctx, sizeof(nsu_requestinfo_t));
        if (reqinfo == NULL)
                fatal("out of memory");
        reqinfo->msg = msg;
@@ -2641,8 +2639,7 @@ sendrequest(isc_sockaddr_t *destaddr, dns_message_t *msg,
  * Get the realm from the users kerberos ticket if possible
  */
 static void
-get_ticket_realm(isc_mem_t *mctx)
-{
+get_ticket_realm(isc_mem_t *mctx) {
        krb5_context ctx;
        krb5_error_code rc;
        krb5_ccache ccache;
@@ -2699,7 +2696,7 @@ start_gssrequest(dns_name_t *master) {
        dns_name_t *servname;
        dns_fixedname_t fname;
        char namestr[DNS_NAME_FORMATSIZE];
-       char keystr[DNS_NAME_FORMATSIZE];
+       char mykeystr[DNS_NAME_FORMATSIZE];
        char *err_message = NULL;
 
        debug("start_gssrequest");
@@ -2708,7 +2705,7 @@ start_gssrequest(dns_name_t *master) {
        if (gssring != NULL)
                dns_tsigkeyring_detach(&gssring);
        gssring = NULL;
-       result = dns_tsigkeyring_create(mctx, &gssring);
+       result = dns_tsigkeyring_create(gmctx, &gssring);
 
        if (result != ISC_R_SUCCESS)
                fatal("dns_tsigkeyring_create failed: %s",
@@ -2716,7 +2713,7 @@ start_gssrequest(dns_name_t *master) {
 
        dns_name_format(master, namestr, sizeof(namestr));
        if (kserver == NULL) {
-               kserver = isc_mem_get(mctx, sizeof(isc_sockaddr_t));
+               kserver = isc_mem_get(gmctx, sizeof(isc_sockaddr_t));
                if (kserver == NULL)
                        fatal("out of memory");
        }
@@ -2729,7 +2726,7 @@ start_gssrequest(dns_name_t *master) {
        servname = dns_fixedname_name(&fname);
 
        if (realm == NULL)
-               get_ticket_realm(mctx);
+               get_ticket_realm(gmctx);
 
        result = isc_string_printf(servicename, sizeof(servicename),
                                   "DNS/%s%s", namestr, realm ? realm : "");
@@ -2747,13 +2744,13 @@ start_gssrequest(dns_name_t *master) {
        keyname = dns_fixedname_name(&fkname);
 
        isc_random_get(&val);
-       result = isc_string_printf(keystr, sizeof(keystr), "%u.sig-%s",
+       result = isc_string_printf(mykeystr, sizeof(mykeystr), "%u.sig-%s",
                                   val, namestr);
        if (result != ISC_R_SUCCESS)
-               fatal("isc_string_printf(keystr) failed: %s",
+               fatal("isc_string_printf(mykeystr) failed: %s",
                      isc_result_totext(result));
-       isc_buffer_init(&buf, keystr, strlen(keystr));
-       isc_buffer_add(&buf, strlen(keystr));
+       isc_buffer_init(&buf, mykeystr, strlen(mykeystr));
+       isc_buffer_add(&buf, strlen(mykeystr));
 
        result = dns_name_fromtext(keyname, &buf, dns_rootname, 0, NULL);
        if (result != ISC_R_SUCCESS)
@@ -2764,7 +2761,7 @@ start_gssrequest(dns_name_t *master) {
        keyname->attributes |= DNS_NAMEATTR_NOCOMPRESS;
 
        rmsg = NULL;
-       result = dns_message_create(mctx, DNS_MESSAGE_INTENTRENDER, &rmsg);
+       result = dns_message_create(gmctx, DNS_MESSAGE_INTENTRENDER, &rmsg);
        if (result != ISC_R_SUCCESS)
                fatal("dns_message_create failed: %s",
                      isc_result_totext(result));
@@ -2773,7 +2770,7 @@ start_gssrequest(dns_name_t *master) {
        context = GSS_C_NO_CONTEXT;
        result = dns_tkey_buildgssquery(rmsg, keyname, servname, NULL, 0,
                                        &context, use_win2k_gsstsig,
-                                       mctx, &err_message);
+                                       gmctx, &err_message);
        if (result == ISC_R_FAILURE)
                fatal("tkey query failed: %s",
                      err_message != NULL ? err_message : "unknown error");
@@ -2794,7 +2791,7 @@ send_gssrequest(isc_sockaddr_t *destaddr, dns_message_t *msg,
        isc_sockaddr_t *srcaddr;
 
        debug("send_gssrequest");
-       reqinfo = isc_mem_get(mctx, sizeof(nsu_gssinfo_t));
+       reqinfo = isc_mem_get(gmctx, sizeof(nsu_gssinfo_t));
        if (reqinfo == NULL)
                fatal("out of memory");
        reqinfo->msg = msg;
@@ -2851,7 +2848,7 @@ recvgss(isc_task_t *task, isc_event_t *event) {
        if (shuttingdown) {
                dns_request_destroy(&request);
                dns_message_destroy(&tsigquery);
-               isc_mem_put(mctx, reqinfo, sizeof(nsu_gssinfo_t));
+               isc_mem_put(gmctx, reqinfo, sizeof(nsu_gssinfo_t));
                isc_event_free(&event);
                maybeshutdown();
                return;
@@ -2863,17 +2860,17 @@ recvgss(isc_task_t *task, isc_event_t *event) {
                dns_request_destroy(&request);
                dns_message_renderreset(tsigquery);
                sendrequest(&servers[ns_inuse], tsigquery, &request);
-               isc_mem_put(mctx, reqinfo, sizeof(nsu_gssinfo_t));
+               isc_mem_put(gmctx, reqinfo, sizeof(nsu_gssinfo_t));
                isc_event_free(&event);
                return;
        }
-       isc_mem_put(mctx, reqinfo, sizeof(nsu_gssinfo_t));
+       isc_mem_put(gmctx, reqinfo, sizeof(nsu_gssinfo_t));
 
        isc_event_free(&event);
        reqev = NULL;
 
        ddebug("recvgss creating rcvmsg");
-       result = dns_message_create(mctx, DNS_MESSAGE_INTENTPARSE, &rcvmsg);
+       result = dns_message_create(gmctx, DNS_MESSAGE_INTENTPARSE, &rcvmsg);
        check_result(result, "dns_message_create");
 
        result = dns_request_getresponse(request, rcvmsg,
@@ -2995,7 +2992,7 @@ start_update(void) {
                return;
        }
 
-       result = dns_message_create(mctx, DNS_MESSAGE_INTENTRENDER,
+       result = dns_message_create(gmctx, DNS_MESSAGE_INTENTRENDER,
                                    &soaquery);
        check_result(result, "dns_message_create");
 
@@ -3072,11 +3069,11 @@ cleanup(void) {
                dns_tsigkeyring_detach(&gssring);
        }
        if (kserver != NULL) {
-               isc_mem_put(mctx, kserver, sizeof(isc_sockaddr_t));
+               isc_mem_put(gmctx, kserver, sizeof(isc_sockaddr_t));
                kserver = NULL;
        }
        if (realm != NULL) {
-               isc_mem_free(mctx, realm);
+               isc_mem_free(gmctx, realm);
                realm = NULL;
        }
 #endif
@@ -3103,12 +3100,12 @@ cleanup(void) {
        dns_name_destroy();
 
        ddebug("Removing log context");
-       isc_log_destroy(&lctx);
+       isc_log_destroy(&glctx);
 
        ddebug("Destroying memory context");
        if (memdebugging)
-               isc_mem_stats(mctx, stderr);
-       isc_mem_destroy(&mctx);
+               isc_mem_stats(gmctx, stderr);
+       isc_mem_destroy(&gmctx);
 }
 
 static void
@@ -3148,14 +3145,14 @@ main(int argc, char **argv) {
 
        pre_parse_args(argc, argv);
 
-       result = isc_mem_create(0, 0, &mctx);
+       result = isc_mem_create(0, 0, &gmctx);
        check_result(result, "isc_mem_create");
 
-       parse_args(argc, argv, mctx, &entropy);
+       parse_args(argc, argv, gmctx, &entropy);
 
        setup_system();
 
-       result = isc_app_onrun(mctx, global_task, getinput, NULL);
+       result = isc_app_onrun(gmctx, global_task, getinput, NULL);
        check_result(result, "isc_app_onrun");
 
        (void)isc_app_run();
index ba4543193e02f5b01c1289df6800e351d8addcac..c707ce4f3ab092bcad18336027f4d476c3d1dfe2 100644 (file)
@@ -79,7 +79,7 @@ static isc_uint32_t algorithm;
 static isccc_region_t secret;
 static isc_boolean_t failed = ISC_FALSE;
 static isc_boolean_t c_flag = ISC_FALSE;
-static isc_mem_t *mctx;
+static isc_mem_t *rndc_mctx;
 static int sends, recvs, connects;
 static char *command;
 static char *args;
@@ -415,7 +415,7 @@ rndc_connected(isc_task_t *task, isc_event_t *event) {
        r.length = len;
        r.base = databuf;
 
-       isccc_ccmsg_init(mctx, sock, &ccmsg);
+       isccc_ccmsg_init(rndc_mctx, sock, &ccmsg);
        isccc_ccmsg_setmaxsize(&ccmsg, 1024 * 1024);
 
        DO("schedule recv", isccc_ccmsg_readmessage(&ccmsg, task,
@@ -838,12 +838,12 @@ main(int argc, char **argv) {
 
        isc_random_get(&serial);
 
-       DO("create memory context", isc_mem_create(0, 0, &mctx));
-       DO("create socket manager", isc_socketmgr_create(mctx, &socketmgr));
-       DO("create task manager", isc_taskmgr_create(mctx, 1, 0, &taskmgr));
+       DO("create memory context", isc_mem_create(0, 0, &rndc_mctx));
+       DO("create socket manager", isc_socketmgr_create(rndc_mctx, &socketmgr));
+       DO("create task manager", isc_taskmgr_create(rndc_mctx, 1, 0, &taskmgr));
        DO("create task", isc_task_create(taskmgr, 0, &task));
 
-       DO("create logging context", isc_log_create(mctx, &log, &logconfig));
+       DO("create logging context", isc_log_create(rndc_mctx, &log, &logconfig));
        isc_log_setcontext(log);
        DO("setting log tag", isc_log_settag(logconfig, progname));
        logdest.file.stream = stderr;
@@ -857,7 +857,7 @@ main(int argc, char **argv) {
        DO("enabling log channel", isc_log_usechannel(logconfig, "stderr",
                                                      NULL, NULL));
 
-       parse_config(mctx, log, keyname, &pctx, &config);
+       parse_config(rndc_mctx, log, keyname, &pctx, &config);
 
        isccc_result_register();
 
@@ -872,7 +872,7 @@ main(int argc, char **argv) {
        for (i = 0; i < argc; i++)
                argslen += strlen(argv[i]) + 1;
 
-       args = isc_mem_get(mctx, argslen);
+       args = isc_mem_get(rndc_mctx, argslen);
        if (args == NULL)
                DO("isc_mem_get", ISC_R_NOMEMORY);
 
@@ -896,7 +896,7 @@ main(int argc, char **argv) {
        if (nserveraddrs == 0)
                get_addresses(servername, (in_port_t) remoteport);
 
-       DO("post event", isc_app_onrun(mctx, task, rndc_start, NULL));
+       DO("post event", isc_app_onrun(rndc_mctx, task, rndc_start, NULL));
 
        result = isc_app_run();
        if (result != ISC_R_SUCCESS)
@@ -914,15 +914,15 @@ main(int argc, char **argv) {
        cfg_obj_destroy(pctx, &config);
        cfg_parser_destroy(&pctx);
 
-       isc_mem_put(mctx, args, argslen);
+       isc_mem_put(rndc_mctx, args, argslen);
        isccc_ccmsg_invalidate(&ccmsg);
 
        dns_name_destroy();
 
        if (show_final_mem)
-               isc_mem_stats(mctx, stderr);
+               isc_mem_stats(rndc_mctx, stderr);
 
-       isc_mem_destroy(&mctx);
+       isc_mem_destroy(&rndc_mctx);
 
        if (failed)
                return (1);
index 6338c4a9c2b2a85be52235a5a87b72172240da41..a82593c4488d1f38b74f18c3c31a488ae2a2c8a7 100755 (executable)
--- a/configure
+++ b/configure
@@ -976,6 +976,7 @@ with_gnu_ld
 with_sysroot
 enable_libtool_lock
 enable_libbind
+enable_warn_shadow
 enable_warn_error
 enable_developer
 enable_seccomp
@@ -1668,6 +1669,7 @@ Optional Features:
                           optimize for fast installation [default=yes]
   --disable-libtool-lock  avoid locking (might break parallel builds)
   --enable-libbind       deprecated
+  --enable-warn-shadow   turn on -Wshadow when compiling
   --enable-warn-error    turn on -Werror when compiling
   --enable-developer      enable developer build settings
   --enable-seccomp        enable support for libseccomp system call filtering
@@ -11411,6 +11413,12 @@ It is available from http://www.isc.org as a separate download." "$LINENO" 5
                ;;
 esac
 
+# Check whether --enable-warn_shadow was given.
+if test "${enable_warn_shadow+set}" = set; then :
+  enableval=$enable_warn_shadow;
+fi
+
+
 # Check whether --enable-warn_error was given.
 if test "${enable_warn_error+set}" = set; then :
   enableval=$enable_warn_error;
@@ -11432,6 +11440,7 @@ yes)
        test "${enable_symtable+set}" = set || enable_symtable=all
        test "${enable_sit+set}" = set || enable_sit=yes
        test "${enable_warn_error+set}" = set || enable_warn_error=yes
+       test "${enable_warn_shadow+set}" = set || enable_warn_shadow=yes
        ;;
 esac
 
@@ -16884,6 +16893,9 @@ $as_echo "no" >&6; }
                BACKTRACECFLAGS="$BACKTRACECFLAGS -Wl,+vnocompatwarnings"
                ;;
        esac
+       if test "X$enable_warn_shadow" = Xyes; then
+               STD_CWARNINGS="$STD_CWARNINGS -Wshadow"
+       fi
        if test "X$enable_warn_error" = Xyes; then
                STD_CWARNINGS="$STD_CWARNINGS -Werror"
        fi
index 2b0c3c1f4128d8b7f8d143f5043a8d11e6967ae4..d566e1c0e841a9d5b0c1a88fad5f9816c65fc79e 100644 (file)
@@ -77,6 +77,8 @@ It is available from http://www.isc.org as a separate download.])
                ;;
 esac
 
+AC_ARG_ENABLE(warn_shadow, [  --enable-warn-shadow       turn on -Wshadow when compiling])
+
 AC_ARG_ENABLE(warn_error, [  --enable-warn-error         turn on -Werror when compiling])
 
 AC_ARG_ENABLE(developer, [  --enable-developer      enable developer build settings])
@@ -90,6 +92,7 @@ yes)
        test "${enable_symtable+set}" = set || enable_symtable=all
        test "${enable_sit+set}" = set || enable_sit=yes
        test "${enable_warn_error+set}" = set || enable_warn_error=yes
+       test "${enable_warn_shadow+set}" = set || enable_warn_shadow=yes
        ;;
 esac
 
@@ -2286,6 +2289,9 @@ if test "X$GCC" = "Xyes"; then
                BACKTRACECFLAGS="$BACKTRACECFLAGS -Wl,+vnocompatwarnings"
                ;;
        esac
+       if test "X$enable_warn_shadow" = Xyes; then
+               STD_CWARNINGS="$STD_CWARNINGS -Wshadow"
+       fi
        if test "X$enable_warn_error" = Xyes; then
                STD_CWARNINGS="$STD_CWARNINGS -Werror"
        fi
index dfd57ede68bf05576de3e1a37101b0ea76555018..618d5bb1ddff99d17d96f00d4f92e6afffb9fe3f 100644 (file)
@@ -102,7 +102,6 @@ check_orderent(const cfg_obj_t *ent, isc_log_t *logctx) {
 
        obj = cfg_tuple_get(ent, "type");
        if (cfg_obj_isstring(obj)) {
-
                DE_CONST(cfg_obj_asstring(obj), r.base);
                r.length = strlen(r.base);
                tresult = dns_rdatatype_fromtext(&rdtype, &r);
@@ -297,7 +296,6 @@ disabled_algorithms(const cfg_obj_t *disabled, isc_log_t *logctx) {
        {
                isc_textregion_t r;
                dns_secalg_t alg;
-               isc_result_t tresult;
 
                DE_CONST(cfg_obj_asstring(cfg_listelt_value(element)), r.base);
                r.length = strlen(r.base);
@@ -345,7 +343,6 @@ disabled_ds_digests(const cfg_obj_t *disabled, isc_log_t *logctx) {
        {
                isc_textregion_t r;
                dns_dsdigest_t digest;
-               isc_result_t tresult;
 
                DE_CONST(cfg_obj_asstring(cfg_listelt_value(element)), r.base);
                r.length = strlen(r.base);
@@ -786,6 +783,14 @@ check_dscp(const cfg_obj_t *options, isc_log_t *logctx) {
        return (result);
 }
 
+static isc_result_t
+check_name(const char *str) {
+       dns_fixedname_t fixed;
+
+       dns_fixedname_init(&fixed);
+       return (dns_name_fromstring(dns_fixedname_name(&fixed), str, 0, NULL));
+}
+
 static isc_result_t
 check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
              optlevel_t optlevel)
@@ -897,7 +902,6 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
        obj = NULL;
        (void)cfg_map_get(options, "preferred-glue", &obj);
        if (obj != NULL) {
-               const char *str;
                str = cfg_obj_asstring(obj);
                if (strcasecmp(str, "a") != 0 &&
                    strcasecmp(str, "aaaa") != 0 &&
@@ -911,25 +915,14 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
        (void)cfg_map_get(options, "root-delegation-only", &obj);
        if (obj != NULL) {
                if (!cfg_obj_isvoid(obj)) {
-                       const cfg_listelt_t *element;
-                       const cfg_obj_t *exclude;
-                       const char *str;
-                       dns_fixedname_t fixed;
-                       dns_name_t *name;
-                       isc_buffer_t b;
-
-                       dns_fixedname_init(&fixed);
-                       name = dns_fixedname_name(&fixed);
                        for (element = cfg_list_first(obj);
                             element != NULL;
                             element = cfg_list_next(element)) {
+                               const cfg_obj_t *exclude;
+
                                exclude = cfg_listelt_value(element);
                                str = cfg_obj_asstring(exclude);
-                               isc_buffer_constinit(&b, str, strlen(str));
-                               isc_buffer_add(&b, strlen(str));
-                               tresult = dns_name_fromtext(name, &b,
-                                                          dns_rootname,
-                                                          0, NULL);
+                               tresult = check_name(str);
                                if (tresult != ISC_R_SUCCESS) {
                                        cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
                                                    "bad domain name '%s'",
@@ -1010,10 +1003,7 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
                                        continue;
                        }
 
-                       isc_buffer_constinit(&b, dlv, strlen(dlv));
-                       isc_buffer_add(&b, strlen(dlv));
-                       tresult = dns_name_fromtext(name, &b, dns_rootname,
-                                                   0, NULL);
+                       tresult = dns_name_fromstring(name, dlv, 0, NULL);
                        if (tresult != ISC_R_SUCCESS) {
                                cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
                                            "bad domain name '%s'", dlv);
@@ -1030,6 +1020,7 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
                                    result == ISC_R_SUCCESS)
                                        result = tresult;
                        }
+
                        /*
                         * XXXMPA to be removed when multiple lookaside
                         * namespaces are supported.
@@ -1044,12 +1035,7 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
 
                        if (!cfg_obj_isvoid(anchor)) {
                                dlv = cfg_obj_asstring(anchor);
-                               isc_buffer_constinit(&b, dlv, strlen(dlv));
-                               isc_buffer_add(&b, strlen(dlv));
-                               tresult = dns_name_fromtext(name, &b,
-                                                           dns_rootname,
-                                                           DNS_NAME_DOWNCASE,
-                                                           NULL);
+                               tresult = check_name(dlv);
                                if (tresult != ISC_R_SUCCESS) {
                                        cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
                                                    "bad domain name '%s'",
@@ -1092,7 +1078,6 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
        obj = NULL;
        (void)cfg_map_get(options, "dnssec-must-be-secure", &obj);
        if (obj != NULL) {
-               isc_symtab_t *symtab = NULL;
                tresult = isc_symtab_create(mctx, 100, freekey, mctx,
                                            ISC_FALSE, &symtab);
                if (tresult != ISC_R_SUCCESS)
@@ -1118,11 +1103,7 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
                (void)cfg_map_get(options, server_contact[i], &obj);
                if (obj != NULL) {
                        str = cfg_obj_asstring(obj);
-                       isc_buffer_constinit(&b, str, strlen(str));
-                       isc_buffer_add(&b, strlen(str));
-                       tresult = dns_name_fromtext(dns_fixedname_name(&fixed),
-                                                   &b, dns_rootname, 0, NULL);
-                       if (tresult != ISC_R_SUCCESS) {
+                       if (check_name(str) != ISC_R_SUCCESS) {
                                cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
                                            "%s: invalid name '%s'",
                                            server_contact[i], str);
@@ -1142,11 +1123,7 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
        {
                obj = cfg_listelt_value(element);
                str = cfg_obj_asstring(obj);
-               isc_buffer_constinit(&b, str, strlen(str));
-               isc_buffer_add(&b, strlen(str));
-               tresult = dns_name_fromtext(dns_fixedname_name(&fixed), &b,
-                                           dns_rootname, 0, NULL);
-               if (tresult != ISC_R_SUCCESS) {
+               if (check_name(str) != ISC_R_SUCCESS) {
                        cfg_obj_log(obj, logctx, ISC_LOG_ERROR,
                                    "disable-empty-zone: invalid name '%s'",
                                    str);
@@ -1175,7 +1152,6 @@ check_options(const cfg_obj_t *options, isc_log_t *logctx, isc_mem_t *mctx,
        obj = NULL;
        (void) cfg_map_get(options, "sit-secret", &obj);
        if (obj != NULL) {
-               isc_buffer_t b;
                unsigned char secret[32];
 
                memset(secret, 0, sizeof(secret));
@@ -2213,7 +2189,6 @@ bind9_check_key(const cfg_obj_t *key, isc_log_t *logctx) {
        }
        if (algorithm[len] == '-') {
                isc_uint16_t digestbits;
-               isc_result_t result;
                result = isc_parse_uint16(&digestbits, algorithm + len + 1, 10);
                if (result == ISC_R_SUCCESS || result == ISC_R_RANGE) {
                        if (result == ISC_R_RANGE ||
@@ -2637,7 +2612,6 @@ check_viewconf(const cfg_obj_t *config, const cfg_obj_t *voptions,
             element != NULL;
             element = cfg_list_next(element))
        {
-               isc_result_t tresult;
                const cfg_obj_t *zone = cfg_listelt_value(element);
 
                tresult = check_zoneconf(zone, voptions, config, symtab,
@@ -3169,10 +3143,10 @@ bind9_check_namedconf(const cfg_obj_t *config, isc_log_t *logctx,
                const cfg_obj_t *voptions = cfg_tuple_get(view, "options");
                const cfg_obj_t *vclassobj = cfg_tuple_get(view, "class");
                dns_rdataclass_t vclass = dns_rdataclass_in;
-               isc_result_t tresult = ISC_R_SUCCESS;
                const char *key = cfg_obj_asstring(vname);
                isc_symvalue_t symvalue;
 
+               tresult = ISC_R_SUCCESS;
                if (cfg_obj_isstring(vclassobj)) {
                        isc_textregion_t r;
 
index c4f8decb572f75315d647cee598499ae9f882130..bebfccc1cef6f2961da8c41beb6788d9c75fc7d6 100644 (file)
@@ -103,18 +103,18 @@ bind9_getaddresses(const char *hostname, in_port_t port,
 
                        if (d != NULL) {
 #ifdef ISC_PLATFORM_HAVESCOPEID
-                               isc_result_t result;
+                               isc_result_t iresult;
 
-                               result = isc_netscope_pton(AF_INET6, d + 1,
-                                                          &in6, &zone);
+                               iresult = isc_netscope_pton(AF_INET6, d + 1,
+                                                           &in6, &zone);
 
-                               if (result != ISC_R_SUCCESS)
-                                       return (result);
+                               if (iresult != ISC_R_SUCCESS)
+                                       return (iresult);
 #else
                                /*
                                 * The extended format is specified while the
                                 * system does not provide the ability to use
-                                * it.  Throw an explicit error instead of
+                                * it.  Throw an explicit error instead of
                                 * ignoring the specified value.
                                 */
                                return (ISC_R_BADADDRESSFORM);
index 6a1a03d9d1a8cb28789a48381d655bd129f535b2..366af5bce38f830c4a16532ece10a46024ecbb09 100644 (file)
@@ -388,9 +388,6 @@ diff_apply(dns_diff_t *diff, dns_db_t *db, dns_dbversion_t *ver,
                                 * Issue a warning and continue.
                                 */
                                if (warn) {
-                                       char classbuf[DNS_RDATATYPE_FORMATSIZE];
-                                       char namebuf[DNS_NAME_FORMATSIZE];
-
                                        dns_name_format(dns_db_origin(db),
                                                        namebuf,
                                                        sizeof(namebuf));
index 51d0ec5601235a1f21a38c2745a3562b1a487f04..da89e105b203ccc8356de7733c762a0fec94e73a 100644 (file)
@@ -1018,7 +1018,6 @@ dump_rdataset_raw(isc_mem_t *mctx, dns_name_t *name, dns_rdataset_t *rdataset,
 
        do {
                dns_rdata_t rdata = DNS_RDATA_INIT;
-               isc_region_t r;
 
                dns_rdataset_current(rdataset, &rdata);
                dns_rdata_toregion(&rdata, &r);
index 88c9239eb6d31886295ddbf3a5332cfbcb799a84..ff97d2d4cae3e5fb1745f8e2d9af0e08813fbfb0 100644 (file)
@@ -3231,14 +3231,13 @@ render_ecs(isc_buffer_t *optbuf, isc_buffer_t *target) {
                inet_ntop(AF_INET6, addr, addr_text, sizeof(addr_text));
        else {
                snprintf(addr_text, sizeof(addr_text),
-                        "Unsupported family %d",
-                        family);
+                        "Unsupported family %u", family);
                ADD_STRING(target, addr_text);
                return (ISC_R_SUCCESS);
        }
 
        ADD_STRING(target, addr_text);
-       sprintf(addr_text, "/%d/%d", addrlen, scopelen);
+       snprintf(addr_text, sizeof(addr_text), "/%d/%d", addrlen, scopelen);
        ADD_STRING(target, addr_text);
        return (ISC_R_SUCCESS);
 }
@@ -3317,7 +3316,6 @@ dns_message_pseudosectiontotext(dns_message_t *msg,
                                continue;
                        } else if (optcode == DNS_OPT_EXPIRE) {
                                if (optlen == 4) {
-                                       char buf[sizeof("4294967296")];
                                        isc_uint32_t secs;
                                        secs = isc_buffer_getuint32(&optbuf);
                                        ADD_STRING(target, "; EXPIRE: ");
@@ -3331,7 +3329,7 @@ dns_message_pseudosectiontotext(dns_message_t *msg,
                                ADD_STRING(target, "; EXPIRE");
                        } else {
                                ADD_STRING(target, "; OPT=");
-                               sprintf(buf, "%u", optcode);
+                               snprintf(buf, sizeof(buf), "%u", optcode);
                                ADD_STRING(target, buf);
                        }
 
@@ -3350,7 +3348,8 @@ dns_message_pseudosectiontotext(dns_message_t *msg,
                                                sep = " ";
                                                break;
                                        }
-                                       sprintf(buf, "%02x%s", optdata[i], sep);
+                                       snprintf(buf, sizeof(buf), "%02x%s",
+                                                optdata[i], sep);
                                        ADD_STRING(target, buf);
                                }
 
index 075fe32567ec4eadc2be662f5e940fed6c77696d..51910c10ed45fbfc3d0e8e142f407f316665c338 100644 (file)
@@ -2053,8 +2053,6 @@ dns_nsec3_noexistnodata(dns_rdatatype_t type, dns_name_t* name,
                    (scope >= 0 && (order > 0 ||
                                    memcmp(hash, nsec3.next, length) < 0)))
                {
-                       char namebuf[DNS_NAME_FORMATSIZE];
-
                        dns_name_format(qname, namebuf, sizeof(namebuf));
                        (*logit)(arg, ISC_LOG_DEBUG(3), "NSEC3 proves "
                                 "name does not exist: '%s'", namebuf);
index 6521279f2d9b457a67e7832f710ccb5509be6b83..402e331cb52453188832c71271cd44ce13fb661d 100644 (file)
@@ -162,7 +162,6 @@ dns_private_chains(dns_db_t *db, dns_dbversion_t *ver,
                for (result = dns_rdataset_first(&privateset);
                     result == ISC_R_SUCCESS;
                     result = dns_rdataset_next(&privateset)) {
-                       unsigned char buf[DNS_NSEC3PARAM_BUFFERSIZE];
                        dns_rdata_t private = DNS_RDATA_INIT;
                        dns_rdata_t rdata = DNS_RDATA_INIT;
 
index 36d9c7291406c13abdceaea8118360f36f92ca5b..69162d74583441cee230ab2f941d404be711c5c9 100644 (file)
@@ -5895,8 +5895,6 @@ allrdatasets(dns_db_t *db, dns_dbnode_t *node, dns_dbversion_t *version,
                        currentversion(db,
                                 (dns_dbversion_t **) (void *)(&rbtversion));
                else {
-                       unsigned int refs;
-
                        INSIST(rbtversion->rbtdb == rbtdb);
 
                        isc_refcount_increment(&rbtversion->references,
index 35fcf5d5822906fefc238f9329119893e5905edd..fbfa9757d6068f90551e720322cb17bb6678b4d8 100644 (file)
@@ -140,7 +140,7 @@ totext_nsec3(ARGS_TOTEXT) {
        unsigned int window, len;
        unsigned char hash;
        unsigned char flags;
-       char buf[sizeof("65535 ")];
+       char buf[sizeof("TYPE65535")];
        isc_uint32_t iterations;
        isc_boolean_t first;
 
@@ -224,7 +224,6 @@ totext_nsec3(ARGS_TOTEXT) {
                                if (dns_rdatatype_isknown(t)) {
                                        RETERR(dns_rdatatype_totext(t, target));
                                } else {
-                                       char buf[sizeof("TYPE65535")];
                                        sprintf(buf, "TYPE%u", t);
                                        RETERR(str_totext(buf, target));
                                }
index 51c3bc7da1786138f8de00dedb80af002670bcb4..d37d3e96b1a463486d156bd5078ba4c0fe6ca17f 100644 (file)
@@ -178,7 +178,6 @@ totext_rrsig(ARGS_TOTEXT) {
        if (dns_rdatatype_isknown(covered) && covered != 0) {
                RETERR(dns_rdatatype_totext(covered, target));
        } else {
-               char buf[sizeof("TYPE65535")];
                sprintf(buf, "TYPE%u", covered);
                RETERR(str_totext(buf, target));
        }
index 803a864067f0bc6cb4154aec806badc576bb4232..f4abfbdb4da7a63611930e10e578ab1a08dfe96d 100644 (file)
@@ -154,7 +154,6 @@ totext_sig(ARGS_TOTEXT) {
        if (dns_rdatatype_isknown(covered) && covered != 0) {
                RETERR(dns_rdatatype_totext(covered, target));
        } else {
-               char buf[sizeof("65535")];
                sprintf(buf, "%u", covered);
                RETERR(str_totext(buf, target));
        }
index 43c7b5b5601e4efa32e16f8039c0dfe992455b9d..260479105d92116a890e9506f95b72abb14c86c6 100644 (file)
@@ -417,7 +417,6 @@ towiresorted(dns_rdataset_t *rdataset, const dns_name_t *owner_name,
                         * 'Random' order.
                         */
                        for (i = 0; i < count; i++) {
-                               dns_rdata_t rdata;
                                isc_uint32_t val;
 
                                isc_random_get(&val);
index e84ddd1a9c1d4947a80fbb9d03e89676f145a088..4224a2d7130f4fb9d614a72de92487036a7f3ab6 100644 (file)
@@ -7638,8 +7638,6 @@ resquery_response(isc_task_t *task, isc_event_t *event) {
                        unsigned int flags, mask;
                        unsigned int version;
 #ifdef ISC_PLATFORM_USESIT
-                       unsigned char sit[64];
-
                        /*
                         * Some servers return BADVERS to unknown
                         * EDNS options.  This cannot be long term
index a90f1be63c2cb40332e184d098bc750b0b2b4cdf..42c92ef076b8a461dd308fa3b29bb421b96a4a0c 100644 (file)
@@ -486,13 +486,13 @@ decode_NegTokenInit(const unsigned char *p, size_t len, NegTokenInit * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        e = decode_MechTypeList(p, len, &(data)->mechTypes, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
@@ -513,16 +513,16 @@ decode_NegTokenInit(const unsigned char *p, size_t len, NegTokenInit * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        (data)->reqFlags = malloc(sizeof(*(data)->reqFlags));
                                        if ((data)->reqFlags == NULL)
                                                return ENOMEM;
                                        e = decode_ContextFlags(p, len, (data)->reqFlags, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
@@ -543,16 +543,16 @@ decode_NegTokenInit(const unsigned char *p, size_t len, NegTokenInit * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        (data)->mechToken = malloc(sizeof(*(data)->mechToken));
                                        if ((data)->mechToken == NULL)
                                                return ENOMEM;
                                        e = decode_octet_string(p, len, (data)->mechToken, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
@@ -573,16 +573,16 @@ decode_NegTokenInit(const unsigned char *p, size_t len, NegTokenInit * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        (data)->mechListMIC = malloc(sizeof(*(data)->mechListMIC));
                                        if ((data)->mechListMIC == NULL)
                                                return ENOMEM;
                                        e = decode_octet_string(p, len, (data)->mechListMIC, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
@@ -710,16 +710,16 @@ decode_NegTokenResp(const unsigned char *p, size_t len, NegTokenResp * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        (data)->negState = malloc(sizeof(*(data)->negState));
                                        if ((data)->negState == NULL)
                                                return ENOMEM;
                                        e = decode_enumerated(p, len, (data)->negState, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
@@ -740,16 +740,16 @@ decode_NegTokenResp(const unsigned char *p, size_t len, NegTokenResp * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        (data)->supportedMech = malloc(sizeof(*(data)->supportedMech));
                                        if ((data)->supportedMech == NULL)
                                                return ENOMEM;
                                        e = decode_MechType(p, len, (data)->supportedMech, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
@@ -770,16 +770,16 @@ decode_NegTokenResp(const unsigned char *p, size_t len, NegTokenResp * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        (data)->responseToken = malloc(sizeof(*(data)->responseToken));
                                        if ((data)->responseToken == NULL)
                                                return ENOMEM;
                                        e = decode_octet_string(p, len, (data)->responseToken, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
@@ -800,16 +800,16 @@ decode_NegTokenResp(const unsigned char *p, size_t len, NegTokenResp * data, siz
                                e = der_get_length(p, len, &newlen, &l);
                                FORW;
                                {
-                                       int dce_fix;
+                                       int mydce_fix;
                                        oldlen = len;
-                                       if ((dce_fix = fix_dce(newlen, &len)) < 0)
+                                       if ((mydce_fix = fix_dce(newlen, &len)) < 0)
                                                return ASN1_BAD_FORMAT;
                                        (data)->mechListMIC = malloc(sizeof(*(data)->mechListMIC));
                                        if ((data)->mechListMIC == NULL)
                                                return ENOMEM;
                                        e = decode_octet_string(p, len, (data)->mechListMIC, &l);
                                        FORW;
-                                       if (dce_fix) {
+                                       if (mydce_fix) {
                                                e = der_match_tag_and_length(p, len, (Der_class) 0, (Der_type) 0, 0, &reallen, &l);
                                                FORW;
                                        } else
index fc4b3c87dddb71bc83c9c63033cb6a9ef8b5977d..e0cb1a668988327d76d91263c71e069cdec8e5d0 100644 (file)
@@ -52,16 +52,16 @@ ATF_TC_HEAD(getoriginnode, tc) {
 ATF_TC_BODY(getoriginnode, tc) {
        dns_db_t *db = NULL;
        dns_dbnode_t *node = NULL;
-       isc_mem_t *mctx = NULL;
+       isc_mem_t *mymctx = NULL;
        isc_result_t result;
 
-       result = isc_mem_create(0, 0, &mctx);
+       result = isc_mem_create(0, 0, &mymctx);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
-       result = isc_hash_create(mctx, NULL, 256);
+       result = isc_hash_create(mymctx, NULL, 256);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
-       result = dns_db_create(mctx, "rbt", dns_rootname, dns_dbtype_zone,
+       result = dns_db_create(mymctx, "rbt", dns_rootname, dns_dbtype_zone,
                            dns_rdataclass_in, 0, NULL, &db);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
@@ -74,7 +74,7 @@ ATF_TC_BODY(getoriginnode, tc) {
        dns_db_detachnode(db, &node);
 
        dns_db_detach(&db);
-       isc_mem_detach(&mctx);
+       isc_mem_detach(&mymctx);
 }
 
 /*
index 618e59905fe0a6bd413960c48e44b11ae90e9771..535f0d22b5f8a84cfc65b4559f0311dbc0187337 100644 (file)
@@ -43,8 +43,8 @@
 static char tempname[11] = "dtXXXXXXXX";
 
 static void
-callback(const char *file, int line, isc_assertiontype_t type,
-        const char *cond)
+local_callback(const char *file, int line, isc_assertiontype_t type,
+              const char *cond)
 {
        UNUSED(file); UNUSED(line); UNUSED(type); UNUSED(cond);
        if (strcmp(tempname, "dtXXXXXXXX"))
@@ -138,7 +138,7 @@ ATF_TC_BODY(attachversion_bad, tc) {
 
        UNUSED(tc);
 
-       attachversion(callback);
+       attachversion(local_callback);
 }
 
 static void
@@ -179,7 +179,7 @@ ATF_TC_BODY(closeversion_bad, tc) {
 
        UNUSED(tc);
 
-       closeversion(callback);
+       closeversion(local_callback);
 }
 
 static void
@@ -227,7 +227,7 @@ ATF_TC_BODY(find_bad, tc) {
 
        UNUSED(tc);
 
-       find(callback);
+       find(local_callback);
 }
 
 static void
@@ -281,7 +281,7 @@ ATF_TC_BODY(allrdatasets_bad, tc) {
 
        UNUSED(tc);
 
-       allrdatasets(callback);
+       allrdatasets(local_callback);
 }
 
 static void
@@ -336,7 +336,7 @@ ATF_TC_BODY(findrdataset_bad, tc) {
 
        UNUSED(tc);
 
-       findrdataset(callback);
+       findrdataset(local_callback);
 }
 
 static void
@@ -391,7 +391,7 @@ ATF_TC_BODY(deleterdataset_bad, tc) {
 
        UNUSED(tc);
 
-       deleterdataset(callback);
+       deleterdataset(local_callback);
 }
 
 static void
@@ -453,7 +453,7 @@ ATF_TC_BODY(subtractrdataset_bad, tc) {
 
        UNUSED(tc);
 
-       subtract(callback);
+       subtract(local_callback);
 }
 
 static void
@@ -501,7 +501,7 @@ ATF_TC_BODY(dump_bad, tc) {
 
        UNUSED(tc);
 
-       dump(callback);
+       dump(local_callback);
 }
 
 static void
@@ -563,7 +563,7 @@ ATF_TC_BODY(addrdataset_bad, tc) {
 
        UNUSED(tc);
 
-       addrdataset(callback);
+       addrdataset(local_callback);
 }
 
 static void
@@ -612,7 +612,7 @@ ATF_TC_BODY(getnsec3parameters_bad, tc) {
 
        UNUSED(tc);
 
-       getnsec3parameters(callback);
+       getnsec3parameters(local_callback);
 }
 
 static void
@@ -706,7 +706,7 @@ ATF_TC_BODY(resigned_bad, tc) {
 
        UNUSED(tc);
 
-       resigned(callback);
+       resigned(local_callback);
 }
 
 /*
index d04d44552f420af59c01a7b8221cf5cbe217a7ac..f7bc3674213c7c20dbd636202dcff2bfae3e8dc5 100644 (file)
@@ -75,10 +75,10 @@ tohexstr(unsigned char *d, unsigned int len, char *out) {
 
        out[0]='\0';
        char c_ret[] = "AA";
-       unsigned int i;
+       unsigned int j;
        strcat(out, "0x");
-       for (i = 0; i < len; i++) {
-               sprintf(c_ret, "%02X", d[i]);
+       for (j = 0; j < len; j++) {
+               sprintf(c_ret, "%02X", d[j]);
                strcat(out, c_ret);
        }
        strcat(out, "\0");
index ecdd582631dd267ac56baf0492a4ff707b2935e3..3de44c8f22a9dd7946a57e38150bb03c16bf705c 100644 (file)
@@ -557,29 +557,29 @@ ATF_TC_BODY(dumpraw, tc) {
        isc_result_t result;
        dns_db_t *db = NULL;
        dns_dbversion_t *version = NULL;
-       char origin[sizeof(TEST_ORIGIN)];
-       dns_name_t dns_origin;
+       char myorigin[sizeof(TEST_ORIGIN)];
+       dns_name_t dnsorigin;
        isc_buffer_t source, target;
-       unsigned char name_buf[BUFLEN];
+       unsigned char namebuf[BUFLEN];
        int len;
 
        UNUSED(tc);
 
-       strcpy(origin, TEST_ORIGIN);
-       len = strlen(origin);
-       isc_buffer_init(&source, origin, len);
+       strcpy(myorigin, TEST_ORIGIN);
+       len = strlen(myorigin);
+       isc_buffer_init(&source, myorigin, len);
        isc_buffer_add(&source, len);
        isc_buffer_setactive(&source, len);
-       isc_buffer_init(&target, name_buf, BUFLEN);
-       dns_name_init(&dns_origin, NULL);
-       result = dns_name_fromtext(&dns_origin, &source, dns_rootname,
+       isc_buffer_init(&target, namebuf, BUFLEN);
+       dns_name_init(&dnsorigin, NULL);
+       result = dns_name_fromtext(&dnsorigin, &source, dns_rootname,
                                   0, &target);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dns_test_begin(NULL, ISC_FALSE);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
-       result = dns_db_create(mctx, "rbt", &dns_origin, dns_dbtype_zone,
+       result = dns_db_create(mctx, "rbt", &dnsorigin, dns_dbtype_zone,
                               dns_rdataclass_in, 0, NULL, &db);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
@@ -624,11 +624,11 @@ static const char *warn_expect_value;
 static isc_boolean_t warn_expect_result;
 
 static void
-warn_expect(struct dns_rdatacallbacks *callbacks, const char *fmt, ...) {
+warn_expect(struct dns_rdatacallbacks *mycallbacks, const char *fmt, ...) {
        char buf[4096];
        va_list ap;
 
-       UNUSED(callbacks);
+       UNUSED(mycallbacks);
 
        va_start(ap, fmt);
        vsnprintf(buf, sizeof(buf), fmt, ap);
index 59ec34dbcef1c843ff3b0c5fab8586c92e80ff66..694d7118edd7d9dce09c273ff6c79be3298f84c7 100644 (file)
@@ -193,7 +193,7 @@ fix_data(dns_rbtnode_t *p, void *base, size_t max, void *arg,
  * Load test data into the RBT.
  */
 static void
-add_test_data(isc_mem_t *mctx, dns_rbt_t *rbt) {
+add_test_data(isc_mem_t *mymctx, dns_rbt_t *rbt) {
        char buffer[1024];
        isc_buffer_t b;
        isc_result_t result;
@@ -202,7 +202,7 @@ add_test_data(isc_mem_t *mctx, dns_rbt_t *rbt) {
        dns_compress_t cctx;
        rbt_testdata_t *testdatap = testdata;
 
-       dns_compress_init(&cctx, -1, mctx);
+       dns_compress_init(&cctx, -1, mymctx);
 
        while (testdatap->name != NULL && testdatap->data.data != NULL) {
                memmove(buffer, testdatap->name, testdatap->name_len);
index 62653186396dbc4aaf3eb70d1e23cdbd26e0c337..74029e3483977700f41bc5a2ad5a5a33f5940b55 100644 (file)
@@ -42,7 +42,7 @@ ATF_TC_HEAD(zonemgr_create, tc) {
        atf_tc_set_md_var(tc, "descr", "create zone manager");
 }
 ATF_TC_BODY(zonemgr_create, tc) {
-       dns_zonemgr_t *zonemgr = NULL;
+       dns_zonemgr_t *myzonemgr = NULL;
        isc_result_t result;
 
        UNUSED(tc);
@@ -51,12 +51,12 @@ ATF_TC_BODY(zonemgr_create, tc) {
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dns_zonemgr_create(mctx, taskmgr, timermgr, socketmgr,
-                                   &zonemgr);
+                                   &myzonemgr);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
-       dns_zonemgr_shutdown(zonemgr);
-       dns_zonemgr_detach(&zonemgr);
-       ATF_REQUIRE_EQ(zonemgr, NULL);
+       dns_zonemgr_shutdown(myzonemgr);
+       dns_zonemgr_detach(&myzonemgr);
+       ATF_REQUIRE_EQ(myzonemgr, NULL);
 
        dns_test_end();
 }
@@ -67,7 +67,7 @@ ATF_TC_HEAD(zonemgr_managezone, tc) {
        atf_tc_set_md_var(tc, "descr", "manage and release a zone");
 }
 ATF_TC_BODY(zonemgr_managezone, tc) {
-       dns_zonemgr_t *zonemgr = NULL;
+       dns_zonemgr_t *myzonemgr = NULL;
        dns_zone_t *zone = NULL;
        isc_result_t result;
 
@@ -77,35 +77,35 @@ ATF_TC_BODY(zonemgr_managezone, tc) {
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dns_zonemgr_create(mctx, taskmgr, timermgr, socketmgr,
-                                   &zonemgr);
+                                   &myzonemgr);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dns_test_makezone("foo", &zone, NULL, ISC_FALSE);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        /* This should not succeed until the dns_zonemgr_setsize() is run */
-       result = dns_zonemgr_managezone(zonemgr, zone);
+       result = dns_zonemgr_managezone(myzonemgr, zone);
        ATF_REQUIRE_EQ(result, ISC_R_FAILURE);
 
-       ATF_REQUIRE_EQ(dns_zonemgr_getcount(zonemgr, DNS_ZONESTATE_ANY), 0);
+       ATF_REQUIRE_EQ(dns_zonemgr_getcount(myzonemgr, DNS_ZONESTATE_ANY), 0);
 
-       result = dns_zonemgr_setsize(zonemgr, 1);
+       result = dns_zonemgr_setsize(myzonemgr, 1);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        /* Now it should succeed */
-       result = dns_zonemgr_managezone(zonemgr, zone);
+       result = dns_zonemgr_managezone(myzonemgr, zone);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
-       ATF_REQUIRE_EQ(dns_zonemgr_getcount(zonemgr, DNS_ZONESTATE_ANY), 1);
+       ATF_REQUIRE_EQ(dns_zonemgr_getcount(myzonemgr, DNS_ZONESTATE_ANY), 1);
 
-       dns_zonemgr_releasezone(zonemgr, zone);
+       dns_zonemgr_releasezone(myzonemgr, zone);
        dns_zone_detach(&zone);
 
-       ATF_REQUIRE_EQ(dns_zonemgr_getcount(zonemgr, DNS_ZONESTATE_ANY), 0);
+       ATF_REQUIRE_EQ(dns_zonemgr_getcount(myzonemgr, DNS_ZONESTATE_ANY), 0);
 
-       dns_zonemgr_shutdown(zonemgr);
-       dns_zonemgr_detach(&zonemgr);
-       ATF_REQUIRE_EQ(zonemgr, NULL);
+       dns_zonemgr_shutdown(myzonemgr);
+       dns_zonemgr_detach(&myzonemgr);
+       ATF_REQUIRE_EQ(myzonemgr, NULL);
 
        dns_test_end();
 }
@@ -115,7 +115,7 @@ ATF_TC_HEAD(zonemgr_createzone, tc) {
        atf_tc_set_md_var(tc, "descr", "create and release a zone");
 }
 ATF_TC_BODY(zonemgr_createzone, tc) {
-       dns_zonemgr_t *zonemgr = NULL;
+       dns_zonemgr_t *myzonemgr = NULL;
        dns_zone_t *zone = NULL;
        isc_result_t result;
 
@@ -125,27 +125,27 @@ ATF_TC_BODY(zonemgr_createzone, tc) {
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dns_zonemgr_create(mctx, taskmgr, timermgr, socketmgr,
-                                   &zonemgr);
+                                   &myzonemgr);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        /* This should not succeed until the dns_zonemgr_setsize() is run */
-       result = dns_zonemgr_createzone(zonemgr, &zone);
+       result = dns_zonemgr_createzone(myzonemgr, &zone);
        ATF_REQUIRE_EQ(result, ISC_R_FAILURE);
 
-       result = dns_zonemgr_setsize(zonemgr, 1);
+       result = dns_zonemgr_setsize(myzonemgr, 1);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        /* Now it should succeed */
-       result = dns_zonemgr_createzone(zonemgr, &zone);
+       result = dns_zonemgr_createzone(myzonemgr, &zone);
        ATF_CHECK_EQ(result, ISC_R_SUCCESS);
        ATF_CHECK(zone != NULL);
 
        if (zone != NULL)
                dns_zone_detach(&zone);
 
-       dns_zonemgr_shutdown(zonemgr);
-       dns_zonemgr_detach(&zonemgr);
-       ATF_REQUIRE_EQ(zonemgr, NULL);
+       dns_zonemgr_shutdown(myzonemgr);
+       dns_zonemgr_detach(&myzonemgr);
+       ATF_REQUIRE_EQ(myzonemgr, NULL);
 
        dns_test_end();
 }
@@ -155,7 +155,7 @@ ATF_TC_HEAD(zonemgr_unreachable, tc) {
        atf_tc_set_md_var(tc, "descr", "manage and release a zone");
 }
 ATF_TC_BODY(zonemgr_unreachable, tc) {
-       dns_zonemgr_t *zonemgr = NULL;
+       dns_zonemgr_t *myzonemgr = NULL;
        dns_zone_t *zone = NULL;
        isc_sockaddr_t addr1, addr2;
        struct in_addr in;
@@ -171,57 +171,57 @@ ATF_TC_BODY(zonemgr_unreachable, tc) {
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dns_zonemgr_create(mctx, taskmgr, timermgr, socketmgr,
-                                   &zonemgr);
+                                   &myzonemgr);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dns_test_makezone("foo", &zone, NULL, ISC_FALSE);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
-       result = dns_zonemgr_setsize(zonemgr, 1);
+       result = dns_zonemgr_setsize(myzonemgr, 1);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
-       result = dns_zonemgr_managezone(zonemgr, zone);
+       result = dns_zonemgr_managezone(myzonemgr, zone);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        in.s_addr = inet_addr("10.53.0.1");
        isc_sockaddr_fromin(&addr1, &in, 2112);
        in.s_addr = inet_addr("10.53.0.2");
        isc_sockaddr_fromin(&addr2, &in, 5150);
-       ATF_CHECK(! dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
+       ATF_CHECK(! dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
        /*
         * We require multiple unreachableadd calls to mark a server as
         * unreachable.
         */
-       dns_zonemgr_unreachableadd(zonemgr, &addr1, &addr2, &now);
-       ATF_CHECK(! dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
-       dns_zonemgr_unreachableadd(zonemgr, &addr1, &addr2, &now);
-       ATF_CHECK(dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
+       dns_zonemgr_unreachableadd(myzonemgr, &addr1, &addr2, &now);
+       ATF_CHECK(! dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
+       dns_zonemgr_unreachableadd(myzonemgr, &addr1, &addr2, &now);
+       ATF_CHECK(dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
 
        in.s_addr = inet_addr("10.53.0.3");
        isc_sockaddr_fromin(&addr2, &in, 5150);
-       ATF_CHECK(! dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
+       ATF_CHECK(! dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
        /*
         * We require multiple unreachableadd calls to mark a server as
         * unreachable.
         */
-       dns_zonemgr_unreachableadd(zonemgr, &addr1, &addr2, &now);
-       dns_zonemgr_unreachableadd(zonemgr, &addr1, &addr2, &now);
-       ATF_CHECK(dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
+       dns_zonemgr_unreachableadd(myzonemgr, &addr1, &addr2, &now);
+       dns_zonemgr_unreachableadd(myzonemgr, &addr1, &addr2, &now);
+       ATF_CHECK(dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
 
-       dns_zonemgr_unreachabledel(zonemgr, &addr1, &addr2);
-       ATF_CHECK(! dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
+       dns_zonemgr_unreachabledel(myzonemgr, &addr1, &addr2);
+       ATF_CHECK(! dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
 
        in.s_addr = inet_addr("10.53.0.2");
        isc_sockaddr_fromin(&addr2, &in, 5150);
-       ATF_CHECK(dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
-       dns_zonemgr_unreachabledel(zonemgr, &addr1, &addr2);
-       ATF_CHECK(! dns_zonemgr_unreachable(zonemgr, &addr1, &addr2, &now));
+       ATF_CHECK(dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
+       dns_zonemgr_unreachabledel(myzonemgr, &addr1, &addr2);
+       ATF_CHECK(! dns_zonemgr_unreachable(myzonemgr, &addr1, &addr2, &now));
 
-       dns_zonemgr_releasezone(zonemgr, zone);
+       dns_zonemgr_releasezone(myzonemgr, zone);
        dns_zone_detach(&zone);
-       dns_zonemgr_shutdown(zonemgr);
-       dns_zonemgr_detach(&zonemgr);
-       ATF_REQUIRE_EQ(zonemgr, NULL);
+       dns_zonemgr_shutdown(myzonemgr);
+       dns_zonemgr_detach(&myzonemgr);
+       ATF_REQUIRE_EQ(myzonemgr, NULL);
 
        dns_test_end();
 }
index 202784c911f1e6966429f2c9bbe1685776e6004c..8f639161be1999ceaef0fb605cb0e1622390add3 100644 (file)
@@ -865,7 +865,7 @@ buildquery(dns_message_t *msg, dns_name_t *name,
        dns_rdataset_t *question = NULL, *tkeyset = NULL;
        dns_rdatalist_t *tkeylist = NULL;
        dns_rdata_t *rdata = NULL;
-       isc_buffer_t *dynbuf = NULL;
+       isc_buffer_t *dynbuf = NULL, *anamebuf = NULL, *qnamebuf = NULL;
        isc_result_t result;
 
        REQUIRE(msg != NULL);
@@ -881,6 +881,8 @@ buildquery(dns_message_t *msg, dns_name_t *name,
                                  dns_rdatatype_tkey);
 
        RETERR(isc_buffer_allocate(msg->mctx, &dynbuf, 4096));
+       RETERR(isc_buffer_allocate(msg->mctx, &anamebuf, DNS_NAME_MAXWIRE));
+       RETERR(isc_buffer_allocate(msg->mctx, &qnamebuf, DNS_NAME_MAXWIRE));
        RETERR(dns_message_gettemprdata(msg, &rdata));
 
        RETERR(dns_rdata_fromstruct(rdata, dns_rdataclass_any,
@@ -900,15 +902,16 @@ buildquery(dns_message_t *msg, dns_name_t *name,
        RETERR(dns_rdatalist_tordataset(tkeylist, tkeyset));
 
        dns_name_init(qname, NULL);
-       dns_name_clone(name, qname);
+       dns_name_copy(name, qname, qnamebuf);
 
        dns_name_init(aname, NULL);
-       dns_name_clone(name, aname);
+       dns_name_copy(name, aname, anamebuf);
 
        ISC_LIST_APPEND(qname->list, question, link);
        ISC_LIST_APPEND(aname->list, tkeyset, link);
 
        dns_message_addname(msg, qname, DNS_SECTION_QUESTION);
+       dns_message_takebuffer(msg, &qnamebuf);
 
        /*
         * Windows 2000 needs this in the answer section, not the additional
@@ -918,6 +921,7 @@ buildquery(dns_message_t *msg, dns_name_t *name,
                dns_message_addname(msg, aname, DNS_SECTION_ANSWER);
        else
                dns_message_addname(msg, aname, DNS_SECTION_ADDITIONAL);
+       dns_message_takebuffer(msg, &anamebuf);
 
        return (ISC_R_SUCCESS);
 
@@ -932,6 +936,10 @@ buildquery(dns_message_t *msg, dns_name_t *name,
        }
        if (dynbuf != NULL)
                isc_buffer_free(&dynbuf);
+       if (qnamebuf != NULL)
+               isc_buffer_free(&qnamebuf);
+       if (anamebuf != NULL)
+               isc_buffer_free(&anamebuf);
        printf("buildquery error\n");
        return (result);
 }
index 2dc40d7e2c0615966f37e027440fcafe004719f8..249d3a68354ffa76fad13f9a8b81ba4731d9505f 100644 (file)
@@ -15,8 +15,6 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-/* $Id$ */
-
 /*! \file */
 
 #include <config.h>
@@ -1827,7 +1825,6 @@ zone_load(dns_zone_t *zone, unsigned int flags) {
         */
        if (strcmp(zone->db_argv[0], "dlz") == 0) {
                dns_dlzdb_t *dlzdb;
-               dns_db_t *db = NULL;
                dns_dlzfindzone_t findzone;
 
                for (dlzdb = ISC_LIST_HEAD(zone->view->dlz_unsearched);
@@ -5843,7 +5840,7 @@ del_sigs(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *ver, dns_name_t *name,
        unsigned int i;
        dns_rdata_rrsig_t rrsig;
        isc_boolean_t found;
-       isc_int64_t warn = 0, maybe = 0;
+       isc_int64_t timewarn = 0, timemaybe = 0;
 
        dns_rdataset_init(&rdataset);
 
@@ -5949,18 +5946,20 @@ del_sigs(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *ver, dns_name_t *name,
                                {
                                        isc_int64_t timeexpire =
                                           dns_time64_from32(rrsig.timeexpire);
-                                       if (warn != 0 && warn > timeexpire)
-                                               warn = timeexpire;
+                                       if (timewarn != 0 &&
+                                           timewarn > timeexpire)
+                                               timewarn = timeexpire;
                                        if (rdata.flags & DNS_RDATA_OFFLINE) {
-                                               if (maybe == 0 ||
-                                                   maybe > timeexpire)
-                                                       maybe = timeexpire;
+                                               if (timemaybe == 0 ||
+                                                   timemaybe > timeexpire)
+                                                       timemaybe = timeexpire;
                                                break;
                                        }
-                                       if (warn == 0)
-                                               warn = maybe;
-                                       if (warn == 0 || warn > timeexpire)
-                                               warn = timeexpire;
+                                       if (timewarn == 0)
+                                               timewarn = timemaybe;
+                                       if (timewarn == 0 ||
+                                           timewarn > timeexpire)
+                                               timewarn = timeexpire;
                                        result = offline(db, ver, zonediff,
                                                         name, rdataset.ttl,
                                                         &rdata);
@@ -5989,12 +5988,13 @@ del_sigs(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *ver, dns_name_t *name,
        dns_rdataset_disassociate(&rdataset);
        if (result == ISC_R_NOMORE)
                result = ISC_R_SUCCESS;
-       if (warn > 0) {
+       if (timewarn > 0) {
 #if defined(STDTIME_ON_32BITS)
-               isc_stdtime_t stdwarn = (isc_stdtime_t)warn;
-               if (warn == stdwarn)
+               isc_stdtime_t stdwarn = (isc_stdtime_t)timewarn;
+               if (timewarn == stdwarn)
 #endif
-                       set_key_expiry_warning(zone, (isc_stdtime_t)warn, now);
+                       set_key_expiry_warning(zone, (isc_stdtime_t)timewarn,
+                                              now);
 #if defined(STDTIME_ON_32BITS)
                else
                        dns_zone_log(zone, ISC_LOG_ERROR,
@@ -7783,12 +7783,12 @@ zone_nsec3chain(dns_zone_t *zone) {
 
        LOCK_ZONE(zone);
        if (ISC_LIST_HEAD(zone->nsec3chain) != NULL) {
-               isc_interval_t i;
+               isc_interval_t interval;
                if (zone->update_disabled || result != ISC_R_SUCCESS)
-                       isc_interval_set(&i, 60, 0);            /* 1 minute */
+                       isc_interval_set(&interval, 60, 0);       /* 1 minute */
                else
-                       isc_interval_set(&i, 0, 10000000);      /* 10 ms */
-               isc_time_nowplusinterval(&zone->nsec3chaintime, &i);
+                       isc_interval_set(&interval, 0, 10000000); /* 10 ms */
+               isc_time_nowplusinterval(&zone->nsec3chaintime, &interval);
        } else
                isc_time_settoepoch(&zone->nsec3chaintime);
        UNLOCK_ZONE(zone);
@@ -8333,12 +8333,12 @@ zone_sign(dns_zone_t *zone) {
                dns_db_detach(&db);
 
        if (ISC_LIST_HEAD(zone->signing) != NULL) {
-               isc_interval_t i;
+               isc_interval_t interval;
                if (zone->update_disabled || result != ISC_R_SUCCESS)
-                       isc_interval_set(&i, 60, 0);            /* 1 minute */
+                       isc_interval_set(&interval, 60, 0);       /* 1 minute */
                else
-                       isc_interval_set(&i, 0, 10000000);      /* 10 ms */
-               isc_time_nowplusinterval(&zone->signingtime, &i);
+                       isc_interval_set(&interval, 0, 10000000); /* 10 ms */
+               isc_time_nowplusinterval(&zone->signingtime, &interval);
        } else
                isc_time_settoepoch(&zone->signingtime);
 
@@ -11246,12 +11246,12 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
 
        serial = soa.serial;
        if (DNS_ZONE_FLAG(zone, DNS_ZONEFLG_LOADED)) {
-               unsigned int soacount;
-               result = zone_get_from_db(zone, zone->db, NULL, &soacount,
+               unsigned int dbsoacount;
+               result = zone_get_from_db(zone, zone->db, NULL, &dbsoacount,
                                          &oldserial, NULL, NULL, NULL, NULL,
                                          NULL);
                RUNTIME_CHECK(result == ISC_R_SUCCESS);
-               RUNTIME_CHECK(soacount > 0U);
+               RUNTIME_CHECK(dbsoacount > 0U);
                zone_debuglog(zone, me, 1, "serial: new %u, old %u",
                              serial, oldserial);
        } else
index 46df64f4efe302cb5eb59099c7d1ea17e20b4d3b..4981da3dea79a723e785cfd17820ea07ed6d006b 100644 (file)
@@ -118,7 +118,7 @@ typedef ISC_LIST(debuglink_t)       debuglist_t;
 static ISC_LIST(isc__mem_t)    contexts;
 
 static isc_once_t              once = ISC_ONCE_INIT;
-static isc_mutex_t             lock;
+static isc_mutex_t             contextslock;
 static isc_mutex_t             createlock;
 
 /*%
@@ -874,7 +874,7 @@ default_memfree(void *arg, void *ptr) {
 static void
 initialize_action(void) {
        RUNTIME_CHECK(isc_mutex_init(&createlock) == ISC_R_SUCCESS);
-       RUNTIME_CHECK(isc_mutex_init(&lock) == ISC_R_SUCCESS);
+       RUNTIME_CHECK(isc_mutex_init(&contextslock) == ISC_R_SUCCESS);
        ISC_LIST_INIT(contexts);
        totallost = 0;
 }
@@ -1002,9 +1002,9 @@ isc_mem_createx2(size_t init_max_size, size_t target_size,
 
        ctx->memalloc_failures = 0;
 
-       LOCK(&lock);
+       LOCK(&contextslock);
        ISC_LIST_INITANDAPPEND(contexts, ctx, link);
-       UNLOCK(&lock);
+       UNLOCK(&contextslock);
 
        *ctxp = (isc_mem_t *)ctx;
        return (ISC_R_SUCCESS);
@@ -1032,10 +1032,10 @@ destroy(isc__mem_t *ctx) {
        unsigned int i;
        isc_ondestroy_t ondest;
 
-       LOCK(&lock);
+       LOCK(&contextslock);
        ISC_LIST_UNLINK(contexts, ctx, link);
        totallost += ctx->inuse;
-       UNLOCK(&lock);
+       UNLOCK(&contextslock);
 
        ctx->common.impmagic = 0;
        ctx->common.magic = 0;
@@ -2293,14 +2293,14 @@ isc_mem_printallactive(FILE *file) {
 
        RUNTIME_CHECK(isc_once_do(&once, initialize_action) == ISC_R_SUCCESS);
 
-       LOCK(&lock);
+       LOCK(&contextslock);
        for (ctx = ISC_LIST_HEAD(contexts);
             ctx != NULL;
             ctx = ISC_LIST_NEXT(ctx, link)) {
                fprintf(file, "context: %p\n", ctx);
                print_active(ctx, file);
        }
-       UNLOCK(&lock);
+       UNLOCK(&contextslock);
 #endif
 }
 
@@ -2312,7 +2312,7 @@ isc_mem_checkdestroyed(FILE *file) {
 
        RUNTIME_CHECK(isc_once_do(&once, initialize_action) == ISC_R_SUCCESS);
 
-       LOCK(&lock);
+       LOCK(&contextslock);
        if (!ISC_LIST_EMPTY(contexts))  {
 #if ISC_MEM_TRACKLINES
                isc__mem_t *ctx;
@@ -2327,7 +2327,7 @@ isc_mem_checkdestroyed(FILE *file) {
 #endif
                INSIST(0);
        }
-       UNLOCK(&lock);
+       UNLOCK(&contextslock);
 }
 
 unsigned int
@@ -2464,18 +2464,18 @@ isc_mem_renderxml(xmlTextWriterPtr writer) {
 
        RUNTIME_CHECK(isc_once_do(&once, initialize_action) == ISC_R_SUCCESS);
 
-       LOCK(&lock);
+       LOCK(&contextslock);
        lost = totallost;
        for (ctx = ISC_LIST_HEAD(contexts);
             ctx != NULL;
             ctx = ISC_LIST_NEXT(ctx, link)) {
                xmlrc = xml_renderctx(ctx, &summary, writer);
                if (xmlrc < 0) {
-                       UNLOCK(&lock);
+                       UNLOCK(&contextslock);
                        goto error;
                }
        }
-       UNLOCK(&lock);
+       UNLOCK(&contextslock);
 
        TRY0(xmlTextWriterEndElement(writer)); /* contexts */
 
@@ -2631,18 +2631,18 @@ isc_mem_renderjson(json_object *memobj) {
        ctxarray = json_object_new_array();
        CHECKMEM(ctxarray);
 
-       LOCK(&lock);
+       LOCK(&contextslock);
        lost = totallost;
        for (ctx = ISC_LIST_HEAD(contexts);
             ctx != NULL;
             ctx = ISC_LIST_NEXT(ctx, link)) {
                result = json_renderctx(ctx, &summary, ctxarray);
                if (result != ISC_R_SUCCESS) {
-                       UNLOCK(&lock);
+                       UNLOCK(&contextslock);
                        goto error;
                }
        }
-       UNLOCK(&lock);
+       UNLOCK(&contextslock);
 
        obj = json_object_new_int64(summary.total);
        CHECKMEM(obj);
index 31f6f3f5b4279f59ea622b4a6b7a099da3b4faba..e702e599ef6d801a5dd5e16ff1dec330eb42f568 100644 (file)
@@ -153,7 +153,6 @@ isc_ratelimiter_enqueue(isc_ratelimiter_t *rl, isc_task_t *task,
        LOCK(&rl->lock);
        if (rl->state == isc_ratelimiter_ratelimited ||
            rl->state == isc_ratelimiter_stalled) {
-               isc_event_t *ev = *eventp;
                ev->ev_sender = task;
                ISC_LIST_APPEND(rl->pending, ev, ev_link);
                *eventp = NULL;
index 2bd2fb4218ed86f40b52dfdeb415c2807209e174..b915c4ea2510de9d25faf7e073cb762f8de74330 100644 (file)
@@ -40,7 +40,7 @@ typedef struct resulttable {
        ISC_LINK(struct resulttable)            link;
 } resulttable;
 
-static const char *text[ISC_R_NRESULTS] = {
+static const char *description[ISC_R_NRESULTS] = {
        "success",                              /*%< 0 */
        "out of memory",                        /*%< 1 */
        "timed out",                            /*%< 2 */
@@ -153,8 +153,8 @@ initialize_action(void) {
        RUNTIME_CHECK(isc_mutex_init(&lock) == ISC_R_SUCCESS);
        ISC_LIST_INIT(tables);
 
-       result = register_table(ISC_RESULTCLASS_ISC, ISC_R_NRESULTS, text,
-                               isc_msgcat, ISC_RESULT_RESULTSET);
+       result = register_table(ISC_RESULTCLASS_ISC, ISC_R_NRESULTS,
+                               description, isc_msgcat, ISC_RESULT_RESULTSET);
        if (result != ISC_R_SUCCESS)
                UNEXPECTED_ERROR(__FILE__, __LINE__,
                                 "register_table() %s: %u",
index 407eff6a1b4787b94c312051fd21efec9919b9e6..a4dbd7c3d20e9f823dde600012f755ea57e98e54 100644 (file)
@@ -42,7 +42,6 @@ unsigned char buffer[1024];
 const char *s;
 char str[2 * ISC_SHA512_DIGESTLENGTH + 1];
 unsigned char key[20];
-int i = 0;
 
 isc_result_t
 tohexstr(unsigned char *d, unsigned int len, char *out);
@@ -95,6 +94,7 @@ ATF_TC_HEAD(isc_sha1, tc) {
 }
 ATF_TC_BODY(isc_sha1, tc) {
        isc_sha1_t sha1;
+       int i;
 
        UNUSED(tc);
 
@@ -217,13 +217,13 @@ ATF_TC_BODY(isc_sha1, tc) {
        }
 }
 
-
 ATF_TC(isc_sha224);
 ATF_TC_HEAD(isc_sha224, tc) {
        atf_tc_set_md_var(tc, "descr", "sha224 examples from RFC4634");
 }
 ATF_TC_BODY(isc_sha224, tc) {
        isc_sha224_t sha224;
+       int i;
 
        UNUSED(tc);
 
@@ -348,7 +348,6 @@ ATF_TC_BODY(isc_sha224, tc) {
 
                testcase++;
        }
-
 }
 
 ATF_TC(isc_sha256);
@@ -357,6 +356,7 @@ ATF_TC_HEAD(isc_sha256, tc) {
 }
 ATF_TC_BODY(isc_sha256, tc) {
        isc_sha256_t sha256;
+       int i;
 
        UNUSED(tc);
 
@@ -480,7 +480,6 @@ ATF_TC_BODY(isc_sha256, tc) {
 
                testcase++;
        }
-
 }
 
 ATF_TC(isc_sha384);
@@ -489,6 +488,7 @@ ATF_TC_HEAD(isc_sha384, tc) {
 }
 ATF_TC_BODY(isc_sha384, tc) {
        isc_sha384_t sha384;
+       int i;
 
        UNUSED(tc);
 
@@ -626,7 +626,6 @@ ATF_TC_BODY(isc_sha384, tc) {
 
                testcase++;
        }
-
 }
 
 ATF_TC(isc_sha512);
@@ -635,6 +634,7 @@ ATF_TC_HEAD(isc_sha512, tc) {
 }
 ATF_TC_BODY(isc_sha512, tc) {
        isc_sha512_t sha512;
+       int i;
 
        UNUSED(tc);
 
@@ -773,7 +773,6 @@ ATF_TC_BODY(isc_sha512, tc) {
 
                testcase++;
        }
-
 }
 
 ATF_TC(isc_md5);
@@ -782,6 +781,7 @@ ATF_TC_HEAD(isc_md5, tc) {
 }
 ATF_TC_BODY(isc_md5, tc) {
        isc_md5_t md5;
+       int i;
 
        UNUSED(tc);
 
@@ -1792,6 +1792,7 @@ ATF_TC_HEAD(isc_crc64, tc) {
 }
 ATF_TC_BODY(isc_crc64, tc) {
        isc_uint64_t crc;
+       int i;
 
        UNUSED(tc);
 
index 3c4444d57ea7a02d4e77cacc1856c1163afeb9af..00e94d04deab2188e5310b6753a9d48bddc019a8 100644 (file)
@@ -1495,10 +1495,10 @@ cfg_print_mapbody(cfg_printer_t *pctx, const cfg_obj_t *obj) {
                        result = isc_symtab_lookup(obj->value.map.symtab,
                                                   clause->name, 0, &symval);
                        if (result == ISC_R_SUCCESS) {
-                               cfg_obj_t *obj = symval.as_pointer;
-                               if (obj->type == &cfg_type_implicitlist) {
+                               cfg_obj_t *symobj = symval.as_pointer;
+                               if (symobj->type == &cfg_type_implicitlist) {
                                        /* Multivalued. */
-                                       cfg_list_t *list = &obj->value.list;
+                                       cfg_list_t *list = &symobj->value.list;
                                        cfg_listelt_t *elt;
                                        for (elt = ISC_LIST_HEAD(*list);
                                             elt != NULL;
@@ -1514,7 +1514,7 @@ cfg_print_mapbody(cfg_printer_t *pctx, const cfg_obj_t *obj) {
                                        print_indent(pctx);
                                        cfg_print_cstr(pctx, clause->name);
                                        cfg_print_chars(pctx, " ", 1);
-                                       cfg_print_obj(pctx, obj);
+                                       cfg_print_obj(pctx, symobj);
                                        cfg_print_chars(pctx, ";\n", 2);
                                }
                        } else if (result == ISC_R_NOTFOUND) {
index 922d2aa45e14ebea2107de890379349ca667a833..1bfc9f368e68a10252812e94a53ecd9172285277 100644 (file)
@@ -232,14 +232,14 @@ struct hostent *
 lwres_gethostbyname_r(const char *name, struct hostent *resbuf,
                char *buf, int buflen, int *error)
 {
-       struct hostent *he;
+       struct hostent *myhe;
        int res;
 
-       he = lwres_getipnodebyname(name, AF_INET, 0, error);
-       if (he == NULL)
+       myhe = lwres_getipnodebyname(name, AF_INET, 0, error);
+       if (myhe == NULL)
                return (NULL);
-       res = copytobuf(he, resbuf, buf, buflen);
-       lwres_freehostent(he);
+       res = copytobuf(myhe, resbuf, buf, buflen);
+       lwres_freehostent(myhe);
        if (res != 0) {
                errno = ERANGE;
                return (NULL);
@@ -253,14 +253,14 @@ lwres_gethostbyaddr_r(const char *addr, int len, int type,
                      struct hostent *resbuf, char *buf, int buflen,
                      int *error)
 {
-       struct hostent *he;
+       struct hostent *myhe;
        int res;
 
-       he = lwres_getipnodebyaddr(addr, len, type, error);
-       if (he == NULL)
+       myhe = lwres_getipnodebyaddr(addr, len, type, error);
+       if (myhe == NULL)
                return (NULL);
-       res = copytobuf(he, resbuf, buf, buflen);
-       lwres_freehostent(he);
+       res = copytobuf(myhe, resbuf, buf, buflen);
+       lwres_freehostent(myhe);
        if (res != 0) {
                errno = ERANGE;
                return (NULL);
@@ -296,7 +296,7 @@ lwres_endhostent_r(void) {
 }
 
 static int
-copytobuf(struct hostent *he, struct hostent *hptr, char *buf, int buflen) {
+copytobuf(struct hostent *src, struct hostent *hptr, char *buf, int buflen) {
        char *cp;
        char **ptr;
        int i, n;
@@ -307,13 +307,13 @@ copytobuf(struct hostent *he, struct hostent *hptr, char *buf, int buflen) {
         */
        nptr = 2; /* NULL ptrs */
        len = (int)((char *)LWRES_ALIGN(buf) - buf);
-       for (i = 0; he->h_addr_list[i]; i++, nptr++) {
-               len += he->h_length;
+       for (i = 0; src->h_addr_list[i]; i++, nptr++) {
+               len += src->h_length;
        }
-       for (i = 0; he->h_aliases[i]; i++, nptr++) {
-               len += strlen(he->h_aliases[i]) + 1;
+       for (i = 0; src->h_aliases[i]; i++, nptr++) {
+               len += strlen(src->h_aliases[i]) + 1;
        }
-       len += strlen(he->h_name) + 1;
+       len += strlen(src->h_name) + 1;
        len += nptr * sizeof(char*);
 
        if (len > buflen) {
@@ -323,8 +323,8 @@ copytobuf(struct hostent *he, struct hostent *hptr, char *buf, int buflen) {
        /*
         * Copy address size and type.
         */
-       hptr->h_addrtype = he->h_addrtype;
-       n = hptr->h_length = he->h_length;
+       hptr->h_addrtype = src->h_addrtype;
+       n = hptr->h_length = src->h_length;
 
        ptr = (char **)LWRES_ALIGN(buf);
        cp = (char *)LWRES_ALIGN(buf) + nptr * sizeof(char *);
@@ -333,8 +333,8 @@ copytobuf(struct hostent *he, struct hostent *hptr, char *buf, int buflen) {
         * Copy address list.
         */
        hptr->h_addr_list = ptr;
-       for (i = 0; he->h_addr_list[i]; i++, ptr++) {
-               memmove(cp, he->h_addr_list[i], n);
+       for (i = 0; src->h_addr_list[i]; i++, ptr++) {
+               memmove(cp, src->h_addr_list[i], n);
                hptr->h_addr_list[i] = cp;
                cp += n;
        }
@@ -344,8 +344,8 @@ copytobuf(struct hostent *he, struct hostent *hptr, char *buf, int buflen) {
        /*
         * Copy official name.
         */
-       n = strlen(he->h_name) + 1;
-       strcpy(cp, he->h_name);
+       n = strlen(src->h_name) + 1;
+       strcpy(cp, src->h_name);
        hptr->h_name = cp;
        cp += n;
 
@@ -353,9 +353,9 @@ copytobuf(struct hostent *he, struct hostent *hptr, char *buf, int buflen) {
         * Copy aliases.
         */
        hptr->h_aliases = ptr;
-       for (i = 0; he->h_aliases[i]; i++) {
-               n = strlen(he->h_aliases[i]) + 1;
-               strcpy(cp, he->h_aliases[i]);
+       for (i = 0; src->h_aliases[i]; i++) {
+               n = strlen(src->h_aliases[i]) + 1;
+               strcpy(cp, src->h_aliases[i]);
                hptr->h_aliases[i] = cp;
                cp += n;
        }
index d38d96f34600d61c8eebe7643be064ace00270c9..926bfde3fd2760ca43e76560d03091c69a479fc4 100644 (file)
@@ -61,7 +61,7 @@ static isc_appctx_t *actx = NULL;
 static isc_mem_t *mctx = NULL;
 static unsigned int outstanding_probes = 0;
 const char *cacheserver = "127.0.0.1";
-static FILE *fp;
+static FILE *input;
 
 typedef enum {
        none,
@@ -979,7 +979,7 @@ probe_domain(struct probe_trans *trans) {
        REQUIRE(outstanding_probes < MAX_PROBES);
 
        /* Construct domain */
-       cp = fgets(buf, sizeof(buf), fp);
+       cp = fgets(buf, sizeof(buf), input);
        if (cp == NULL)
                return (ISC_R_NOMORE);
        if ((cp = strchr(buf, '\n')) != NULL) /* zap NL if any */
@@ -1128,10 +1128,10 @@ main(int argc, char *argv[]) {
 
        /* Open input file */
        if (argc == 0)
-               fp = stdin;
+               input = stdin;
        else {
-               fp = fopen(argv[0], "r");
-               if (fp == NULL) {
+               input = fopen(argv[0], "r");
+               if (input == NULL) {
                        fprintf(stderr, "failed to open input file: %s\n",
                                argv[0]);
                        exit(1);