]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Remove dns_rdataset_equals as unneeded
authorMark Andrews <marka@isc.org>
Thu, 4 Sep 2025 00:02:43 +0000 (10:02 +1000)
committerMark Andrews <marka@isc.org>
Wed, 10 Sep 2025 22:27:50 +0000 (08:27 +1000)
(cherry picked from commit 7d765a35141882b87ffb2b41ddc3500bac8595f0)

lib/dns/include/dns/rdataset.h
lib/dns/keytable.c
lib/dns/ncache.c
lib/dns/rbtdb.c
lib/dns/rdatalist.c
lib/dns/rdataset.c
lib/dns/sdb.c
lib/dns/sdlz.c

index fec02474cf84961579cff224235aa753c33eb521..caa49c5845a115144c725aa408d8cdbaae7e080e 100644 (file)
@@ -86,8 +86,6 @@ typedef struct dns_rdatasetmethods {
        void (*getownercase)(const dns_rdataset_t *rdataset, dns_name_t *name);
        isc_result_t (*addglue)(dns_rdataset_t  *rdataset,
                                dns_dbversion_t *version, dns_message_t *msg);
-       bool (*equals)(const dns_rdataset_t *rdataset1,
-                      const dns_rdataset_t *rdataset2);
 } dns_rdatasetmethods_t;
 
 #define DNS_RDATASET_MAGIC     ISC_MAGIC('D', 'N', 'S', 'R')
@@ -631,14 +629,4 @@ dns_trust_totext(dns_trust_t trust);
  * Display trust in textual form.
  */
 
-bool
-dns_rdataset_equals(const dns_rdataset_t *rdataset1,
-                   const dns_rdataset_t *rdataset2);
-/*%<
- * Returns true if the rdata in the rdataset is equal.
- *
- * Requires:
- * \li 'rdataset1' is a valid rdataset.
- * \li 'rdataset2' is a valid rdataset.
- */
 ISC_LANG_ENDDECLS
index 215afa340b42be205f5cf31739fbfe9b540b1446..bab4db0a75038d51b9f214279e7deef451de3aa3 100644 (file)
@@ -91,7 +91,6 @@ static dns_rdatasetmethods_t methods = {
        NULL,
        NULL,
        NULL, /* addglue */
-       NULL, /* equals */
 };
 
 static void
index 4018aa97382cb0029fb5958ca7ae24d89a9f36db..c99c2665aeca894a3cf18bf834489007a3d802f1 100644 (file)
@@ -525,7 +525,6 @@ static dns_rdatasetmethods_t rdataset_methods = {
        NULL,              /* setownercase */
        NULL,              /* getownercase */
        NULL,              /* addglue */
-       NULL,              /* equals */
 };
 
 isc_result_t
index 8fedf2cff9f11de38736aa738027f7637409ad49..82cb74db260575224c3f0c00d70681d2c7d9a212 100644 (file)
@@ -610,9 +610,6 @@ rdataset_getownercase(const dns_rdataset_t *rdataset, dns_name_t *name);
 static isc_result_t
 rdataset_addglue(dns_rdataset_t *rdataset, dns_dbversion_t *version,
                 dns_message_t *msg);
-static bool
-rdataset_equals(const dns_rdataset_t *rdataset1,
-               const dns_rdataset_t *rdataset2);
 static void
 free_gluetable(rbtdb_version_t *version);
 static isc_result_t
@@ -633,8 +630,7 @@ static dns_rdatasetmethods_t rdataset_methods = { rdataset_disassociate,
                                                  rdataset_clearprefetch,
                                                  rdataset_setownercase,
                                                  rdataset_getownercase,
-                                                 rdataset_addglue,
-                                                 rdataset_equals };
+                                                 rdataset_addglue };
 
 static dns_rdatasetmethods_t slab_methods = {
        rdataset_disassociate,
@@ -653,7 +649,6 @@ static dns_rdatasetmethods_t slab_methods = {
        NULL, /* setownercase */
        NULL, /* getownercase */
        NULL, /* addglue */
-       NULL, /* equals */
 };
 
 static void
@@ -10407,23 +10402,6 @@ no_glue:
        /* UNREACHABLE */
 }
 
-static bool
-rdataset_equals(const dns_rdataset_t *rdataset1,
-               const dns_rdataset_t *rdataset2) {
-       if (rdataset1->rdclass != rdataset2->rdclass ||
-           rdataset1->type != rdataset2->type)
-       {
-               return false;
-       }
-
-       uint8_t *header1 = (uint8_t *)rdataset1->private3 -
-                          sizeof(rdatasetheader_t);
-       uint8_t *header2 = (uint8_t *)rdataset2->private3 -
-                          sizeof(rdatasetheader_t);
-       return dns_rdataslab_equalx(header1, header2, sizeof(rdatasetheader_t),
-                                   rdataset1->rdclass, rdataset2->type);
-}
-
 /*%
  * Routines for LRU-based cache management.
  */
index 5cccea514983068d9d39690ae20861836fddaef5..98036f9cb322c853d7b64391cdfdcb2411998f4d 100644 (file)
@@ -43,7 +43,6 @@ static dns_rdatasetmethods_t methods = {
        isc__rdatalist_setownercase,
        isc__rdatalist_getownercase,
        NULL, /* addglue */
-       NULL, /* equals */
 };
 
 void
index 67508716d7b4187445062bc6dcaf9964fa64dde7..e190156fa7300e93e4cdb203360c0f04b510f179 100644 (file)
@@ -197,7 +197,6 @@ static dns_rdatasetmethods_t question_methods = {
        NULL, /* setownercase */
        NULL, /* getownercase */
        NULL, /* addglue */
-       NULL, /* equals */
 };
 
 void
@@ -753,18 +752,3 @@ dns_rdataset_addglue(dns_rdataset_t *rdataset, dns_dbversion_t *version,
 
        return (rdataset->methods->addglue)(rdataset, version, msg);
 }
-
-bool
-dns_rdataset_equals(const dns_rdataset_t *rdataset1,
-                   const dns_rdataset_t *rdataset2) {
-       REQUIRE(DNS_RDATASET_VALID(rdataset1));
-       REQUIRE(DNS_RDATASET_VALID(rdataset2));
-
-       if (rdataset1->methods->equals != NULL &&
-           rdataset1->methods->equals == rdataset2->methods->equals)
-       {
-               return (rdataset1->methods->equals)(rdataset1, rdataset2);
-       }
-
-       return false;
-}
index 40fe6f7ab2b1f8c4242a61e6da0ccf3709030be3..b29931e6626861faa593a0a31e710fa11ebffa58 100644 (file)
@@ -1440,7 +1440,6 @@ static dns_rdatasetmethods_t sdb_rdataset_methods = {
        NULL, /* setownercase */
        NULL, /* getownercase */
        NULL, /* addglue */
-       NULL, /* equals */
 };
 
 static void
index ebd7fdd50b66f35b6d7bf445d8f7b63a8732d5ae..259d7687970c4216aae0d76368a4b8be42cefc6e 100644 (file)
@@ -1452,7 +1452,6 @@ static dns_rdatasetmethods_t rdataset_methods = {
        NULL, /* setownercase */
        NULL, /* getownercase */
        NULL, /* addglue */
-       NULL, /* equals */
 };
 
 static void