From: Mark Andrews Date: Fri, 29 Oct 2021 08:11:34 +0000 (+1100) Subject: Address cppcheck reports regarding function pointer X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=ff47c966f426407fa81b353095f4771bc0caaa85;p=thirdparty%2Fbind9.git Address cppcheck reports regarding function pointer cppcheck reports local address being returned with function pointers that are not deferrenced e.g. int function(int (*action)(int *)) { int local = 0; return ((action)(&local)); } replace with return ((*action)(&local)); --- diff --git a/lib/dns/name.c b/lib/dns/name.c index 295a5271882..a9b814ee231 100644 --- a/lib/dns/name.c +++ b/lib/dns/name.c @@ -2315,7 +2315,7 @@ dns_name_digest(const dns_name_t *name, dns_digestfunc_t digest, void *arg) { isc_buffer_usedregion(&buffer, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } bool diff --git a/lib/dns/rbt.c b/lib/dns/rbt.c index fa311094310..71d04c5f1e0 100644 --- a/lib/dns/rbt.c +++ b/lib/dns/rbt.c @@ -1051,9 +1051,9 @@ dns_rbt_findnode(dns_rbt_t *rbt, const dns_name_t *name, dns_name_t *foundname, return (result); } - result = (callback)(current, - callback_name, - callback_arg); + result = (*callback)(current, + callback_name, + callback_arg); if (result != DNS_R_CONTINUE) { saved_result = result; /* diff --git a/lib/dns/rdata.c b/lib/dns/rdata.c index 44e6ce45c01..a5e7831b06a 100644 --- a/lib/dns/rdata.c +++ b/lib/dns/rdata.c @@ -1309,7 +1309,7 @@ dns_rdata_digest(dns_rdata_t *rdata, dns_digestfunc_t digest, void *arg) { if (use_default) { dns_rdata_toregion(rdata, &r); - result = (digest)(arg, &r); + result = (*digest)(arg, &r); } return (result); diff --git a/lib/dns/rdata/ch_3/a_1.c b/lib/dns/rdata/ch_3/a_1.c index 7c5cad9e736..68c1e256664 100644 --- a/lib/dns/rdata/ch_3/a_1.c +++ b/lib/dns/rdata/ch_3/a_1.c @@ -281,7 +281,7 @@ digest_ch_a(ARGS_DIGEST) { dns_name_fromregion(&name, &r); isc_region_consume(&r, name_length(&name)); RETERR(dns_name_digest(&name, digest, arg)); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/afsdb_18.c b/lib/dns/rdata/generic/afsdb_18.c index 71c6ce61e54..f7de42e4fb9 100644 --- a/lib/dns/rdata/generic/afsdb_18.c +++ b/lib/dns/rdata/generic/afsdb_18.c @@ -254,7 +254,7 @@ additionaldata_afsdb(ARGS_ADDLDATA) { isc_region_consume(®ion, 2); dns_name_fromregion(&name, ®ion); - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t @@ -268,7 +268,7 @@ digest_afsdb(ARGS_DIGEST) { r2 = r1; isc_region_consume(&r2, 2); r1.length = 2; - RETERR((digest)(arg, &r1)); + RETERR((*digest)(arg, &r1)); dns_name_init(&name, NULL); dns_name_fromregion(&name, &r2); diff --git a/lib/dns/rdata/generic/amtrelay_260.c b/lib/dns/rdata/generic/amtrelay_260.c index d68267e86f7..0e1ab3ba535 100644 --- a/lib/dns/rdata/generic/amtrelay_260.c +++ b/lib/dns/rdata/generic/amtrelay_260.c @@ -409,7 +409,7 @@ digest_amtrelay(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_amtrelay); dns_rdata_toregion(rdata, ®ion); - return ((digest)(arg, ®ion)); + return ((*digest)(arg, ®ion)); } static inline bool diff --git a/lib/dns/rdata/generic/avc_258.c b/lib/dns/rdata/generic/avc_258.c index 5f3de214be4..5de398428eb 100644 --- a/lib/dns/rdata/generic/avc_258.c +++ b/lib/dns/rdata/generic/avc_258.c @@ -110,7 +110,7 @@ digest_avc(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/caa_257.c b/lib/dns/rdata/generic/caa_257.c index b443aa7d13c..767ac9654ab 100644 --- a/lib/dns/rdata/generic/caa_257.c +++ b/lib/dns/rdata/generic/caa_257.c @@ -589,7 +589,7 @@ digest_caa(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/cdnskey_60.c b/lib/dns/rdata/generic/cdnskey_60.c index 9eea7d6b8cf..c90b028ca4d 100644 --- a/lib/dns/rdata/generic/cdnskey_60.c +++ b/lib/dns/rdata/generic/cdnskey_60.c @@ -124,7 +124,7 @@ digest_cdnskey(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/cds_59.c b/lib/dns/rdata/generic/cds_59.c index 26cc70d4276..0a56aa0dcf3 100644 --- a/lib/dns/rdata/generic/cds_59.c +++ b/lib/dns/rdata/generic/cds_59.c @@ -131,7 +131,7 @@ digest_cds(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/cert_37.c b/lib/dns/rdata/generic/cert_37.c index 34f5ef7c9f7..cbe7ea36b72 100644 --- a/lib/dns/rdata/generic/cert_37.c +++ b/lib/dns/rdata/generic/cert_37.c @@ -250,7 +250,7 @@ digest_cert(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/csync_62.c b/lib/dns/rdata/generic/csync_62.c index eb41bcb640b..06048b6f515 100644 --- a/lib/dns/rdata/generic/csync_62.c +++ b/lib/dns/rdata/generic/csync_62.c @@ -228,7 +228,7 @@ digest_csync(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_csync); dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/dlv_32769.c b/lib/dns/rdata/generic/dlv_32769.c index 21ae76df86d..154605a6727 100644 --- a/lib/dns/rdata/generic/dlv_32769.c +++ b/lib/dns/rdata/generic/dlv_32769.c @@ -127,7 +127,7 @@ digest_dlv(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/dnskey_48.c b/lib/dns/rdata/generic/dnskey_48.c index ca557bf59db..906cf02f14d 100644 --- a/lib/dns/rdata/generic/dnskey_48.c +++ b/lib/dns/rdata/generic/dnskey_48.c @@ -125,7 +125,7 @@ digest_dnskey(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/doa_259.c b/lib/dns/rdata/generic/doa_259.c index 5de79296c04..c256aec213d 100644 --- a/lib/dns/rdata/generic/doa_259.c +++ b/lib/dns/rdata/generic/doa_259.c @@ -326,7 +326,7 @@ digest_doa(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/ds_43.c b/lib/dns/rdata/generic/ds_43.c index 0b91301c315..d4a47378c6e 100644 --- a/lib/dns/rdata/generic/ds_43.c +++ b/lib/dns/rdata/generic/ds_43.c @@ -350,7 +350,7 @@ digest_ds(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/eui48_108.c b/lib/dns/rdata/generic/eui48_108.c index 1a9820bbeea..657b90865ce 100644 --- a/lib/dns/rdata/generic/eui48_108.c +++ b/lib/dns/rdata/generic/eui48_108.c @@ -175,7 +175,7 @@ digest_eui48(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/eui64_109.c b/lib/dns/rdata/generic/eui64_109.c index 50220c5199d..3ecc98ff72d 100644 --- a/lib/dns/rdata/generic/eui64_109.c +++ b/lib/dns/rdata/generic/eui64_109.c @@ -178,7 +178,7 @@ digest_eui64(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/gpos_27.c b/lib/dns/rdata/generic/gpos_27.c index 8bdfbdaae1a..f8cb90ef3f9 100644 --- a/lib/dns/rdata/generic/gpos_27.c +++ b/lib/dns/rdata/generic/gpos_27.c @@ -220,7 +220,7 @@ digest_gpos(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/hinfo_13.c b/lib/dns/rdata/generic/hinfo_13.c index 98dbe1fd627..b6e1438171f 100644 --- a/lib/dns/rdata/generic/hinfo_13.c +++ b/lib/dns/rdata/generic/hinfo_13.c @@ -184,7 +184,7 @@ digest_hinfo(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/hip_55.c b/lib/dns/rdata/generic/hip_55.c index 6487c555a99..59fa0aed9ed 100644 --- a/lib/dns/rdata/generic/hip_55.c +++ b/lib/dns/rdata/generic/hip_55.c @@ -395,7 +395,7 @@ digest_hip(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_hip); dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/ipseckey_45.c b/lib/dns/rdata/generic/ipseckey_45.c index f12c6dc48c0..caf837fb0c9 100644 --- a/lib/dns/rdata/generic/ipseckey_45.c +++ b/lib/dns/rdata/generic/ipseckey_45.c @@ -454,7 +454,7 @@ digest_ipseckey(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_ipseckey); dns_rdata_toregion(rdata, ®ion); - return ((digest)(arg, ®ion)); + return ((*digest)(arg, ®ion)); } static inline bool diff --git a/lib/dns/rdata/generic/isdn_20.c b/lib/dns/rdata/generic/isdn_20.c index 3054590d7c4..277ec13cd42 100644 --- a/lib/dns/rdata/generic/isdn_20.c +++ b/lib/dns/rdata/generic/isdn_20.c @@ -211,7 +211,7 @@ digest_isdn(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/key_25.c b/lib/dns/rdata/generic/key_25.c index 1351bfdd23a..0141f85ed05 100644 --- a/lib/dns/rdata/generic/key_25.c +++ b/lib/dns/rdata/generic/key_25.c @@ -431,7 +431,7 @@ digest_key(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/keydata_65533.c b/lib/dns/rdata/generic/keydata_65533.c index ae8d438e2e8..97ef942c0ab 100644 --- a/lib/dns/rdata/generic/keydata_65533.c +++ b/lib/dns/rdata/generic/keydata_65533.c @@ -427,7 +427,7 @@ digest_keydata(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/l32_105.c b/lib/dns/rdata/generic/l32_105.c index b2f02df6468..7530841bf60 100644 --- a/lib/dns/rdata/generic/l32_105.c +++ b/lib/dns/rdata/generic/l32_105.c @@ -194,7 +194,7 @@ digest_l32(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/l64_106.c b/lib/dns/rdata/generic/l64_106.c index 6ff8fa83bd3..356d1d82f54 100644 --- a/lib/dns/rdata/generic/l64_106.c +++ b/lib/dns/rdata/generic/l64_106.c @@ -188,7 +188,7 @@ digest_l64(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/loc_29.c b/lib/dns/rdata/generic/loc_29.c index 8c8afc374c9..bdd5a7517bb 100644 --- a/lib/dns/rdata/generic/loc_29.c +++ b/lib/dns/rdata/generic/loc_29.c @@ -800,7 +800,7 @@ digest_loc(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/lp_107.c b/lib/dns/rdata/generic/lp_107.c index 5fa2709dbc0..1420362e097 100644 --- a/lib/dns/rdata/generic/lp_107.c +++ b/lib/dns/rdata/generic/lp_107.c @@ -200,11 +200,11 @@ additionaldata_lp(ARGS_ADDLDATA) { isc_region_consume(®ion, 2); dns_name_fromregion(&name, ®ion); - result = (add)(arg, &name, dns_rdatatype_l32, NULL); + result = (*add)(arg, &name, dns_rdatatype_l32, NULL); if (result != ISC_R_SUCCESS) { return (result); } - return ((add)(arg, &name, dns_rdatatype_l64, NULL)); + return ((*add)(arg, &name, dns_rdatatype_l64, NULL)); } static inline isc_result_t @@ -214,7 +214,7 @@ digest_lp(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_lp); dns_rdata_toregion(rdata, ®ion); - return ((digest)(arg, ®ion)); + return ((*digest)(arg, ®ion)); } static inline bool diff --git a/lib/dns/rdata/generic/mb_7.c b/lib/dns/rdata/generic/mb_7.c index f5a04dd2024..11c128b63b7 100644 --- a/lib/dns/rdata/generic/mb_7.c +++ b/lib/dns/rdata/generic/mb_7.c @@ -185,7 +185,7 @@ additionaldata_mb(ARGS_ADDLDATA) { dns_rdata_toregion(rdata, ®ion); dns_name_fromregion(&name, ®ion); - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t diff --git a/lib/dns/rdata/generic/md_3.c b/lib/dns/rdata/generic/md_3.c index 899607b1c87..69ccf95d687 100644 --- a/lib/dns/rdata/generic/md_3.c +++ b/lib/dns/rdata/generic/md_3.c @@ -186,7 +186,7 @@ additionaldata_md(ARGS_ADDLDATA) { dns_rdata_toregion(rdata, ®ion); dns_name_fromregion(&name, ®ion); - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t diff --git a/lib/dns/rdata/generic/mf_4.c b/lib/dns/rdata/generic/mf_4.c index f2363817917..469d35cd9ed 100644 --- a/lib/dns/rdata/generic/mf_4.c +++ b/lib/dns/rdata/generic/mf_4.c @@ -185,7 +185,7 @@ additionaldata_mf(ARGS_ADDLDATA) { dns_rdata_toregion(rdata, ®ion); dns_name_fromregion(&name, ®ion); - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t diff --git a/lib/dns/rdata/generic/mx_15.c b/lib/dns/rdata/generic/mx_15.c index a1aea668295..88fe8d6ef4a 100644 --- a/lib/dns/rdata/generic/mx_15.c +++ b/lib/dns/rdata/generic/mx_15.c @@ -284,7 +284,7 @@ additionaldata_mx(ARGS_ADDLDATA) { return (ISC_R_SUCCESS); } - result = (add)(arg, &name, dns_rdatatype_a, NULL); + result = (*add)(arg, &name, dns_rdatatype_a, NULL); if (result != ISC_R_SUCCESS) { return (result); } @@ -296,8 +296,8 @@ additionaldata_mx(ARGS_ADDLDATA) { return (ISC_R_SUCCESS); } - return ((add)(arg, dns_fixedname_name(&fixed), dns_rdatatype_tlsa, - NULL)); + return ((*add)(arg, dns_fixedname_name(&fixed), dns_rdatatype_tlsa, + NULL)); } static inline isc_result_t @@ -311,7 +311,7 @@ digest_mx(ARGS_DIGEST) { r2 = r1; isc_region_consume(&r2, 2); r1.length = 2; - RETERR((digest)(arg, &r1)); + RETERR((*digest)(arg, &r1)); dns_name_init(&name, NULL); dns_name_fromregion(&name, &r2); return (dns_name_digest(&name, digest, arg)); diff --git a/lib/dns/rdata/generic/naptr_35.c b/lib/dns/rdata/generic/naptr_35.c index 440d481d438..654eea9e9f3 100644 --- a/lib/dns/rdata/generic/naptr_35.c +++ b/lib/dns/rdata/generic/naptr_35.c @@ -640,7 +640,7 @@ additionaldata_naptr(ARGS_ADDLDATA) { dns_name_fromregion(&name, &sr); if (atype != 0) { - return ((add)(arg, &name, atype, NULL)); + return ((*add)(arg, &name, atype, NULL)); } return (ISC_R_SUCCESS); @@ -690,7 +690,7 @@ digest_naptr(ARGS_DIGEST) { * Digest the RR up to the replacement name. */ r1.length = length; - result = (digest)(arg, &r1); + result = (*digest)(arg, &r1); if (result != ISC_R_SUCCESS) { return (result); } diff --git a/lib/dns/rdata/generic/nid_104.c b/lib/dns/rdata/generic/nid_104.c index d8f9afa8155..7ff52a05bae 100644 --- a/lib/dns/rdata/generic/nid_104.c +++ b/lib/dns/rdata/generic/nid_104.c @@ -188,7 +188,7 @@ digest_nid(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/ninfo_56.c b/lib/dns/rdata/generic/ninfo_56.c index a14356c0c7f..88eb9a54f3a 100644 --- a/lib/dns/rdata/generic/ninfo_56.c +++ b/lib/dns/rdata/generic/ninfo_56.c @@ -110,7 +110,7 @@ digest_ninfo(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/ns_2.c b/lib/dns/rdata/generic/ns_2.c index 84c6b35da1f..93df8fd6e79 100644 --- a/lib/dns/rdata/generic/ns_2.c +++ b/lib/dns/rdata/generic/ns_2.c @@ -196,7 +196,7 @@ additionaldata_ns(ARGS_ADDLDATA) { dns_rdata_toregion(rdata, ®ion); dns_name_fromregion(&name, ®ion); - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t diff --git a/lib/dns/rdata/generic/nsec3_50.c b/lib/dns/rdata/generic/nsec3_50.c index 8f62a61533b..7edc8f699cb 100644 --- a/lib/dns/rdata/generic/nsec3_50.c +++ b/lib/dns/rdata/generic/nsec3_50.c @@ -376,7 +376,7 @@ digest_nsec3(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_nsec3); dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/nsec3param_51.c b/lib/dns/rdata/generic/nsec3param_51.c index db3a97413ca..44e6c8461db 100644 --- a/lib/dns/rdata/generic/nsec3param_51.c +++ b/lib/dns/rdata/generic/nsec3param_51.c @@ -286,7 +286,7 @@ digest_nsec3param(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_nsec3param); dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/nsec_47.c b/lib/dns/rdata/generic/nsec_47.c index 251268cec0f..4c4ed5b3da0 100644 --- a/lib/dns/rdata/generic/nsec_47.c +++ b/lib/dns/rdata/generic/nsec_47.c @@ -227,7 +227,7 @@ digest_nsec(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_nsec); dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/null_10.c b/lib/dns/rdata/generic/null_10.c index f13d052e249..3b3bb7c657c 100644 --- a/lib/dns/rdata/generic/null_10.c +++ b/lib/dns/rdata/generic/null_10.c @@ -151,7 +151,7 @@ digest_null(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/nxt_30.c b/lib/dns/rdata/generic/nxt_30.c index 8071996514b..eadc716756e 100644 --- a/lib/dns/rdata/generic/nxt_30.c +++ b/lib/dns/rdata/generic/nxt_30.c @@ -316,7 +316,7 @@ digest_nxt(ARGS_DIGEST) { } isc_region_consume(&r, name_length(&name)); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/openpgpkey_61.c b/lib/dns/rdata/generic/openpgpkey_61.c index 8c4bec53e47..e01b5e696fc 100644 --- a/lib/dns/rdata/generic/openpgpkey_61.c +++ b/lib/dns/rdata/generic/openpgpkey_61.c @@ -202,7 +202,7 @@ digest_openpgpkey(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/proforma.c b/lib/dns/rdata/generic/proforma.c index 1db5d9e4694..4c85bd53c42 100644 --- a/lib/dns/rdata/generic/proforma.c +++ b/lib/dns/rdata/generic/proforma.c @@ -118,7 +118,7 @@ static inline isc_result_t digest_ #(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool checkowner_ #(ARGS_CHECKOWNER) { diff --git a/lib/dns/rdata/generic/rkey_57.c b/lib/dns/rdata/generic/rkey_57.c index afa294daae3..77dad340375 100644 --- a/lib/dns/rdata/generic/rkey_57.c +++ b/lib/dns/rdata/generic/rkey_57.c @@ -121,7 +121,7 @@ digest_rkey(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/rt_21.c b/lib/dns/rdata/generic/rt_21.c index 32efc23e24e..c6278dd330d 100644 --- a/lib/dns/rdata/generic/rt_21.c +++ b/lib/dns/rdata/generic/rt_21.c @@ -248,15 +248,15 @@ additionaldata_rt(ARGS_ADDLDATA) { isc_region_consume(®ion, 2); dns_name_fromregion(&name, ®ion); - result = (add)(arg, &name, dns_rdatatype_x25, NULL); + result = (*add)(arg, &name, dns_rdatatype_x25, NULL); if (result != ISC_R_SUCCESS) { return (result); } - result = (add)(arg, &name, dns_rdatatype_isdn, NULL); + result = (*add)(arg, &name, dns_rdatatype_isdn, NULL); if (result != ISC_R_SUCCESS) { return (result); } - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t @@ -271,7 +271,7 @@ digest_rt(ARGS_DIGEST) { r2 = r1; isc_region_consume(&r2, 2); r1.length = 2; - result = (digest)(arg, &r1); + result = (*digest)(arg, &r1); if (result != ISC_R_SUCCESS) { return (result); } diff --git a/lib/dns/rdata/generic/sink_40.c b/lib/dns/rdata/generic/sink_40.c index 5d85652d9cf..a2fd801abdd 100644 --- a/lib/dns/rdata/generic/sink_40.c +++ b/lib/dns/rdata/generic/sink_40.c @@ -257,7 +257,7 @@ digest_sink(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/smimea_53.c b/lib/dns/rdata/generic/smimea_53.c index dd3894b30de..432a7870faa 100644 --- a/lib/dns/rdata/generic/smimea_53.c +++ b/lib/dns/rdata/generic/smimea_53.c @@ -117,7 +117,7 @@ digest_smimea(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/soa_6.c b/lib/dns/rdata/generic/soa_6.c index a6b687c0d4a..3cb25923357 100644 --- a/lib/dns/rdata/generic/soa_6.c +++ b/lib/dns/rdata/generic/soa_6.c @@ -389,7 +389,7 @@ digest_soa(ARGS_DIGEST) { RETERR(dns_name_digest(&name, digest, arg)); isc_region_consume(&r, name_length(&name)); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/spf_99.c b/lib/dns/rdata/generic/spf_99.c index 29d5c54fa5e..85515623aa5 100644 --- a/lib/dns/rdata/generic/spf_99.c +++ b/lib/dns/rdata/generic/spf_99.c @@ -111,7 +111,7 @@ digest_spf(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/sshfp_44.c b/lib/dns/rdata/generic/sshfp_44.c index 8c91d8fbcce..f437beadf0b 100644 --- a/lib/dns/rdata/generic/sshfp_44.c +++ b/lib/dns/rdata/generic/sshfp_44.c @@ -261,7 +261,7 @@ digest_sshfp(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/ta_32768.c b/lib/dns/rdata/generic/ta_32768.c index 69875c36ab5..ef9aff8f80a 100644 --- a/lib/dns/rdata/generic/ta_32768.c +++ b/lib/dns/rdata/generic/ta_32768.c @@ -127,7 +127,7 @@ digest_ta(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/talink_58.c b/lib/dns/rdata/generic/talink_58.c index 95e954bbff8..ce410785faf 100644 --- a/lib/dns/rdata/generic/talink_58.c +++ b/lib/dns/rdata/generic/talink_58.c @@ -223,7 +223,7 @@ digest_talink(ARGS_DIGEST) { REQUIRE(rdata->type == dns_rdatatype_talink); dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/tlsa_52.c b/lib/dns/rdata/generic/tlsa_52.c index 7ec2aaaa678..fe5aac0d8a5 100644 --- a/lib/dns/rdata/generic/tlsa_52.c +++ b/lib/dns/rdata/generic/tlsa_52.c @@ -304,7 +304,7 @@ digest_tlsa(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/txt_16.c b/lib/dns/rdata/generic/txt_16.c index b0f1c0c9aa0..381a27e0daf 100644 --- a/lib/dns/rdata/generic/txt_16.c +++ b/lib/dns/rdata/generic/txt_16.c @@ -246,7 +246,7 @@ digest_txt(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/uri_256.c b/lib/dns/rdata/generic/uri_256.c index b66b5834232..594225ffdc4 100644 --- a/lib/dns/rdata/generic/uri_256.c +++ b/lib/dns/rdata/generic/uri_256.c @@ -283,7 +283,7 @@ digest_uri(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/x25_19.c b/lib/dns/rdata/generic/x25_19.c index ec27cecae60..7e8a6ad969f 100644 --- a/lib/dns/rdata/generic/x25_19.c +++ b/lib/dns/rdata/generic/x25_19.c @@ -197,7 +197,7 @@ digest_x25(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/generic/zonemd_63.c b/lib/dns/rdata/generic/zonemd_63.c index e4dae5548e9..e11f8823ee5 100644 --- a/lib/dns/rdata/generic/zonemd_63.c +++ b/lib/dns/rdata/generic/zonemd_63.c @@ -315,7 +315,7 @@ digest_zonemd(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/hs_4/a_1.c b/lib/dns/rdata/hs_4/a_1.c index 484321594f6..ef0cab97507 100644 --- a/lib/dns/rdata/hs_4/a_1.c +++ b/lib/dns/rdata/hs_4/a_1.c @@ -197,7 +197,7 @@ digest_hs_a(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/a6_38.c b/lib/dns/rdata/in_1/a6_38.c index dd6c941276c..8c5fb3f1544 100644 --- a/lib/dns/rdata/in_1/a6_38.c +++ b/lib/dns/rdata/in_1/a6_38.c @@ -424,7 +424,7 @@ digest_in_a6(ARGS_DIGEST) { octets = 1 + 16 - prefixlen / 8; r1.length = octets; - result = (digest)(arg, &r1); + result = (*digest)(arg, &r1); if (result != ISC_R_SUCCESS) { return (result); } diff --git a/lib/dns/rdata/in_1/a_1.c b/lib/dns/rdata/in_1/a_1.c index cf544f02794..6fa606602af 100644 --- a/lib/dns/rdata/in_1/a_1.c +++ b/lib/dns/rdata/in_1/a_1.c @@ -202,7 +202,7 @@ digest_in_a(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/aaaa_28.c b/lib/dns/rdata/in_1/aaaa_28.c index 5ac6d649db1..de6406886de 100644 --- a/lib/dns/rdata/in_1/aaaa_28.c +++ b/lib/dns/rdata/in_1/aaaa_28.c @@ -214,7 +214,7 @@ digest_in_aaaa(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/apl_42.c b/lib/dns/rdata/in_1/apl_42.c index 5ba4527b830..32d173187b3 100644 --- a/lib/dns/rdata/in_1/apl_42.c +++ b/lib/dns/rdata/in_1/apl_42.c @@ -445,7 +445,7 @@ digest_in_apl(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/atma_34.c b/lib/dns/rdata/in_1/atma_34.c index 11fb2032679..b8cd241b3c3 100644 --- a/lib/dns/rdata/in_1/atma_34.c +++ b/lib/dns/rdata/in_1/atma_34.c @@ -280,7 +280,7 @@ digest_in_atma(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/dhcid_49.c b/lib/dns/rdata/in_1/dhcid_49.c index 8353e69f104..0259e26cc06 100644 --- a/lib/dns/rdata/in_1/dhcid_49.c +++ b/lib/dns/rdata/in_1/dhcid_49.c @@ -198,7 +198,7 @@ digest_in_dhcid(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/eid_31.c b/lib/dns/rdata/in_1/eid_31.c index 454ad2d570b..da8645cfa7a 100644 --- a/lib/dns/rdata/in_1/eid_31.c +++ b/lib/dns/rdata/in_1/eid_31.c @@ -187,7 +187,7 @@ digest_in_eid(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/https_65.c b/lib/dns/rdata/in_1/https_65.c index ed5bfb98ff6..3eb450abce4 100644 --- a/lib/dns/rdata/in_1/https_65.c +++ b/lib/dns/rdata/in_1/https_65.c @@ -123,7 +123,7 @@ digest_in_https(ARGS_DIGEST) { REQUIRE(rdata->rdclass == dns_rdataclass_in); dns_rdata_toregion(rdata, ®ion1); - return ((digest)(arg, ®ion1)); + return ((*digest)(arg, ®ion1)); } static inline bool diff --git a/lib/dns/rdata/in_1/kx_36.c b/lib/dns/rdata/in_1/kx_36.c index af45afb0d3f..9398dd22a80 100644 --- a/lib/dns/rdata/in_1/kx_36.c +++ b/lib/dns/rdata/in_1/kx_36.c @@ -235,7 +235,7 @@ additionaldata_in_kx(ARGS_ADDLDATA) { isc_region_consume(®ion, 2); dns_name_fromregion(&name, ®ion); - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t @@ -250,7 +250,7 @@ digest_in_kx(ARGS_DIGEST) { r2 = r1; isc_region_consume(&r2, 2); r1.length = 2; - RETERR((digest)(arg, &r1)); + RETERR((*digest)(arg, &r1)); dns_name_init(&name, NULL); dns_name_fromregion(&name, &r2); return (dns_name_digest(&name, digest, arg)); diff --git a/lib/dns/rdata/in_1/nimloc_32.c b/lib/dns/rdata/in_1/nimloc_32.c index ce4cc5f2af9..dcd17540595 100644 --- a/lib/dns/rdata/in_1/nimloc_32.c +++ b/lib/dns/rdata/in_1/nimloc_32.c @@ -187,7 +187,7 @@ digest_in_nimloc(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/nsap_22.c b/lib/dns/rdata/in_1/nsap_22.c index fa2d3d6d3b1..c54d873a555 100644 --- a/lib/dns/rdata/in_1/nsap_22.c +++ b/lib/dns/rdata/in_1/nsap_22.c @@ -222,7 +222,7 @@ digest_in_nsap(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/rdata/in_1/px_26.c b/lib/dns/rdata/in_1/px_26.c index a6423067234..b33ab93972a 100644 --- a/lib/dns/rdata/in_1/px_26.c +++ b/lib/dns/rdata/in_1/px_26.c @@ -320,7 +320,7 @@ digest_in_px(ARGS_DIGEST) { r2 = r1; isc_region_consume(&r2, 2); r1.length = 2; - result = (digest)(arg, &r1); + result = (*digest)(arg, &r1); if (result != ISC_R_SUCCESS) { return (result); } diff --git a/lib/dns/rdata/in_1/srv_33.c b/lib/dns/rdata/in_1/srv_33.c index f0e72cf3fb3..54991777bc6 100644 --- a/lib/dns/rdata/in_1/srv_33.c +++ b/lib/dns/rdata/in_1/srv_33.c @@ -326,7 +326,7 @@ additionaldata_in_srv(ARGS_ADDLDATA) { return (ISC_R_SUCCESS); } - result = (add)(arg, &name, dns_rdatatype_a, NULL); + result = (*add)(arg, &name, dns_rdatatype_a, NULL); if (result != ISC_R_SUCCESS) { return (result); } @@ -345,8 +345,8 @@ additionaldata_in_srv(ARGS_ADDLDATA) { return (ISC_R_SUCCESS); } - return ((add)(arg, dns_fixedname_name(&fixed), dns_rdatatype_tlsa, - NULL)); + return ((*add)(arg, dns_fixedname_name(&fixed), dns_rdatatype_tlsa, + NULL)); } static inline isc_result_t @@ -361,7 +361,7 @@ digest_in_srv(ARGS_DIGEST) { r2 = r1; isc_region_consume(&r2, 6); r1.length = 6; - RETERR((digest)(arg, &r1)); + RETERR((*digest)(arg, &r1)); dns_name_init(&name, NULL); dns_name_fromregion(&name, &r2); return (dns_name_digest(&name, digest, arg)); diff --git a/lib/dns/rdata/in_1/svcb_64.c b/lib/dns/rdata/in_1/svcb_64.c index 068f93619b8..305ee1d423f 100644 --- a/lib/dns/rdata/in_1/svcb_64.c +++ b/lib/dns/rdata/in_1/svcb_64.c @@ -1095,7 +1095,7 @@ generic_additionaldata_in_svcb(ARGS_ADDLDATA) { return (ISC_R_SUCCESS); } /* Only lookup address records */ - return ((add)(arg, owner, dns_rdatatype_a, NULL)); + return ((*add)(arg, owner, dns_rdatatype_a, NULL)); } /* @@ -1104,7 +1104,7 @@ generic_additionaldata_in_svcb(ARGS_ADDLDATA) { dns_rdataset_init(&rdataset); fname = dns_fixedname_initname(&fixed); do { - RETERR((add)(arg, &name, dns_rdatatype_cname, &rdataset)); + RETERR((*add)(arg, &name, dns_rdatatype_cname, &rdataset)); if (dns_rdataset_isassociated(&rdataset)) { isc_result_t result; result = dns_rdataset_first(&rdataset); @@ -1138,7 +1138,7 @@ generic_additionaldata_in_svcb(ARGS_ADDLDATA) { * Look up HTTPS/SVCB records when processing the alias form. */ if (alias) { - RETERR((add)(arg, &name, rdata->type, &rdataset)); + RETERR((*add)(arg, &name, rdata->type, &rdataset)); /* * Don't return A or AAAA if this is not the last element * in the HTTP / SVCB chain. @@ -1148,7 +1148,7 @@ generic_additionaldata_in_svcb(ARGS_ADDLDATA) { return (ISC_R_SUCCESS); } } - return ((add)(arg, &name, dns_rdatatype_a, NULL)); + return ((*add)(arg, &name, dns_rdatatype_a, NULL)); } static inline isc_result_t @@ -1167,7 +1167,7 @@ digest_in_svcb(ARGS_DIGEST) { REQUIRE(rdata->rdclass == dns_rdataclass_in); dns_rdata_toregion(rdata, ®ion1); - return ((digest)(arg, ®ion1)); + return ((*digest)(arg, ®ion1)); } static inline bool diff --git a/lib/dns/rdata/in_1/wks_11.c b/lib/dns/rdata/in_1/wks_11.c index b837c559201..ca21a95baaf 100644 --- a/lib/dns/rdata/in_1/wks_11.c +++ b/lib/dns/rdata/in_1/wks_11.c @@ -380,7 +380,7 @@ digest_in_wks(ARGS_DIGEST) { dns_rdata_toregion(rdata, &r); - return ((digest)(arg, &r)); + return ((*digest)(arg, &r)); } static inline bool diff --git a/lib/dns/zone.c b/lib/dns/zone.c index 20d31386f08..ead2b2b4ebf 100644 --- a/lib/dns/zone.c +++ b/lib/dns/zone.c @@ -2999,7 +2999,8 @@ zone_check_glue(dns_zone_t *zone, dns_db_t *db, dns_name_t *name, */ if (!dns_name_issubdomain(name, &zone->origin)) { if (zone->checkns != NULL) { - return ((zone->checkns)(zone, name, owner, NULL, NULL)); + return ((*zone->checkns)(zone, name, owner, NULL, + NULL)); } return (true); } diff --git a/lib/dns/zt.c b/lib/dns/zt.c index 0d235a21bbb..70ff0ff0efe 100644 --- a/lib/dns/zt.c +++ b/lib/dns/zt.c @@ -548,7 +548,7 @@ dns_zt_apply(dns_zt_t *zt, bool stop, isc_result_t *sub, if (result == ISC_R_SUCCESS) { zone = node->data; if (zone != NULL) { - result = (action)(zone, uap); + result = (*action)(zone, uap); } if (result != ISC_R_SUCCESS && stop) { tresult = result; diff --git a/lib/isc/heap.c b/lib/isc/heap.c index 678c462872f..1490a480162 100644 --- a/lib/isc/heap.c +++ b/lib/isc/heap.c @@ -278,6 +278,6 @@ isc_heap_foreach(isc_heap_t *heap, isc_heapaction_t action, void *uap) { REQUIRE(action != NULL); for (i = 1; i <= heap->last; i++) { - (action)(heap->array[i], uap); + (*action)(heap->array[i], uap); } } diff --git a/lib/isccc/symtab.c b/lib/isccc/symtab.c index eabbe580cb6..494c20363d9 100644 --- a/lib/isccc/symtab.c +++ b/lib/isccc/symtab.c @@ -277,7 +277,7 @@ isccc_symtab_foreach(isccc_symtab_t *symtab, isccc_symtabforeachaction_t action, for (elt = ISC_LIST_HEAD(symtab->table[i]); elt != NULL; elt = nelt) { nelt = ISC_LIST_NEXT(elt, link); - if ((action)(elt->key, elt->type, elt->value, arg)) { + if ((*action)(elt->key, elt->type, elt->value, arg)) { free_elt(symtab, i, elt); } }