Adam Tkac 76a982
diff -up bind-9.7.3/bin/dig/dighost.c.cleanup bind-9.7.3/bin/dig/dighost.c
Adam Tkac 76a982
--- bind-9.7.3/bin/dig/dighost.c.cleanup	2011-02-18 15:28:59.505352049 +0100
Adam Tkac 76a982
+++ bind-9.7.3/bin/dig/dighost.c	2011-02-18 15:29:22.065302188 +0100
Adam Tkac 76a982
@@ -4300,7 +4300,6 @@ opentmpkey(isc_mem_t *mctx, const char *
Adam Tkac 76a982
 	char *tempnamekey = NULL;
Adam Tkac 76a982
 	int tempnamelen;
Adam Tkac 76a982
 	int tempnamekeylen;
Adam Tkac 76a982
-	char *x;
Adam Tkac 76a982
 	char *cp;
Adam Tkac 76a982
 	isc_uint32_t which;
Adam Tkac 76a982
 
Adam Tkac 76a982
@@ -4323,11 +4322,11 @@ opentmpkey(isc_mem_t *mctx, const char *
Adam Tkac 76a982
 			return (ISC_R_FAILURE);
Adam Tkac 76a982
 		}
Adam Tkac 76a982
 
Adam Tkac 76a982
-		x = cp--;
Adam Tkac 76a982
+		cp--;
Adam Tkac 76a982
 		while (cp >= tempname && *cp == 'X') {
Adam Tkac 76a982
 			isc_random_get(&which);
Adam Tkac 76a982
 			*cp = alphnum[which % (sizeof(alphnum) - 1)];
Adam Tkac 76a982
-			x = cp--;
Adam Tkac 76a982
+			cp--;
Adam Tkac 76a982
 		}
Adam Tkac 76a982
 
Adam Tkac 76a982
 		tempnamekeylen = tempnamelen+5;
Adam Tkac 76a982
diff -up bind-9.7.3/bin/dnssec/dnssec-signzone.c.cleanup bind-9.7.3/bin/dnssec/dnssec-signzone.c
Adam Tkac 76a982
--- bind-9.7.3/bin/dnssec/dnssec-signzone.c.cleanup	2011-02-18 15:29:34.615274529 +0100
Adam Tkac 76a982
+++ bind-9.7.3/bin/dnssec/dnssec-signzone.c	2011-02-18 15:29:51.075239402 +0100
Adam Tkac 76a982
@@ -1467,7 +1467,6 @@ verifyzone(void) {
Adam Tkac 76a982
 	isc_boolean_t done = ISC_FALSE;
Adam Tkac 76a982
 	isc_boolean_t first = ISC_TRUE;
Adam Tkac 76a982
 	isc_boolean_t goodksk = ISC_FALSE;
Adam Tkac 76a982
-	isc_boolean_t goodzsk = ISC_FALSE;
Adam Tkac 76a982
 	isc_result_t result;
Adam Tkac 76a982
 	unsigned char revoked_ksk[256];
Adam Tkac 76a982
 	unsigned char revoked_zsk[256];
Adam Tkac 76a982
@@ -1569,7 +1568,6 @@ verifyzone(void) {
Adam Tkac 76a982
 #endif
Adam Tkac 76a982
 			if (zsk_algorithms[dnskey.algorithm] != 255)
Adam Tkac 76a982
 				zsk_algorithms[dnskey.algorithm]++;
Adam Tkac 76a982
-			goodzsk = ISC_TRUE;
Adam Tkac 76a982
 		} else {
Adam Tkac 76a982
 			if (standby_zsk[dnskey.algorithm] != 255)
Adam Tkac 76a982
 				standby_zsk[dnskey.algorithm]++;
Adam Tkac 76a982
diff -up bind-9.7.3/bin/named/server.c.cleanup bind-9.7.3/bin/named/server.c
Adam Tkac 76a982
--- bind-9.7.3/bin/named/server.c.cleanup	2011-02-18 15:27:38.565530746 +0100
Adam Tkac 76a982
+++ bind-9.7.3/bin/named/server.c	2011-02-18 15:27:55.065494317 +0100
Adam Tkac 76a982
@@ -1413,7 +1413,6 @@ configure_view(dns_view_t *view, cfg_par
Adam Tkac 76a982
 	isc_uint32_t max_clients_per_query;
Adam Tkac 76a982
 	const char *sep = ": view ";
Adam Tkac 76a982
 	const char *viewname = view->name;
Adam Tkac 76a982
-	const char *forview = " for view ";
Adam Tkac 76a982
 	isc_boolean_t rfc1918;
Adam Tkac 76a982
 	isc_boolean_t empty_zones_enable;
Adam Tkac 76a982
 	const cfg_obj_t *disablelist = NULL;
Adam Tkac 76a982
@@ -1458,7 +1457,6 @@ configure_view(dns_view_t *view, cfg_par
Adam Tkac 76a982
 	if (!strcmp(viewname, "_default")) {
Adam Tkac 76a982
 		sep = "";
Adam Tkac 76a982
 		viewname = "";
Adam Tkac 76a982
-		forview = "";
Adam Tkac 76a982
 	}
Adam Tkac 76a982
 
Adam Tkac 76a982
 	/*
Adam Tkac 76a982
diff -up bind-9.7.3/contrib/dlz/drivers/dlz_postgres_driver.c.cleanup bind-9.7.3/contrib/dlz/drivers/dlz_postgres_driver.c
Adam Tkac 76a982
--- bind-9.7.3/contrib/dlz/drivers/dlz_postgres_driver.c.cleanup	2011-02-18 15:28:16.135447795 +0100
Adam Tkac 76a982
+++ bind-9.7.3/contrib/dlz/drivers/dlz_postgres_driver.c	2011-02-18 15:28:43.815387080 +0100
Adam Tkac 76a982
@@ -292,8 +292,10 @@ postgres_get_resultset(const char *zone,
Adam Tkac 76a982
 	unsigned int i = 0;
Adam Tkac 76a982
 	unsigned int j = 0;
Adam Tkac 76a982
 
Adam Tkac 76a982
+#if 0
Adam Tkac 76a982
 	/* temporarily get a unique thread # */
Adam Tkac 76a982
 	unsigned int dlz_thread_num = 1+(int) (1000.0*rand()/(RAND_MAX+1.0));
Adam Tkac 76a982
+#endif
Adam Tkac 76a982
 
Adam Tkac 76a982
 	REQUIRE(*rs == NULL);
Adam Tkac 76a982
 
Adam Tkac 76a982
diff -up bind-9.7.3/lib/bind9/check.c.cleanup bind-9.7.3/lib/bind9/check.c
Adam Tkac 76a982
--- bind-9.7.3/lib/bind9/check.c.cleanup	2011-02-18 15:26:31.305679252 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/bind9/check.c	2011-02-18 15:27:21.205572047 +0100
Adam Tkac 76a982
@@ -1818,8 +1818,6 @@ check_trusted_key(const cfg_obj_t *key, 
Adam Tkac 76a982
 		  isc_log_t *logctx)
Adam Tkac 76a982
 {
Adam Tkac 76a982
 	const char *keystr, *keynamestr;
Adam Tkac 76a982
-	dns_fixedname_t fkeyname;
Adam Tkac 76a982
-	dns_name_t *keyname;
Adam Tkac 76a982
 	isc_buffer_t keydatabuf;
Adam Tkac 76a982
 	isc_region_t r;
Adam Tkac 76a982
 	isc_result_t result = ISC_R_SUCCESS;
Adam Tkac 76a982
@@ -1830,7 +1828,6 @@ check_trusted_key(const cfg_obj_t *key, 
Adam Tkac 76a982
 	flags = cfg_obj_asuint32(cfg_tuple_get(key, "flags"));
Adam Tkac 76a982
 	proto = cfg_obj_asuint32(cfg_tuple_get(key, "protocol"));
Adam Tkac 76a982
 	alg = cfg_obj_asuint32(cfg_tuple_get(key, "algorithm"));
Adam Tkac 76a982
-	keyname = dns_fixedname_name(&fkeyname);
Adam Tkac 76a982
 	keynamestr = cfg_obj_asstring(cfg_tuple_get(key, "name"));
Adam Tkac 76a982
 
Adam Tkac 76a982
 	if (flags > 0xffff) {
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/adb.c.cleanup bind-9.7.3/lib/dns/adb.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/adb.c.cleanup	2011-02-18 15:18:50.436696481 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/adb.c	2011-02-18 15:19:16.686638556 +0100
Adam Tkac 76a982
@@ -1853,7 +1853,6 @@ check_expire_name(dns_adbname_t **namep,
Adam Tkac 76a982
 static void
Adam Tkac 76a982
 check_stale_name(dns_adb_t *adb, int bucket, isc_stdtime_t now) {
Adam Tkac 76a982
 	int victims, max_victims;
Adam Tkac 76a982
-	isc_boolean_t result;
Adam Tkac 76a982
 	dns_adbname_t *victim, *next_victim;
Adam Tkac 76a982
 	isc_boolean_t overmem = isc_mem_isovermem(adb->mctx);
Adam Tkac 76a982
 	int scans = 0;
Adam Tkac 76a982
@@ -1875,7 +1874,7 @@ check_stale_name(dns_adb_t *adb, int buc
Adam Tkac 76a982
 		INSIST(!NAME_DEAD(victim));
Adam Tkac 76a982
 		scans++;
Adam Tkac 76a982
 		next_victim = ISC_LIST_PREV(victim, plink);
Adam Tkac 76a982
-		result = check_expire_name(&victim, now);
Adam Tkac 76a982
+		(void) check_expire_name(&victim, now);
Adam Tkac 76a982
 		if (victim == NULL) {
Adam Tkac 76a982
 			victims++;
Adam Tkac 76a982
 			goto next;
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/diff.c.cleanup bind-9.7.3/lib/dns/diff.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/diff.c.cleanup	2011-02-18 15:20:02.006538543 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/diff.c	2011-02-18 15:20:28.426480612 +0100
Adam Tkac 76a982
@@ -264,7 +264,6 @@ diff_apply(dns_diff_t *diff, dns_db_t *d
Adam Tkac 76a982
 			dns_rdataset_t rds;
Adam Tkac 76a982
 			dns_rdataset_t ardataset;
Adam Tkac 76a982
 			dns_rdataset_t *modified = NULL;
Adam Tkac 76a982
-			isc_boolean_t offline;
Adam Tkac 76a982
 
Adam Tkac 76a982
 			op = t->op;
Adam Tkac 76a982
 			type = t->rdata.type;
Adam Tkac 76a982
@@ -301,7 +300,6 @@ diff_apply(dns_diff_t *diff, dns_db_t *d
Adam Tkac 76a982
 				CHECK(dns_db_findnsec3node(db, name, ISC_TRUE,
Adam Tkac 76a982
 							   &node));
Adam Tkac 76a982
 
Adam Tkac 76a982
-			offline = ISC_FALSE;
Adam Tkac 76a982
 			while (t != NULL &&
Adam Tkac 76a982
 			       dns_name_equal(&t->name, name) &&
Adam Tkac 76a982
 			       t->op == op &&
Adam Tkac 76a982
@@ -323,8 +321,6 @@ diff_apply(dns_diff_t *diff, dns_db_t *d
Adam Tkac 76a982
 						namebuf, typebuf, classbuf,
Adam Tkac 76a982
 						(unsigned long) t->ttl,
Adam Tkac 76a982
 						(unsigned long) rdl.ttl);
Adam Tkac 76a982
-				if (t->rdata.flags & DNS_RDATA_OFFLINE)
Adam Tkac 76a982
-					offline = ISC_TRUE;
Adam Tkac 76a982
 				ISC_LIST_APPEND(rdl.rdata, &t->rdata, link);
Adam Tkac 76a982
 				t = ISC_LIST_NEXT(t, link);
Adam Tkac 76a982
 			}
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/dnssec.c.cleanup bind-9.7.3/lib/dns/dnssec.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/dnssec.c.cleanup	2011-02-18 15:20:39.306456222 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/dnssec.c	2011-02-18 15:21:31.896340160 +0100
Adam Tkac 76a982
@@ -1068,7 +1068,6 @@ isc_result_t
Adam Tkac 76a982
 dns_dnsseckey_create(isc_mem_t *mctx, dst_key_t **dstkey,
Adam Tkac 76a982
 		     dns_dnsseckey_t **dkp)
Adam Tkac 76a982
 {
Adam Tkac 76a982
-	isc_result_t result;
Adam Tkac 76a982
 	dns_dnsseckey_t *dk;
Adam Tkac 76a982
 	int major, minor;
Adam Tkac 76a982
 
Adam Tkac 76a982
@@ -1094,7 +1093,7 @@ dns_dnsseckey_create(isc_mem_t *mctx, ds
Adam Tkac 76a982
 	dk->ksk = ISC_TF((dst_key_flags(dk->key) & DNS_KEYFLAG_KSK) != 0);
Adam Tkac 76a982
 
Adam Tkac 76a982
 	/* Is this an old-style key? */
Adam Tkac 76a982
-	result = dst_key_getprivateformat(dk->key, &major, &minor);
Adam Tkac 76a982
+	(void) dst_key_getprivateformat(dk->key, &major, &minor);
Adam Tkac 76a982
 
Adam Tkac 76a982
 	/* Smart signing started with key format 1.3 */
Adam Tkac 76a982
 	dk->legacy = ISC_TF(major == 1 && minor <= 2);
Adam Tkac 76a982
@@ -1673,9 +1672,6 @@ dns_dnssec_updatekeys(dns_dnsseckeylist_
Adam Tkac 76a982
 
Adam Tkac 76a982
 		/* No match found in keys; add the new key. */
Adam Tkac 76a982
 		if (key2 == NULL) {
Adam Tkac 76a982
-			dns_dnsseckey_t *next;
Adam Tkac 76a982
-
Adam Tkac 76a982
-			next = ISC_LIST_NEXT(key1, link);
Adam Tkac 76a982
 			ISC_LIST_UNLINK(*newkeys, key1, link);
Adam Tkac 76a982
 			ISC_LIST_APPEND(*keys, key1, link);
Adam Tkac 76a982
 
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/gssapictx.c.cleanup bind-9.7.3/lib/dns/gssapictx.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/gssapictx.c.cleanup	2011-02-18 15:25:53.975761658 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/gssapictx.c	2011-02-18 15:26:05.945735228 +0100
Adam Tkac 76a982
@@ -119,7 +119,6 @@ name_to_gbuffer(dns_name_t *name, isc_bu
Adam Tkac 76a982
 {
Adam Tkac 76a982
 	dns_name_t tname, *namep;
Adam Tkac 76a982
 	isc_region_t r;
Adam Tkac 76a982
-	isc_result_t result;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	if (!dns_name_isabsolute(name))
Adam Tkac 76a982
 		namep = name;
Adam Tkac 76a982
@@ -132,7 +131,7 @@ name_to_gbuffer(dns_name_t *name, isc_bu
Adam Tkac 76a982
 		namep = &tname;
Adam Tkac 76a982
 	}
Adam Tkac 76a982
 
Adam Tkac 76a982
-	result = dns_name_toprincipal(namep, buffer);
Adam Tkac 76a982
+	(void) dns_name_toprincipal(namep, buffer);
Adam Tkac 76a982
 	isc_buffer_putuint8(buffer, 0);
Adam Tkac 76a982
 	isc_buffer_usedregion(buffer, &r);
Adam Tkac 76a982
 	REGION_TO_GBUFFER(r, *gbuffer);
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/rdata/generic/dlv_32769.c.cleanup bind-9.7.3/lib/dns/rdata/generic/dlv_32769.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/rdata/generic/dlv_32769.c.cleanup	2011-02-18 15:22:02.456272719 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/rdata/generic/dlv_32769.c	2011-02-18 15:22:27.126218257 +0100
Adam Tkac 76a982
@@ -33,7 +33,6 @@ static inline isc_result_t
Adam Tkac 76a982
 fromtext_dlv(ARGS_FROMTEXT) {
Adam Tkac 76a982
 	isc_token_t token;
Adam Tkac 76a982
 	unsigned char c;
Adam Tkac 76a982
-	int length;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	REQUIRE(type == 32769);
Adam Tkac 76a982
 
Adam Tkac 76a982
@@ -71,15 +70,6 @@ fromtext_dlv(ARGS_FROMTEXT) {
Adam Tkac 76a982
 	RETERR(uint8_tobuffer(token.value.as_ulong, target));
Adam Tkac 76a982
 	c = (unsigned char) token.value.as_ulong;
Adam Tkac 76a982
 
Adam Tkac 76a982
-	/*
Adam Tkac 76a982
-	 * Digest.
Adam Tkac 76a982
-	 */
Adam Tkac 76a982
-	if (c == DNS_DSDIGEST_SHA1)
Adam Tkac 76a982
-		length = ISC_SHA1_DIGESTLENGTH;
Adam Tkac 76a982
-	else if (c == DNS_DSDIGEST_SHA256)
Adam Tkac 76a982
-		length = ISC_SHA256_DIGESTLENGTH;
Adam Tkac 76a982
-	else
Adam Tkac 76a982
-		length = -1;
Adam Tkac 76a982
 	return (isc_hex_tobuffer(lexer, target, -1));
Adam Tkac 76a982
 }
Adam Tkac 76a982
 
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/request.c.cleanup bind-9.7.3/lib/dns/request.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/request.c.cleanup	2011-02-18 15:22:40.566188593 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/request.c	2011-02-18 15:23:25.766088820 +0100
Adam Tkac 76a982
@@ -428,12 +428,10 @@ req_send(dns_request_t *request, isc_tas
Adam Tkac 76a982
 	isc_region_t r;
Adam Tkac 76a982
 	isc_socket_t *socket;
Adam Tkac 76a982
 	isc_result_t result;
Adam Tkac 76a982
-	unsigned int dispattr;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	req_log(ISC_LOG_DEBUG(3), "req_send: request %p", request);
Adam Tkac 76a982
 
Adam Tkac 76a982
 	REQUIRE(VALID_REQUEST(request));
Adam Tkac 76a982
-	dispattr = dns_dispatch_getattributes(request->dispatch);
Adam Tkac 76a982
 	socket = req_getsocket(request);
Adam Tkac 76a982
 	isc_buffer_usedregion(request->query, &r);
Adam Tkac 76a982
 	/*
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/tkey.c.cleanup bind-9.7.3/lib/dns/tkey.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/tkey.c.cleanup	2011-02-18 15:23:35.386067584 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/tkey.c	2011-02-18 15:23:52.796029159 +0100
Adam Tkac 76a982
@@ -70,10 +70,9 @@ static void
Adam Tkac 76a982
 _dns_tkey_dumpmessage(dns_message_t *msg) {
Adam Tkac 76a982
 	isc_buffer_t outbuf;
Adam Tkac 76a982
 	unsigned char output[4096];
Adam Tkac 76a982
-	isc_result_t result;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	isc_buffer_init(&outbuf, output, sizeof(output));
Adam Tkac 76a982
-	result = dns_message_totext(msg, &dns_master_style_debug, 0,
Adam Tkac 76a982
+	(void) dns_message_totext(msg, &dns_master_style_debug, 0,
Adam Tkac 76a982
 				    &outbuf);
Adam Tkac 76a982
 	/* XXXMLG ignore result */
Adam Tkac 76a982
 	fprintf(stderr, "%.*s\n", (int)isc_buffer_usedlength(&outbuf),
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/validator.c.cleanup bind-9.7.3/lib/dns/validator.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/validator.c.cleanup	2011-02-18 15:24:02.276008230 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/validator.c	2011-02-18 15:24:20.245971006 +0100
Adam Tkac 76a982
@@ -1223,7 +1223,6 @@ authvalidated(isc_task_t *task, isc_even
Adam Tkac 76a982
 	dns_validatorevent_t *devent;
Adam Tkac 76a982
 	dns_validator_t *val;
Adam Tkac 76a982
 	dns_rdataset_t *rdataset;
Adam Tkac 76a982
-	dns_rdataset_t *sigrdataset;
Adam Tkac 76a982
 	isc_boolean_t want_destroy;
Adam Tkac 76a982
 	isc_result_t result;
Adam Tkac 76a982
 	isc_boolean_t exists, data;
Adam Tkac 76a982
@@ -1233,7 +1232,6 @@ authvalidated(isc_task_t *task, isc_even
Adam Tkac 76a982
 
Adam Tkac 76a982
 	devent = (dns_validatorevent_t *)event;
Adam Tkac 76a982
 	rdataset = devent->rdataset;
Adam Tkac 76a982
-	sigrdataset = devent->sigrdataset;
Adam Tkac 76a982
 	val = devent->ev_arg;
Adam Tkac 76a982
 	result = devent->result;
Adam Tkac 76a982
 	dns_validator_destroy(&val->subvalidator);
Adam Tkac 76a982
diff -up bind-9.7.3/lib/dns/zone.c.cleanup bind-9.7.3/lib/dns/zone.c
Adam Tkac 76a982
--- bind-9.7.3/lib/dns/zone.c.cleanup	2011-02-18 15:24:33.825938590 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/dns/zone.c	2011-02-18 15:25:38.275796311 +0100
Adam Tkac 76a982
@@ -5647,11 +5647,11 @@ zone_nsec3chain(dns_zone_t *zone) {
Adam Tkac 76a982
 	dns_nsec3chainlist_t cleanup;
Adam Tkac 76a982
 	dst_key_t *zone_keys[MAXZONEKEYS];
Adam Tkac 76a982
 	isc_int32_t signatures;
Adam Tkac 76a982
-	isc_boolean_t check_ksk, keyset_kskonly, is_ksk;
Adam Tkac 76a982
+	isc_boolean_t check_ksk, keyset_kskonly;
Adam Tkac 76a982
 	isc_boolean_t delegation;
Adam Tkac 76a982
 	isc_boolean_t first;
Adam Tkac 76a982
 	isc_result_t result;
Adam Tkac 76a982
-	isc_stdtime_t now, inception, soaexpire, expire, stop;
Adam Tkac 76a982
+	isc_stdtime_t now, inception, soaexpire, expire;
Adam Tkac 76a982
 	isc_uint32_t jitter;
Adam Tkac 76a982
 	unsigned int i;
Adam Tkac 76a982
 	unsigned int nkeys = 0;
Adam Tkac 76a982
@@ -5716,7 +5716,6 @@ zone_nsec3chain(dns_zone_t *zone) {
Adam Tkac 76a982
 	 */
Adam Tkac 76a982
 	isc_random_get(&jitter);
Adam Tkac 76a982
 	expire = soaexpire - jitter % 3600;
Adam Tkac 76a982
-	stop = now + 5;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	check_ksk = DNS_ZONE_OPTION(zone, DNS_ZONEOPT_UPDATECHECKKSK);
Adam Tkac 76a982
 	keyset_kskonly = DNS_ZONE_OPTION(zone, DNS_ZONEOPT_DNSKEYKSKONLY);
Adam Tkac 76a982
@@ -5762,7 +5761,6 @@ zone_nsec3chain(dns_zone_t *zone) {
Adam Tkac 76a982
 		if (NSEC3REMOVE(nsec3chain->nsec3param.flags))
Adam Tkac 76a982
 			goto next_addchain;
Adam Tkac 76a982
 
Adam Tkac 76a982
-		is_ksk = ISC_FALSE;
Adam Tkac 76a982
 		delegation = ISC_FALSE;
Adam Tkac 76a982
 		dns_dbiterator_current(nsec3chain->dbiterator, &node, name);
Adam Tkac 76a982
 
Adam Tkac 76a982
@@ -6481,12 +6479,11 @@ zone_sign(dns_zone_t *zone) {
Adam Tkac 76a982
 	isc_boolean_t build_nsec3 = ISC_FALSE;
Adam Tkac 76a982
 	isc_boolean_t first;
Adam Tkac 76a982
 	isc_result_t result;
Adam Tkac 76a982
-	isc_stdtime_t now, inception, soaexpire, expire, stop;
Adam Tkac 76a982
+	isc_stdtime_t now, inception, soaexpire, expire;
Adam Tkac 76a982
 	isc_uint32_t jitter;
Adam Tkac 76a982
 	unsigned int i, j;
Adam Tkac 76a982
 	unsigned int nkeys = 0;
Adam Tkac 76a982
 	isc_uint32_t nodes;
Adam Tkac 76a982
-	isc_boolean_t was_ksk;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	dns_rdataset_init(&rdataset);
Adam Tkac 76a982
 	dns_fixedname_init(&fixed);
Adam Tkac 76a982
@@ -6538,7 +6535,6 @@ zone_sign(dns_zone_t *zone) {
Adam Tkac 76a982
 	 */
Adam Tkac 76a982
 	isc_random_get(&jitter);
Adam Tkac 76a982
 	expire = soaexpire - jitter % 3600;
Adam Tkac 76a982
-	stop = now + 5;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	/*
Adam Tkac 76a982
 	 * We keep pulling nodes off each iterator in turn until
Adam Tkac 76a982
@@ -6583,8 +6579,6 @@ zone_sign(dns_zone_t *zone) {
Adam Tkac 76a982
 
Adam Tkac 76a982
 		delegation = ISC_FALSE;
Adam Tkac 76a982
 
Adam Tkac 76a982
-		was_ksk = ISC_FALSE;
Adam Tkac 76a982
-
Adam Tkac 76a982
 		if (first && signing->delete) {
Adam Tkac 76a982
 			/*
Adam Tkac 76a982
 			 * Remove the key we are deleting from consideration.
Adam Tkac 76a982
diff -up bind-9.7.3/lib/isc/httpd.c.cleanup bind-9.7.3/lib/isc/httpd.c
Adam Tkac 76a982
--- bind-9.7.3/lib/isc/httpd.c.cleanup	2011-02-18 15:17:22.936889562 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/isc/httpd.c	2011-02-18 15:17:55.846816943 +0100
Adam Tkac 76a982
@@ -912,7 +912,6 @@ isc_httpd_senddone(isc_task_t *task, isc
Adam Tkac 76a982
 {
Adam Tkac 76a982
 	isc_httpd_t *httpd = ev->ev_arg;
Adam Tkac 76a982
 	isc_region_t r;
Adam Tkac 76a982
-	isc_result_t result;
Adam Tkac 76a982
 	isc_socketevent_t *sev = (isc_socketevent_t *)ev;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	ENTER("senddone");
Adam Tkac 76a982
@@ -963,8 +962,8 @@ isc_httpd_senddone(isc_task_t *task, isc
Adam Tkac 76a982
 
Adam Tkac 76a982
 	r.base = (unsigned char *)httpd->recvbuf;
Adam Tkac 76a982
 	r.length = HTTP_RECVLEN - 1;
Adam Tkac 76a982
-	result = isc_socket_recv(httpd->sock, &r, 1, task, isc_httpd_recvdone,
Adam Tkac 76a982
-				 httpd);
Adam Tkac 76a982
+	(void) isc_socket_recv(httpd->sock, &r, 1, task, isc_httpd_recvdone,
Adam Tkac 76a982
+			       httpd);
Adam Tkac 76a982
 
Adam Tkac 76a982
 out:
Adam Tkac 76a982
 	isc_event_free(&ev;;
Adam Tkac 76a982
diff -up bind-9.7.3/lib/isc/string.c.cleanup bind-9.7.3/lib/isc/string.c
Adam Tkac 76a982
--- bind-9.7.3/lib/isc/string.c.cleanup	2011-02-18 15:18:09.786786183 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/isc/string.c	2011-02-18 15:18:31.016739337 +0100
Adam Tkac 76a982
@@ -167,12 +167,11 @@ isc_string_printf(char *target, size_t s
Adam Tkac 76a982
 void
Adam Tkac 76a982
 isc_string_printf_truncate(char *target, size_t size, const char *format, ...) {
Adam Tkac 76a982
 	va_list args;
Adam Tkac 76a982
-	size_t n;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	REQUIRE(size > 0U);
Adam Tkac 76a982
 
Adam Tkac 76a982
 	va_start(args, format);
Adam Tkac 76a982
-	n = vsnprintf(target, size, format, args);
Adam Tkac 76a982
+	(void) vsnprintf(target, size, format, args);
Adam Tkac 76a982
 	va_end(args);
Adam Tkac 76a982
 
Adam Tkac 76a982
 	ENSURE(strlen(target) < size);
Adam Tkac 76a982
diff -up bind-9.7.3/lib/isc/timer.c.cleanup bind-9.7.3/lib/isc/timer.c
Adam Tkac 76a982
--- bind-9.7.3/lib/isc/timer.c.cleanup	2011-02-18 15:30:32.385146946 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/isc/timer.c	2011-02-18 15:31:22.585036094 +0100
Adam Tkac 76a982
@@ -333,7 +333,9 @@ schedule(isc__timer_t *timer, isc_time_t
Adam Tkac 76a982
 
Adam Tkac 76a982
 static inline void
Adam Tkac 76a982
 deschedule(isc__timer_t *timer) {
Adam Tkac 76a982
+#ifdef USE_TIMER_THREAD
Adam Tkac 76a982
 	isc_boolean_t need_wakeup = ISC_FALSE;
Adam Tkac 76a982
+#endif /* USE_TIMER_THREAD */
Adam Tkac 76a982
 	isc__timermgr_t *manager;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	/*
Adam Tkac 76a982
@@ -342,8 +344,10 @@ deschedule(isc__timer_t *timer) {
Adam Tkac 76a982
 
Adam Tkac 76a982
 	manager = timer->manager;
Adam Tkac 76a982
 	if (timer->index > 0) {
Adam Tkac 76a982
+#ifdef USE_TIMER_THREAD
Adam Tkac 76a982
 		if (timer->index == 1)
Adam Tkac 76a982
 			need_wakeup = ISC_TRUE;
Adam Tkac 76a982
+#endif /* USE_TIMER_THREAD */
Adam Tkac 76a982
 		isc_heap_delete(manager->heap, timer->index);
Adam Tkac 76a982
 		timer->index = 0;
Adam Tkac 76a982
 		INSIST(manager->nscheduled > 0);
Adam Tkac 76a982
diff -up bind-9.7.3/lib/isc/unix/socket.c.cleanup bind-9.7.3/lib/isc/unix/socket.c
Adam Tkac 76a982
--- bind-9.7.3/lib/isc/unix/socket.c.cleanup	2011-02-18 15:15:10.247182323 +0100
Adam Tkac 76a982
+++ bind-9.7.3/lib/isc/unix/socket.c	2011-02-18 15:16:48.806964868 +0100
Adam Tkac 76a982
@@ -2751,7 +2751,6 @@ isc__socket_close(isc_socket_t *sock0) {
Adam Tkac 76a982
 	isc__socket_t *sock = (isc__socket_t *)sock0;
Adam Tkac 76a982
 	int fd;
Adam Tkac 76a982
 	isc__socketmgr_t *manager;
Adam Tkac 76a982
-	isc_sockettype_t type;
Adam Tkac 76a982
 
Adam Tkac 76a982
 	REQUIRE(VALID_SOCKET(sock));
Adam Tkac 76a982
 
Adam Tkac 76a982
@@ -2771,7 +2770,6 @@ isc__socket_close(isc_socket_t *sock0) {
Adam Tkac 76a982
 	INSIST(sock->connect_ev == NULL);
Adam Tkac 76a982
 
Adam Tkac 76a982
 	manager = sock->manager;
Adam Tkac 76a982
-	type = sock->type;
Adam Tkac 76a982
 	fd = sock->fd;
Adam Tkac 76a982
 	sock->fd = -1;
Adam Tkac 76a982
 	memset(sock->name, 0, sizeof(sock->name));
Adam Tkac 76a982
@@ -3689,7 +3687,6 @@ static isc_threadresult_t
Adam Tkac 76a982
 watcher(void *uap) {
Adam Tkac 76a982
 	isc__socketmgr_t *manager = uap;
Adam Tkac 76a982
 	isc_boolean_t done;
Adam Tkac 76a982
-	int ctlfd;
Adam Tkac 76a982
 	int cc;
Adam Tkac 76a982
 #ifdef USE_KQUEUE
Adam Tkac 76a982
 	const char *fnname = "kevent()";
Adam Tkac 76a982
@@ -3701,6 +3698,7 @@ watcher(void *uap) {
Adam Tkac 76a982
 #elif defined (USE_SELECT)
Adam Tkac 76a982
 	const char *fnname = "select()";
Adam Tkac 76a982
 	int maxfd;
Adam Tkac 76a982
+	int ctlfd;
Adam Tkac 76a982
 #endif
Adam Tkac 76a982
 	char strbuf[ISC_STRERRORSIZE];
Adam Tkac 76a982
 #ifdef ISC_SOCKET_USE_POLLWATCH
Adam Tkac 76a982
@@ -3710,7 +3708,9 @@ watcher(void *uap) {
Adam Tkac 76a982
 	/*
Adam Tkac 76a982
 	 * Get the control fd here.  This will never change.
Adam Tkac 76a982
 	 */
Adam Tkac 76a982
+#ifdef USE_SELECT
Adam Tkac 76a982
 	ctlfd = manager->pipe_fds[0];
Adam Tkac 76a982
+#endif
Adam Tkac 76a982
 	done = ISC_FALSE;
Adam Tkac 76a982
 	while (!done) {
Adam Tkac 76a982
 		do {