]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
convert if to switch in get_key
authorEvan Hunt <each@isc.org>
Fri, 16 Aug 2019 23:27:50 +0000 (16:27 -0700)
committerEvan Hunt <each@isc.org>
Fri, 15 Nov 2019 22:26:07 +0000 (14:26 -0800)
lib/dns/validator.c

index fafc1fbdadb128c5c62609294811ca012fee9c9e..33ca5865ffb22c028e952aeb397606574df584b4 100644 (file)
@@ -1229,7 +1229,8 @@ get_key(dns_validator_t *val, dns_rdata_rrsig_t *siginfo) {
         * Do we know about this key?
         */
        result = view_find(val, &siginfo->signer, dns_rdatatype_dnskey);
-       if (result == ISC_R_SUCCESS) {
+       switch (result) {
+       case ISC_R_SUCCESS:
                /*
                 * We have an rrset for the given keyname.
                 */
@@ -1283,7 +1284,9 @@ get_key(dns_validator_t *val, dns_rdata_rrsig_t *siginfo) {
                                result = DNS_R_CONTINUE;
                        }
                }
-       } else if (result == ISC_R_NOTFOUND) {
+               break;
+
+       case ISC_R_NOTFOUND:
                /*
                 * We don't know anything about this key.
                 */
@@ -1294,18 +1297,23 @@ get_key(dns_validator_t *val, dns_rdata_rrsig_t *siginfo) {
                        return (result);
                }
                return (DNS_R_WAIT);
-       } else if (result ==  DNS_R_NCACHENXDOMAIN ||
-                  result == DNS_R_NCACHENXRRSET ||
-                  result == DNS_R_EMPTYNAME ||
-                  result == DNS_R_NXDOMAIN ||
-                  result == DNS_R_NXRRSET)
-       {
+
+       case DNS_R_NCACHENXDOMAIN:
+       case DNS_R_NCACHENXRRSET:
+       case DNS_R_EMPTYNAME:
+       case DNS_R_NXDOMAIN:
+       case DNS_R_NXRRSET:
                /*
                 * This key doesn't exist.
                 */
                result = DNS_R_CONTINUE;
-       } else if (result == DNS_R_BROKENCHAIN) {
+               break;
+
+       case DNS_R_BROKENCHAIN:
                return (result);
+
+       default:
+               break;
        }
 
        if (dns_rdataset_isassociated(&val->frdataset) &&