From: Mark Andrews <marka@isc.org>
Date: Wed, 29 Jan 2020 19:57:51 +0000 (+1100)
Subject: use enum
X-Git-Tag: v9.16.0~39^2~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7c0d9dac9fa961e0bea49b86304b28ac41b2ae8b;p=thirdparty%2Fbind9.git

use enum
---

diff --git a/lib/dns/zone.c b/lib/dns/zone.c
index 215c31a3169..c60e9f356a9 100644
--- a/lib/dns/zone.c
+++ b/lib/dns/zone.c
@@ -19137,6 +19137,8 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 	unsigned char algorithms[256];
 	unsigned int i;
 
+	enum { notexpected = 0, expected = 1, found = 2 };
+
 	REQUIRE(DNS_ZONE_VALID(zone));
 
 	result = dns_db_getoriginnode(db, &node);
@@ -19188,7 +19190,7 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 	 */
 	if (dns_rdataset_isassociated(&cds)) {
 		bool delete = false;
-		memset(algorithms, 0, sizeof(algorithms));
+		memset(algorithms, notexpected, sizeof(algorithms));
 		for (result = dns_rdataset_first(&cds);
 		     result == ISC_R_SUCCESS;
 		     result = dns_rdataset_next(&cds)) {
@@ -19208,7 +19210,7 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 			}
 			CHECK(dns_rdata_tostruct(&crdata, &structcds, NULL));
 			if (algorithms[structcds.algorithm] == 0) {
-				algorithms[structcds.algorithm] = 1;
+				algorithms[structcds.algorithm] = expected;
 			}
 			for (result = dns_rdataset_first(&dnskey);
 			     result == ISC_R_SUCCESS;
@@ -19224,7 +19226,7 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 				    memcmp(crdata.data, dsrdata.data,
 					   dsrdata.length) == 0)
 				{
-					algorithms[structcds.algorithm] = 2;
+					algorithms[structcds.algorithm] = found;
 				}
 			}
 			if (result != ISC_R_NOMORE) {
@@ -19233,11 +19235,11 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 		}
 		for (i = 0; i < sizeof(algorithms); i++) {
 			if (delete) {
-				if (algorithms[i] != 0) {
+				if (algorithms[i] != notexpected) {
 					result = DNS_R_BADCDS;
 					goto failure;
 				}
-			} else if (algorithms[i] == 1) {
+			} else if (algorithms[i] == expected) {
 				result = DNS_R_BADCDS;
 				goto failure;
 			}
@@ -19251,7 +19253,7 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 	 */
 	if (dns_rdataset_isassociated(&cdnskey)) {
 		bool delete = false;
-		memset(algorithms, 0, sizeof(algorithms));
+		memset(algorithms, notexpected, sizeof(algorithms));
 		for (result = dns_rdataset_first(&cdnskey);
 		     result == ISC_R_SUCCESS;
 		     result = dns_rdataset_next(&cdnskey)) {
@@ -19265,7 +19267,7 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 			 * and 2 zero octets.
 			 */
 			if (crdata.length == 5U &&
-			    memcmp(crdata.data, "\0\0\003\0", 5) == 0)
+			    memcmp(crdata.data, "\0\0\3\0", 5) == 0)
 			{
 				delete = true;
 				continue;
@@ -19273,7 +19275,7 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 			CHECK(dns_rdata_tostruct(&crdata, &structcdnskey,
 						 NULL));
 			if (algorithms[structcdnskey.algorithm] == 0) {
-				algorithms[structcdnskey.algorithm] = 1;
+				algorithms[structcdnskey.algorithm] = expected;
 			}
 			for (result = dns_rdataset_first(&dnskey);
 			     result == ISC_R_SUCCESS;
@@ -19285,7 +19287,8 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 				    memcmp(crdata.data, rdata.data,
 					   rdata.length) == 0)
 				{
-					algorithms[structcdnskey.algorithm] = 2;
+					algorithms[structcdnskey.algorithm] =
+							 found;
 				}
 			}
 			if (result != ISC_R_NOMORE) {
@@ -19294,11 +19297,11 @@ dns_zone_cdscheck(dns_zone_t *zone, dns_db_t *db, dns_dbversion_t *version) {
 		}
 		for (i = 0; i < sizeof(algorithms); i++) {
 			if (delete) {
-				if (algorithms[i] != 0) {
+				if (algorithms[i] != notexpected) {
 					result = DNS_R_BADCDNSKEY;
 					goto failure;
 				}
-			} else if (algorithms[i] == 1) {
+			} else if (algorithms[i] == expected) {
 				result = DNS_R_BADCDNSKEY;
 				goto failure;
 			}