]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Fix formatting
authorAlessio Podda <alessio@isc.org>
Fri, 2 Jan 2026 12:47:15 +0000 (13:47 +0100)
committerAlessio Podda <alessio@isc.org>
Mon, 2 Feb 2026 09:32:38 +0000 (10:32 +0100)
Cleanup formatting after IXFR changes.

(cherry picked from commit ad0a38209292b04db8bc6202af4cdcbe6e5a8e4a)

lib/dns/db.c
lib/dns/diff.c
lib/dns/include/dns/db.h
lib/dns/include/dns/diff.h
lib/dns/master.c
lib/dns/qpzone.c
lib/dns/rbt-zonedb.c
tests/dns/master_test.c
tests/dns/qpzone_test.c

index 878135e687e316ebdd7de15171983cae0c369bf7..eb771bee41e441027f29bb08736aced85fcec1df 100644 (file)
@@ -316,7 +316,8 @@ dns_db_endload(dns_db_t *db, dns_rdatacallbacks_t *callbacks) {
 }
 
 isc_result_t
-dns_db_beginupdate(dns_db_t *db, dns_dbversion_t *ver, dns_rdatacallbacks_t *callbacks) {
+dns_db_beginupdate(dns_db_t *db, dns_dbversion_t *ver,
+                  dns_rdatacallbacks_t *callbacks) {
        /*
         * Begin updating 'db'.
         */
index 7608ce134af3f1174880fb8c38c23f63ba1136d3..0262211c28a548fe9e52c04235b956dad5eef743 100644 (file)
@@ -253,7 +253,7 @@ update_rdataset(dns_db_t *db, dns_dbversion_t *ver, dns_name_t *name,
                options = DNS_DBADD_MERGE | DNS_DBADD_EXACT |
                          DNS_DBADD_EXACTTTL;
                CHECK(dns_db_addrdataset(db, node, ver, 0, rds, options,
-                                           &ardataset));
+                                        &ardataset));
                switch (result) {
                case ISC_R_SUCCESS:
                case DNS_R_UNCHANGED:
index 0d7f60109295d42e7459651234e3fb2ef8e2bea5..c703a601763268f3757f53d502c8e90ce900b017 100644 (file)
@@ -83,12 +83,11 @@ typedef struct dns_dbmethods {
        isc_result_t (*beginload)(dns_db_t             *db,
                                  dns_rdatacallbacks_t *callbacks);
        isc_result_t (*endload)(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
-       isc_result_t (*beginupdate)(dns_db_t           *db,
-                                   dns_dbversion_t            *ver,
+       isc_result_t (*beginupdate)(dns_db_t *db, dns_dbversion_t *ver,
                                    dns_rdatacallbacks_t *callbacks);
-       isc_result_t (*commitupdate)(dns_db_t          *db,
+       isc_result_t (*commitupdate)(dns_db_t             *db,
                                     dns_rdatacallbacks_t *callbacks);
-       isc_result_t (*abortupdate)(dns_db_t          *db,
+       isc_result_t (*abortupdate)(dns_db_t             *db,
                                    dns_rdatacallbacks_t *callbacks);
        void (*currentversion)(dns_db_t *db, dns_dbversion_t **versionp);
        isc_result_t (*newversion)(dns_db_t *db, dns_dbversion_t **versionp);
@@ -545,7 +544,8 @@ dns_db_endload(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
  */
 
 isc_result_t
-dns_db_beginupdate(dns_db_t *db, dns_dbversion_t *ver, dns_rdatacallbacks_t *callbacks);
+dns_db_beginupdate(dns_db_t *db, dns_dbversion_t *ver,
+                  dns_rdatacallbacks_t *callbacks);
 /*%<
  * Begin updating 'db'.
  *
@@ -574,7 +574,7 @@ dns_db_beginupdate(dns_db_t *db, dns_dbversion_t *ver, dns_rdatacallbacks_t *cal
 isc_result_t
 dns_db_commitupdate(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
 /*%<
- * Commit the update to 'db'. Must be safe to double-call or call after 
+ * Commit the update to 'db'. Must be safe to double-call or call after
  * dns_db_abortupdate.
  *
  * Requires:
@@ -583,11 +583,13 @@ dns_db_commitupdate(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
  *
  * \li 'callbacks' is a valid dns_rdatacallbacks_t structure.
  *
- * \li callbacks->add_private is not NULL and is a valid database update context.
+ * \li callbacks->add_private is not NULL and is a valid database update
+ * context.
  *
  * Ensures:
  *
- * \li 'callbacks' is returned to its state prior to calling dns_db_beginupdate()
+ * \li 'callbacks' is returned to its state prior to calling
+ * dns_db_beginupdate()
  *
  * Returns:
  *
@@ -600,7 +602,7 @@ dns_db_commitupdate(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
 isc_result_t
 dns_db_abortupdate(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
 /*%<
- * Abort the update to 'db'. Must be safe to double-call or call after 
+ * Abort the update to 'db'. Must be safe to double-call or call after
  * dns_db_commitupdate.
  *
  * Requires:
@@ -609,11 +611,13 @@ dns_db_abortupdate(dns_db_t *db, dns_rdatacallbacks_t *callbacks);
  *
  * \li 'callbacks' is a valid dns_rdatacallbacks_t structure.
  *
- * \li callbacks->add_private is not NULL and is a valid database update context.
+ * \li callbacks->add_private is not NULL and is a valid database update
+ * context.
  *
  * Ensures:
  *
- * \li 'callbacks' is returned to its state prior to calling dns_db_beginupdate()
+ * \li 'callbacks' is returned to its state prior to calling
+ * dns_db_beginupdate()
  *
  * Returns:
  *
index d2c8cb38c02983446ac3d784cc14946720fc54fa..db37565d67f4f97cb43abf628296b4dd1e186675 100644 (file)
@@ -60,7 +60,6 @@
  * timeexpire.
  */
 
-
 typedef struct dns_difftuple dns_difftuple_t;
 typedef ISC_LIST(dns_difftuple_t) dns_difftuplelist_t;
 
@@ -260,13 +259,14 @@ dns_diff_applysilently(const dns_diff_t *diff, dns_db_t *db,
  */
 
 typedef struct {
-       dns_db_t *db;
+       dns_db_t        *db;
        dns_dbversion_t *ver;
-       bool warn;
+       bool             warn;
 } dns_updatectx_t;
 
 isc_result_t
-dns_diff_apply_with_callbacks(const dns_diff_t *diff, dns_rdatacallbacks_t *callbacks);
+dns_diff_apply_with_callbacks(const dns_diff_t    *diff,
+                             dns_rdatacallbacks_t *callbacks);
 /*%<
  * Apply 'diff' to the database using the provided callbacks and context.
  * The context contains the database, version, and warning flag.
index f1a698d98612948de6c8952b05cf222ad0ff274d..2fed2999f8044d9d12f580a4d26c4a2ba80aa8d0 100644 (file)
@@ -2949,7 +2949,8 @@ commit(dns_rdatacallbacks_t *callbacks, dns_loadctx_t *lctx,
                        dataset.resign = resign_fromlist(this, lctx);
                }
                result = callbacks->update(callbacks->add_private, owner,
-                                       &dataset, DNS_DIFFOP_ADD DNS__DB_FILELINE);
+                                          &dataset,
+                                          DNS_DIFFOP_ADD DNS__DB_FILELINE);
                if (result == ISC_R_NOMEMORY) {
                        (*error)(callbacks, "dns_master_load: %s",
                                 isc_result_totext(result));
index efcb555888f99685728b139a4df2f63a8060fbf8..ae09403435c57f49fd37c709f5e0a5af919ba1b5 100644 (file)
@@ -108,11 +108,11 @@ typedef struct qpznode qpznode_t;
  * Qpzone-specific update context that extends dns_updatectx_t, used in IXFR.
  */
 typedef struct qpzone_updatectx {
-       dns_updatectx_t base;
-       dns_qp_t *qp;
-       dns_qp_t *nsec;
-       dns_qp_t *nsec3;
-       dns_qpread_t qpr;
+       dns_updatectx_t base;
+       dns_qp_t *qp;
+       dns_qp_t *nsec;
+       dns_qp_t *nsec3;
+       dns_qpread_t qpr;
 } qpzone_updatectx_t;
 
 typedef struct qpz_changed {
@@ -2058,8 +2058,8 @@ addwildcards(qpzonedb_t *qpdb, dns_qp_t *qp, const dns_name_t *name) {
 }
 
 static isc_result_t
-loading_addrdataset(void *arg, const dns_name_t *name,
-                   dns_rdataset_t *rdataset, dns_diffop_t op ISC_ATTR_UNUSED DNS__DB_FLARG) {
+loading_addrdataset(void *arg, const dns_name_t *name, dns_rdataset_t *rdataset,
+                   dns_diffop_t op ISC_ATTR_UNUSED DNS__DB_FLARG) {
        qpz_load_t *loadctx = arg;
        qpzonedb_t *qpdb = (qpzonedb_t *)loadctx->db;
        qpznode_t *node = NULL;
@@ -2454,7 +2454,6 @@ begin_transaction(dns_qpmulti_t *dbtree, dns_qpread_t *qprp, bool create) {
        if (create) {
                dns_qpmulti_write(dbtree, &qp);
        } else {
-
                dns_qpmulti_query(dbtree, qprp);
                qp = (dns_qp_t *)qprp;
        }
@@ -2468,14 +2467,14 @@ end_transaction(dns_qpmulti_t *dbtree, dns_qp_t *qp, bool create) {
                dns_qp_compact(qp, DNS_QPGC_MAYBE);
                dns_qpmulti_commit(dbtree, &qp);
        } else {
-               dns_qpread_t *qprp = (dns_qpread_t*) qp;
+               dns_qpread_t *qprp = (dns_qpread_t *)qp;
                dns_qpread_destroy(dbtree, qprp);
        }
 }
 
 static isc_result_t
-findnodeintree(qpzonedb_t *qpdb, dns_qp_t *qp, const dns_name_t *name, bool create,
-              bool nsec3, dns_dbnode_t **nodep DNS__DB_FLARG) {
+findnodeintree(qpzonedb_t *qpdb, dns_qp_t *qp, const dns_name_t *name,
+              bool create, bool nsec3, dns_dbnode_t **nodep DNS__DB_FLARG) {
        isc_result_t result;
        qpznode_t *node = NULL;
 
@@ -2518,10 +2517,9 @@ findnodeintree(qpzonedb_t *qpdb, dns_qp_t *qp, const dns_name_t *name, bool crea
                 */
                qpznode_erefs_increment(qpdb, node DNS__DB_FLARG_PASS);
 
-
                if (nsec3) {
                        node->nsec = DNS_DB_NSEC_NSEC3;
-               } else  {
+               } else {
                        /*
                         * Add empty non-terminal nodes to help with wildcards.
                         */
@@ -2553,7 +2551,8 @@ findnode(dns_db_t *db, const dns_name_t *name, bool create,
        dns_qpread_t qpr = { 0 };
        dns_qp_t *qp = begin_transaction(qpdb->tree, &qpr, create);
 
-       isc_result_t result =  findnodeintree(qpdb, qp, name, create, false, nodep DNS__DB_FLARG_PASS);
+       isc_result_t result = findnodeintree(qpdb, qp, name, create, false,
+                                            nodep DNS__DB_FLARG_PASS);
 
        end_transaction(qpdb->tree, qp, create);
 
@@ -2570,7 +2569,8 @@ findnsec3node(dns_db_t *db, const dns_name_t *name, bool create,
        dns_qpread_t qpr = { 0 };
        dns_qp_t *qp = begin_transaction(qpdb->nsec3, &qpr, create);
 
-       isc_result_t result =  findnodeintree(qpdb, qp, name, create, true, nodep DNS__DB_FLARG_PASS);
+       isc_result_t result = findnodeintree(qpdb, qp, name, create, true,
+                                            nodep DNS__DB_FLARG_PASS);
 
        end_transaction(qpdb->nsec3, qp, create);
 
@@ -4724,12 +4724,10 @@ createiterator(dns_db_t *db, unsigned int options,
  */
 static isc_result_t
 qpzone_addrdataset_inner(dns_db_t *db, dns_dbnode_t *dbnode,
-                  dns_dbversion_t *dbversion,
-                  isc_stdtime_t now ISC_ATTR_UNUSED, dns_rdataset_t *rdataset,
-                  unsigned int options,
-                  dns_rdataset_t *addedrdataset,
-                  dns_qp_t *nsec) {
-
+                        dns_dbversion_t *dbversion,
+                        isc_stdtime_t now ISC_ATTR_UNUSED,
+                        dns_rdataset_t *rdataset, unsigned int options,
+                        dns_rdataset_t *addedrdataset, dns_qp_t *nsec) {
        isc_result_t result;
        qpzonedb_t *qpdb = (qpzonedb_t *)db;
        qpznode_t *node = (qpznode_t *)dbnode;
@@ -4801,7 +4799,8 @@ qpzone_addrdataset_inner(dns_db_t *db, dns_dbnode_t *dbnode,
         * Add to the auxiliary NSEC tree if we're adding an NSEC record.
         */
 
-       bool is_nsec = node->nsec != DNS_DB_NSEC_HAS_NSEC && rdataset->type == dns_rdatatype_nsec;
+       bool is_nsec = node->nsec != DNS_DB_NSEC_HAS_NSEC &&
+                      rdataset->type == dns_rdatatype_nsec;
        REQUIRE(!is_nsec || nsec != NULL);
 
        /*
@@ -4857,14 +4856,14 @@ static isc_result_t
 addrdataset(dns_db_t *db, dns_dbnode_t *dbnode, dns_dbversion_t *dbversion,
            isc_stdtime_t now ISC_ATTR_UNUSED, dns_rdataset_t *rdataset,
            unsigned int options, dns_rdataset_t *addedrdataset DNS__DB_FLARG) {
-
        qpzonedb_t *qpdb = (qpzonedb_t *)db;
        qpznode_t *node = (qpznode_t *)dbnode;
        dns_qp_t *nsec = NULL;
 
        REQUIRE(VALID_QPZONE(qpdb));
 
-       bool is_nsec = node->nsec != DNS_DB_NSEC_HAS_NSEC && rdataset->type == dns_rdatatype_nsec;
+       bool is_nsec = node->nsec != DNS_DB_NSEC_HAS_NSEC &&
+                      rdataset->type == dns_rdatatype_nsec;
 
        /*
         * Add to the auxiliary NSEC tree if we're adding an NSEC record.
@@ -5312,8 +5311,9 @@ setmaxtypepername(dns_db_t *db, uint32_t value) {
  * meant to reuse the same qp transaction for multiple operations.
  */
 static isc_result_t
-qpzone_update_rdataset(qpzonedb_t *qpdb, qpz_version_t *version, qpzone_updatectx_t *ctx,
-                      dns_name_t *name, dns_rdataset_t *rds, dns_diffop_t op) {
+qpzone_update_rdataset(qpzonedb_t *qpdb, qpz_version_t *version,
+                      qpzone_updatectx_t *ctx, dns_name_t *name,
+                      dns_rdataset_t *rds, dns_diffop_t op) {
        isc_result_t result;
        unsigned int options;
        dns_rdataset_t ardataset;
@@ -5322,10 +5322,12 @@ qpzone_update_rdataset(qpzonedb_t *qpdb, qpz_version_t *version, qpzone_updatect
 
        dns_rdataset_init(&ardataset);
 
-       is_nsec3 = (rds->type == dns_rdatatype_nsec3 || rds->covers == dns_rdatatype_nsec3);
+       is_nsec3 = (rds->type == dns_rdatatype_nsec3 ||
+                   rds->covers == dns_rdatatype_nsec3);
        dns_qp_t *dbtree = is_nsec3 ? ctx->nsec3 : ctx->qp;
 
-       result = findnodeintree(qpdb, dbtree, name, true, is_nsec3, &node DNS__DB_FLARG_PASS);
+       result = findnodeintree(qpdb, dbtree, name, true, is_nsec3,
+                               &node DNS__DB_FLARG_PASS);
 
        if (result != ISC_R_SUCCESS) {
                goto cleanup;
@@ -5340,10 +5342,9 @@ qpzone_update_rdataset(qpzonedb_t *qpdb, qpz_version_t *version, qpzone_updatect
                 */
                options = DNS_DBADD_MERGE | DNS_DBADD_EXACT |
                          DNS_DBADD_EXACTTTL;
-               result = qpzone_addrdataset_inner((dns_db_t *)qpdb, node,
-                                           (dns_dbversion_t *)version, 0,
-                                           rds, options,
-                                           &ardataset, ctx->nsec DNS__DB_FLARG_PASS);
+               result = qpzone_addrdataset_inner(
+                       (dns_db_t *)qpdb, node, (dns_dbversion_t *)version, 0,
+                       rds, options, &ardataset, ctx->nsec DNS__DB_FLARG_PASS);
                switch (result) {
                case ISC_R_SUCCESS:
                case DNS_R_UNCHANGED:
@@ -5359,17 +5360,17 @@ qpzone_update_rdataset(qpzonedb_t *qpdb, qpz_version_t *version, qpzone_updatect
        case DNS_DIFFOP_DEL:
        case DNS_DIFFOP_DELRESIGN:
                options = DNS_DBSUB_EXACT | DNS_DBSUB_WANTOLD;
-               result = subtractrdataset((dns_db_t *)qpdb, node,
-                                                (dns_dbversion_t *)version,
-                                                rds, options,
-                                                &ardataset DNS__DB_FLARG_PASS);
+               result = subtractrdataset(
+                       (dns_db_t *)qpdb, node, (dns_dbversion_t *)version, rds,
+                       options, &ardataset DNS__DB_FLARG_PASS);
                break;
        default:
                UNREACHABLE();
        }
 
        bool is_resign = rds->type == dns_rdatatype_rrsig &&
-                   (op == DNS_DIFFOP_DELRESIGN || op == DNS_DIFFOP_ADDRESIGN);
+                        (op == DNS_DIFFOP_DELRESIGN ||
+                         op == DNS_DIFFOP_ADDRESIGN);
 
        if (result == ISC_R_SUCCESS && is_resign) {
                isc_stdtime_t resign;
@@ -5379,7 +5380,7 @@ qpzone_update_rdataset(qpzonedb_t *qpdb, qpz_version_t *version, qpzone_updatect
 
 cleanup:
        if (node != NULL) {
-               dns_db_detachnode((dns_db_t*) qpdb, &node);
+               dns_db_detachnode((dns_db_t *)qpdb, &node);
        }
        if (dns_rdataset_isassociated(&ardataset)) {
                dns_rdataset_disassociate(&ardataset);
@@ -5394,19 +5395,21 @@ qpzone_update_callback(void *arg, const dns_name_t *name, dns_rdataset_t *rds,
        qpzonedb_t *qpdb = (qpzonedb_t *)ctx->base.db;
        qpz_version_t *version = (qpz_version_t *)ctx->base.ver;
 
-       return qpzone_update_rdataset(qpdb, version, ctx, (dns_name_t *)name, rds, op);
+       return qpzone_update_rdataset(qpdb, version, ctx, (dns_name_t *)name,
+                                     rds, op);
 }
 
 static isc_result_t
-qpzone_beginupdate(dns_db_t *db, dns_dbversion_t *ver, dns_rdatacallbacks_t *callbacks) {
+qpzone_beginupdate(dns_db_t *db, dns_dbversion_t *ver,
+                  dns_rdatacallbacks_t *callbacks) {
        qpzonedb_t *qpdb = (qpzonedb_t *)db;
 
        REQUIRE(VALID_QPZONE(qpdb));
        REQUIRE(ver != NULL);
        REQUIRE(DNS_CALLBACK_VALID(callbacks));
 
-       qpzone_updatectx_tctx = isc_mem_get(qpdb->common.mctx, sizeof(*ctx));
-       *ctx = (qpzone_updatectx_t) {
+       qpzone_updatectx_t *ctx = isc_mem_get(qpdb->common.mctx, sizeof(*ctx));
+       *ctx = (qpzone_updatectx_t){
                .base.db = db,
                .base.ver = ver,
                .base.warn = true,
index 7541630106a35d3ab8356c1baf5c0e4432c5c287..98c1d6ba18a138d169143baa318156f6b0473aae 100644 (file)
@@ -1679,8 +1679,8 @@ done:
 }
 
 static isc_result_t
-loading_addrdataset(void *arg, const dns_name_t *name,
-                   dns_rdataset_t *rdataset, dns_diffop_t op ISC_ATTR_UNUSED DNS__DB_FLARG) {
+loading_addrdataset(void *arg, const dns_name_t *name, dns_rdataset_t *rdataset,
+                   dns_diffop_t op ISC_ATTR_UNUSED DNS__DB_FLARG) {
        rbtdb_load_t *loadctx = arg;
        dns_rbtdb_t *rbtdb = (dns_rbtdb_t *)loadctx->db;
        dns_rbtnode_t *node = NULL;
index a1779fc5b11908b1a502eaa426f5d2f74011066b..80403412ce428732fc20b3cb97760c59d166f7a5 100644 (file)
@@ -63,8 +63,8 @@ static void
 rawdata_callback(dns_zone_t *zone, dns_masterrawheader_t *header);
 
 static isc_result_t
-add_callback(void *arg, const dns_name_t *owner,
-            dns_rdataset_t *dataset, dns_diffop_t op DNS__DB_FLARG) {
+add_callback(void *arg, const dns_name_t *owner, dns_rdataset_t *dataset,
+            dns_diffop_t op DNS__DB_FLARG) {
        char buf[BIGBUFLEN];
        isc_buffer_t target;
        isc_result_t result;
index ce7b0c475b66770efe0189351cb29787b6c0320d..02790bba238ff9841e9f21df4b2673a78ed5a44b 100644 (file)
 /*
  * Macro that uses a for loop to execute a cleanup at the end of scope.
  */
-#define WITH_NEWVERSION(db, version_var, should_commit) \
-    for (dns_dbversion_t *version_var = NULL, \
-         *_tmp = ({ \
-             isc_result_t _result = dns_db_newversion(db, &version_var); \
-             assert_int_equal(_result, ISC_R_SUCCESS); \
-             (dns_dbversion_t*)1; \
-         }); \
-         _tmp != NULL; \
-         _tmp = ({ \
-             dns_db_closeversion(db, &version_var, should_commit); \
-             (dns_dbversion_t*)NULL; \
-         }))
-
+#define WITH_NEWVERSION(db, version_var, should_commit)                      \
+       for (dns_dbversion_t *version_var = NULL, *_tmp = ({                 \
+                    isc_result_t _result = dns_db_newversion(db,            \
+                                                             &version_var); \
+                    assert_int_equal(_result, ISC_R_SUCCESS);               \
+                    (dns_dbversion_t *)1;                                   \
+            });                                                             \
+            _tmp != NULL;                                                   \
+            _tmp = ({                                                       \
+                    dns_db_closeversion(db, &version_var, should_commit);   \
+                    (dns_dbversion_t *)NULL;                                \
+            }))
 
 const char *ownercase_vectors[12][2] = {
        {
@@ -115,42 +114,43 @@ const char *ownercase_vectors[12][2] = {
        }
 };
 
-static unsigned char example_org_data[] = { 7, 'e', 'x', 'a', 'm', 'p', 'l', 'e', 3, 'o', 'r', 'g', 0 };
+static unsigned char example_org_data[] = { 7,  'e', 'x', 'a', 'm', 'p', 'l',
+                                           'e', 3,   'o', 'r', 'g', 0 };
 static unsigned char example_org_offsets[] = { 0, 8, 12 };
-static dns_name_t example_org_name = DNS_NAME_INITABSOLUTE(example_org_data, example_org_offsets);
+static dns_name_t example_org_name = DNS_NAME_INITABSOLUTE(example_org_data,
+                                                          example_org_offsets);
 
 /* IPv6 test addresses */
 static unsigned char aaaa_test_data[][16] = {
        { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1 }, /* ::1 */
-       { 0x20, 0x01, 0x0d, 0xb8, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2 } /* 2001:db8::2 */
+       { 0x20, 0x01, 0x0d, 0xb8, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
+         2 } /* 2001:db8::2
+              */
 };
 
 /* RRSIG test signatures */
 static unsigned char rrsig_signature1[64] = {
-       0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08,
-       0x09, 0x0a, 0x0b, 0x0c, 0x0d, 0x0e, 0x0f, 0x10,
-       0x11, 0x12, 0x13, 0x14, 0x15, 0x16, 0x17, 0x18,
-       0x19, 0x1a, 0x1b, 0x1c, 0x1d, 0x1e, 0x1f, 0x20,
-       0x21, 0x22, 0x23, 0x24, 0x25, 0x26, 0x27, 0x28,
-       0x29, 0x2a, 0x2b, 0x2c, 0x2d, 0x2e, 0x2f, 0x30,
-       0x31, 0x32, 0x33, 0x34, 0x35, 0x36, 0x37, 0x38,
-       0x39, 0x3a, 0x3b, 0x3c, 0x3d, 0x3e, 0x3f, 0x40
+       0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x09, 0x0a, 0x0b,
+       0x0c, 0x0d, 0x0e, 0x0f, 0x10, 0x11, 0x12, 0x13, 0x14, 0x15, 0x16,
+       0x17, 0x18, 0x19, 0x1a, 0x1b, 0x1c, 0x1d, 0x1e, 0x1f, 0x20, 0x21,
+       0x22, 0x23, 0x24, 0x25, 0x26, 0x27, 0x28, 0x29, 0x2a, 0x2b, 0x2c,
+       0x2d, 0x2e, 0x2f, 0x30, 0x31, 0x32, 0x33, 0x34, 0x35, 0x36, 0x37,
+       0x38, 0x39, 0x3a, 0x3b, 0x3c, 0x3d, 0x3e, 0x3f, 0x40
 };
 
 static unsigned char rrsig_signature2[64] = {
-       0x41, 0x42, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48,
-       0x49, 0x4a, 0x4b, 0x4c, 0x4d, 0x4e, 0x4f, 0x50,
-       0x51, 0x52, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58,
-       0x59, 0x5a, 0x5b, 0x5c, 0x5d, 0x5e, 0x5f, 0x60,
-       0x61, 0x62, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68,
-       0x69, 0x6a, 0x6b, 0x6c, 0x6d, 0x6e, 0x6f, 0x70,
-       0x71, 0x72, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78,
-       0x79, 0x7a, 0x7b, 0x7c, 0x7d, 0x7e, 0x7f, 0x80
+       0x41, 0x42, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48, 0x49, 0x4a, 0x4b,
+       0x4c, 0x4d, 0x4e, 0x4f, 0x50, 0x51, 0x52, 0x53, 0x54, 0x55, 0x56,
+       0x57, 0x58, 0x59, 0x5a, 0x5b, 0x5c, 0x5d, 0x5e, 0x5f, 0x60, 0x61,
+       0x62, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68, 0x69, 0x6a, 0x6b, 0x6c,
+       0x6d, 0x6e, 0x6f, 0x70, 0x71, 0x72, 0x73, 0x74, 0x75, 0x76, 0x77,
+       0x78, 0x79, 0x7a, 0x7b, 0x7c, 0x7d, 0x7e, 0x7f, 0x80
 };
 
 /* RRSIG test structures */
 static dns_rdata_rrsig_t rrsig_test_data1 = {
-       .common = { .rdtype = dns_rdatatype_rrsig, .rdclass = dns_rdataclass_in },
+       .common = { .rdtype = dns_rdatatype_rrsig,
+                   .rdclass = dns_rdataclass_in },
        .covered = dns_rdatatype_a,
        .algorithm = DST_ALG_RSASHA256,
        .labels = 2,
@@ -164,7 +164,8 @@ static dns_rdata_rrsig_t rrsig_test_data1 = {
 };
 
 static dns_rdata_rrsig_t rrsig_test_data2 = {
-       .common = { .rdtype = dns_rdatatype_rrsig, .rdclass = dns_rdataclass_in },
+       .common = { .rdtype = dns_rdatatype_rrsig,
+                   .rdclass = dns_rdataclass_in },
        .covered = dns_rdatatype_a,
        .algorithm = DST_ALG_RSASHA256,
        .labels = 2,
@@ -287,7 +288,8 @@ apply_dns_update(dns_db_t *db, dns_dbversion_t *version, const dns_name_t *name,
        dns_rdataset_init(&rdataset);
        dns_rdatalist_tordataset(&rdatalist, &rdataset);
 
-       isc_result_t callback_result = callbacks.update(callbacks.add_private, name, &rdataset, op);
+       isc_result_t callback_result = callbacks.update(callbacks.add_private,
+                                                       name, &rdataset, op);
        assert_int_equal(result, ISC_R_SUCCESS);
 
        dns_rdataset_disassociate(&rdataset);
@@ -310,8 +312,7 @@ verify_aaaa_records(dns_db_t *db, dns_dbversion_t *version,
        dns_name_t *found_name = dns_fixedname_initname(&found_fname);
 
        /* Allocate zero-initialized found flags array */
-       found_ips = isc_mem_cget(mctx, (size_t)expected_count,
-                                sizeof(bool));
+       found_ips = isc_mem_cget(mctx, (size_t)expected_count, sizeof(bool));
 
        dns_rdataset_init(&rdataset);
 
@@ -326,7 +327,8 @@ verify_aaaa_records(dns_db_t *db, dns_dbversion_t *version,
 
        /* Iterate through all AAAA records */
        for (result = dns_rdataset_first(&rdataset); result == ISC_R_SUCCESS;
-            result = dns_rdataset_next(&rdataset)) {
+            result = dns_rdataset_next(&rdataset))
+       {
                dns_rdata_t rdata = DNS_RDATA_INIT;
                dns_rdataset_current(&rdataset, &rdata);
 
@@ -354,8 +356,7 @@ verify_aaaa_records(dns_db_t *db, dns_dbversion_t *version,
 
        dns_db_detachnode(db, &node);
        dns_rdataset_disassociate(&rdataset);
-       isc_mem_cput(mctx, found_ips, (size_t)expected_count,
-                    sizeof(bool));
+       isc_mem_cput(mctx, found_ips, (size_t)expected_count, sizeof(bool));
 }
 
 ISC_RUN_TEST_IMPL(setownercase) {
@@ -423,29 +424,38 @@ ISC_RUN_TEST_IMPL(diffop_add_sub) {
        assert_non_null(db);
 
        WITH_NEWVERSION(db, version, true) {
-               apply_dns_update(db, version, &example_org_name, dns_rdatatype_aaaa,
-                                dns_rdataclass_in, 300, aaaa_test_data[0], 16, DNS_DIFFOP_ADD);
+               apply_dns_update(db, version, &example_org_name,
+                                dns_rdatatype_aaaa, dns_rdataclass_in, 300,
+                                aaaa_test_data[0], 16, DNS_DIFFOP_ADD);
        }
 
        WITH_NEWVERSION(db, version, true) {
-               result = apply_dns_update(db, version, &example_org_name, dns_rdatatype_aaaa,
-                                dns_rdataclass_in, 300, aaaa_test_data[1], 16, DNS_DIFFOP_ADD);
+               result = apply_dns_update(db, version, &example_org_name,
+                                         dns_rdatatype_aaaa, dns_rdataclass_in,
+                                         300, aaaa_test_data[1], 16,
+                                         DNS_DIFFOP_ADD);
                assert_int_equal(result, ISC_R_SUCCESS);
 
-               verify_aaaa_records(db, version, &example_org_name, aaaa_test_data, 2, 300);
+               verify_aaaa_records(db, version, &example_org_name,
+                                   aaaa_test_data, 2, 300);
        }
 
        WITH_NEWVERSION(db, version, true) {
-               result = apply_dns_update(db, version, &example_org_name, dns_rdatatype_aaaa,
-                                dns_rdataclass_in, 300, aaaa_test_data[0], 16, DNS_DIFFOP_DEL);
+               result = apply_dns_update(db, version, &example_org_name,
+                                         dns_rdatatype_aaaa, dns_rdataclass_in,
+                                         300, aaaa_test_data[0], 16,
+                                         DNS_DIFFOP_DEL);
                assert_int_equal(result, ISC_R_SUCCESS);
 
-               verify_aaaa_records(db, version, &example_org_name, &aaaa_test_data[1], 1, 300);
+               verify_aaaa_records(db, version, &example_org_name,
+                                   &aaaa_test_data[1], 1, 300);
        }
 
        WITH_NEWVERSION(db, version, false) {
-               result = apply_dns_update(db, version, &example_org_name, dns_rdatatype_aaaa,
-                                                      dns_rdataclass_in, 600, aaaa_test_data[0], 16, DNS_DIFFOP_ADD);
+               result = apply_dns_update(db, version, &example_org_name,
+                                         dns_rdatatype_aaaa, dns_rdataclass_in,
+                                         600, aaaa_test_data[0], 16,
+                                         DNS_DIFFOP_ADD);
                assert_int_equal(result, DNS_R_NOTEXACT);
        }
 
@@ -463,11 +473,15 @@ ISC_RUN_TEST_IMPL(diffop_addresign) {
        unsigned char rrsig_data1[512], rrsig_data2[512];
 
        isc_buffer_init(&buffer1, rrsig_data1, sizeof(rrsig_data1));
-       result = dns_rdata_fromstruct(&rdata1, dns_rdataclass_in, dns_rdatatype_rrsig, &rrsig_test_data1, &buffer1);
+       result = dns_rdata_fromstruct(&rdata1, dns_rdataclass_in,
+                                     dns_rdatatype_rrsig, &rrsig_test_data1,
+                                     &buffer1);
        assert_int_equal(result, ISC_R_SUCCESS);
 
        isc_buffer_init(&buffer2, rrsig_data2, sizeof(rrsig_data2));
-       result = dns_rdata_fromstruct(&rdata2, dns_rdataclass_in, dns_rdatatype_rrsig, &rrsig_test_data2, &buffer2);
+       result = dns_rdata_fromstruct(&rdata2, dns_rdataclass_in,
+                                     dns_rdatatype_rrsig, &rrsig_test_data2,
+                                     &buffer2);
        assert_int_equal(result, ISC_R_SUCCESS);
 
        result = dns__qpzone_create(mctx, &example_org_name, dns_dbtype_zone,
@@ -476,26 +490,34 @@ ISC_RUN_TEST_IMPL(diffop_addresign) {
        assert_non_null(db);
 
        WITH_NEWVERSION(db, version, true) {
-               result = apply_dns_update(db, version, &example_org_name, dns_rdatatype_rrsig,
-                                         dns_rdataclass_in, 300, rdata1.data, rdata1.length, DNS_DIFFOP_ADDRESIGN);
+               result = apply_dns_update(db, version, &example_org_name,
+                                         dns_rdatatype_rrsig,
+                                         dns_rdataclass_in, 300, rdata1.data,
+                                         rdata1.length, DNS_DIFFOP_ADDRESIGN);
                assert_int_equal(result, ISC_R_SUCCESS);
        }
 
        WITH_NEWVERSION(db, version, true) {
-               result = apply_dns_update(db, version, &example_org_name, dns_rdatatype_rrsig,
-                                         dns_rdataclass_in, 300, rdata2.data, rdata2.length, DNS_DIFFOP_ADDRESIGN);
+               result = apply_dns_update(db, version, &example_org_name,
+                                         dns_rdatatype_rrsig,
+                                         dns_rdataclass_in, 300, rdata2.data,
+                                         rdata2.length, DNS_DIFFOP_ADDRESIGN);
                assert_int_equal(result, ISC_R_SUCCESS);
        }
 
        WITH_NEWVERSION(db, version, true) {
-               result = apply_dns_update(db, version, &example_org_name, dns_rdatatype_rrsig,
-                                         dns_rdataclass_in, 300, rdata1.data, rdata1.length, DNS_DIFFOP_DELRESIGN);
+               result = apply_dns_update(db, version, &example_org_name,
+                                         dns_rdatatype_rrsig,
+                                         dns_rdataclass_in, 300, rdata1.data,
+                                         rdata1.length, DNS_DIFFOP_DELRESIGN);
                assert_int_equal(result, ISC_R_SUCCESS);
        }
 
        WITH_NEWVERSION(db, version, true) {
-               result = apply_dns_update(db, version, &example_org_name, dns_rdatatype_rrsig,
-                                         dns_rdataclass_in, 300, rdata2.data, rdata2.length, DNS_DIFFOP_DELRESIGN);
+               result = apply_dns_update(db, version, &example_org_name,
+                                         dns_rdatatype_rrsig,
+                                         dns_rdataclass_in, 300, rdata2.data,
+                                         rdata2.length, DNS_DIFFOP_DELRESIGN);
                assert_int_equal(result, DNS_R_NXRRSET);
        }