use enum

diff --git a/lib/dns/zone.c b/lib/dns/zone.c
index 215c31a3169a691762ed9d24452838e9eea7cde9..c60e9f356a938bb827171d0469388e7edf5e04d9 100644 (file)
--- 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;
                        }