]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
libdns refactoring: get rid of multiple versions of dns_xfrin_create, dst_key_generat...
authorWitold Kręcicki <wpk@isc.org>
Wed, 4 Apr 2018 07:44:50 +0000 (09:44 +0200)
committerWitold Kręcicki <wpk@isc.org>
Fri, 6 Apr 2018 06:04:41 +0000 (08:04 +0200)
37 files changed:
bin/confgen/keygen.c
bin/dig/dighost.c
bin/dnssec/dnssec-cds.c
bin/dnssec/dnssec-dsfromkey.c
bin/dnssec/dnssec-importkey.c
bin/dnssec/dnssec-keyfromlabel.c
bin/dnssec/dnssec-keygen.c
bin/dnssec/dnssec-revoke.c
bin/dnssec/dnssec-settime.c
bin/dnssec/dnssec-signzone.c
bin/dnssec/dnssec-verify.c
bin/named/server.c
bin/nsupdate/nsupdate.c
bin/tests/makejournal.c
bin/tests/optional/dst_test.c
bin/tests/optional/gsstest.c
bin/tests/optional/sig0_test.c
bin/tests/system/pipelined/pipequeries.c
bin/tests/system/rsabigexponent/bigkey.c
bin/tests/system/tkey/keycreate.c
bin/tests/system/tkey/keydelete.c
bin/tools/mdig.c
lib/dns/dnssec.c
lib/dns/dst_api.c
lib/dns/include/dns/xfrin.h
lib/dns/include/dst/dst.h
lib/dns/lib.c
lib/dns/tests/dnstest.c
lib/dns/tests/dst_test.c
lib/dns/tests/dstrandom_test.c
lib/dns/tests/rsa_test.c
lib/dns/tests/tsig_test.c
lib/dns/tsig.c
lib/dns/win32/libdns.def.in
lib/dns/xfrin.c
lib/dns/zone.c
lib/ns/tests/nstest.c

index 7391a3e29047f089789f0015e46f695776689eb3..21e59631ae3e59c239a7dcac5358e8a0155cbdec 100644 (file)
@@ -170,12 +170,12 @@ generate_key(isc_mem_t *mctx, const char *randomfile, dns_secalg_t alg,
 
        entropy_flags = ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY;
 
-       DO("initialize dst library", dst_lib_init(mctx, ectx, entropy_flags));
+       DO("initialize dst library", dst_lib_init(mctx, ectx, NULL, entropy_flags));
 
        DO("generate key", dst_key_generate(dns_rootname, alg,
-                                           keysize, 0, 0,
-                                           DNS_KEYPROTO_ANY,
-                                           dns_rdataclass_in, mctx, &key));
+                                           keysize, 0, 0, DNS_KEYPROTO_ANY,
+                                           dns_rdataclass_in, mctx, &key,
+                                           NULL));
 
        isc_buffer_init(&key_rawbuffer, &key_rawsecret, sizeof(key_rawsecret));
 
index 8d865327f874fd93a8d5f33ef094d4da991c0f1f..2347c14ded46f8a5c4ebf5d5180d8da9a6a1937a 100644 (file)
@@ -1400,7 +1400,7 @@ setup_libs(void) {
        result = isc_entropy_create(mctx, &entp);
        check_result(result, "isc_entropy_create");
 
-       result = dst_lib_init(mctx, entp, 0);
+       result = dst_lib_init(mctx, entp, NULL, 0);
        check_result(result, "dst_lib_init");
        is_dst_up = ISC_TRUE;
 
index c7ab16ace03d17172bb341480a9b93a9a3f05f29..a08fdc66dcc7b7e19e395e4db0db739692d36877 100644 (file)
@@ -1237,7 +1237,7 @@ main(int argc, char *argv[]) {
        if (ectx == NULL) {
                setup_entropy(mctx, NULL, &ectx);
        }
-       result = dst_lib_init(mctx, ectx,
+       result = dst_lib_init(mctx, ectx, NULL,
                              ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
        if (result != ISC_R_SUCCESS) {
                fatal("could not initialize dst: %s",
index 383bee36be05c1568ccbb7e88f3c0ae53671f9a7..051b6b5bba1eb329baf94cd9e850dbf0e4ca1c78 100644 (file)
@@ -479,7 +479,7 @@ main(int argc, char **argv) {
 
        if (ectx == NULL)
                setup_entropy(mctx, NULL, &ectx);
-       result = dst_lib_init(mctx, ectx,
+       result = dst_lib_init(mctx, ectx, NULL,
                              ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
        if (result != ISC_R_SUCCESS)
                fatal("could not initialize dst: %s",
index fa8788e2366262c86a3214f3654ec5a971e03d20..e96d4fe50bc8a3d6a3c0c7b4259e4e17387b3385 100644 (file)
@@ -408,7 +408,7 @@ main(int argc, char **argv) {
 
        if (ectx == NULL)
                setup_entropy(mctx, NULL, &ectx);
-       result = dst_lib_init(mctx, ectx,
+       result = dst_lib_init(mctx, ectx, NULL,
                              ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
        if (result != ISC_R_SUCCESS)
                fatal("could not initialize dst: %s",
index 69cece8bfa29629b327c0fe2d73411d95eed7c41..8e8797146d6018930552505708abf3282cb3eeb6 100644 (file)
@@ -349,8 +349,8 @@ main(int argc, char **argv) {
 
        if (ectx == NULL)
                setup_entropy(mctx, NULL, &ectx);
-       ret = dst_lib_init2(mctx, ectx, engine,
-                           ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
+       ret = dst_lib_init(mctx, ectx, engine,
+                          ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
        if (ret != ISC_R_SUCCESS)
                fatal("could not initialize dst: %s",
                      isc_result_totext(ret));
index cc8b4898f8c850904df8e570671eddc3d8a02088..ba20c32bf20d541af46f3b9fe524344556a5c4c9 100644 (file)
@@ -496,8 +496,8 @@ main(int argc, char **argv) {
 
        if (ectx == NULL)
                setup_entropy(mctx, NULL, &ectx);
-       ret = dst_lib_init2(mctx, ectx, engine,
-                           ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
+       ret = dst_lib_init(mctx, ectx, engine,
+                          ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
        if (ret != ISC_R_SUCCESS)
                fatal("could not initialize dst: %s",
                      isc_result_totext(ret));
@@ -862,15 +862,15 @@ main(int argc, char **argv) {
 
                if (!quiet && show_progress) {
                        fprintf(stderr, "Generating key pair.");
-                       ret = dst_key_generate2(name, alg, size, param, flags,
-                                               protocol, rdclass, mctx, &key,
-                                               &progress);
+                       ret = dst_key_generate(name, alg, size, param, flags,
+                                              protocol, rdclass, mctx, &key,
+                                              &progress);
                        putc('\n', stderr);
                        fflush(stderr);
                } else {
-                       ret = dst_key_generate2(name, alg, size, param, flags,
-                                               protocol, rdclass, mctx, &key,
-                                               NULL);
+                       ret = dst_key_generate(name, alg, size, param, flags,
+                                              protocol, rdclass, mctx, &key,
+                                              NULL);
                }
 
                isc_entropy_stopcallbacksources(ectx);
index 1d7ae8f275041ebd64b26ebaa36a88d01a224646..3363c1d59b54cdd4ed68e5a74b15f4a58a0c1a5f 100644 (file)
@@ -182,8 +182,8 @@ main(int argc, char **argv) {
 
        if (ectx == NULL)
                setup_entropy(mctx, NULL, &ectx);
-       result = dst_lib_init2(mctx, ectx, engine,
-                              ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
+       result = dst_lib_init(mctx, ectx, engine,
+                             ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
        if (result != ISC_R_SUCCESS)
                fatal("Could not initialize dst: %s",
                      isc_result_totext(result));
index e82a08e5590a95d12922141259fe6068cc593afc..221152361f106cc7b76a7c9d2c7aa102d2355615 100644 (file)
@@ -380,8 +380,8 @@ main(int argc, char **argv) {
 
        if (ectx == NULL)
                setup_entropy(mctx, NULL, &ectx);
-       result = dst_lib_init2(mctx, ectx, engine,
-                              ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
+       result = dst_lib_init(mctx, ectx, engine,
+                             ISC_ENTROPY_BLOCKING | ISC_ENTROPY_GOODONLY);
        if (result != ISC_R_SUCCESS)
                fatal("Could not initialize dst: %s",
                      isc_result_totext(result));
index 7e504c2bcd599ff099206db70646388a86bbb9d5..c5e21ef8d6badd79465839be53537bbc1c17ac6b 100644 (file)
@@ -3505,7 +3505,7 @@ main(int argc, char *argv[]) {
        if (!pseudorandom)
                eflags |= ISC_ENTROPY_GOODONLY;
 
-       result = dst_lib_init2(mctx, ectx, engine, eflags);
+       result = dst_lib_init(mctx, ectx, engine, eflags);
        if (result != ISC_R_SUCCESS)
                fatal("could not initialize dst: %s",
                      isc_result_totext(result));
index 54acb7a57f6bc0f9840e741510219c321c36431f..21b157cd4a76f53742880da89391e302cb19d45b 100644 (file)
@@ -281,7 +281,7 @@ main(int argc, char *argv[]) {
        if (ectx == NULL)
                setup_entropy(mctx, NULL, &ectx);
 
-       result = dst_lib_init2(mctx, ectx, engine, ISC_ENTROPY_BLOCKING);
+       result = dst_lib_init(mctx, ectx, engine, ISC_ENTROPY_BLOCKING);
        if (result != ISC_R_SUCCESS)
                fatal("could not initialize dst: %s",
                      isc_result_totext(result));
index d42c001785871b70b2f39f645996c24b62643f44..7511a680f254fcabcae7763afdce709c617e6e13 100644 (file)
@@ -6820,7 +6820,7 @@ generate_session_key(const char *filename, const char *keynamestr,
        /* generate key */
        result = dst_key_generate(keyname, algtype, bits, 1, 0,
                                  DNS_KEYPROTO_ANY, dns_rdataclass_in,
-                                 mctx, &key);
+                                 mctx, &key, NULL);
        if (result != ISC_R_SUCCESS)
                return (result);
 
@@ -9424,8 +9424,8 @@ named_server_create(isc_mem_t *mctx, named_server_t **serverp) {
        server->in_roothints = NULL;
 
        /* Must be first. */
-       CHECKFATAL(dst_lib_init2(named_g_mctx, named_g_entropy,
-                                named_g_engine, ISC_ENTROPY_GOODONLY),
+       CHECKFATAL(dst_lib_init(named_g_mctx, named_g_entropy,
+                               named_g_engine, ISC_ENTROPY_GOODONLY),
                   "initializing DST");
 
        CHECKFATAL(dns_rootns_create(mctx, dns_rdataclass_in, NULL,
index 5baf409313f2249e8a440c0c983d178c122d462f..4c416ab88062ca0e2d2cdc3e9d0d9c71382baaf0 100644 (file)
@@ -989,7 +989,7 @@ setup_system(void) {
        result = isc_task_onshutdown(global_task, shutdown_program, NULL);
        check_result(result, "isc_task_onshutdown");
 
-       result = dst_lib_init(gmctx, entropy, 0);
+       result = dst_lib_init(gmctx, entropy, NULL, 0);
        check_result(result, "dst_lib_init");
        is_dst_up = ISC_TRUE;
 
index 03481b9ce2aa43df9681b8d149a93fff518fd175..36cd32b3321a768830148318619719aafd8324d5 100644 (file)
@@ -101,7 +101,7 @@ main(int argc, char **argv) {
        CHECK(isc_mem_create(0, 0, &mctx));
        CHECK(isc_entropy_create(mctx, &ectx));
 
-       CHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_BLOCKING));
+       CHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_BLOCKING));
        dst_active = ISC_TRUE;
 
        CHECK(isc_log_create(mctx, &lctx, &logconfig));
index f2b1bc15bf8a1ff06d0778cfce886b3a3b86a092..30f071e4f1a8704f0a824311d5e60d1e004deb06 100644 (file)
@@ -52,8 +52,8 @@ use(dst_key_t *key, isc_mem_t *mctx) {
        isc_buffer_add(&databuf, strlen(data));
        isc_buffer_usedregion(&databuf, &datareg);
 
-       ret = dst_context_create3(key, mctx,
-                                 DNS_LOGCATEGORY_GENERAL, ISC_TRUE, &ctx);
+       ret = dst_context_create(key, mctx,
+                                DNS_LOGCATEGORY_GENERAL, ISC_TRUE, 0, &ctx);
        if (ret != ISC_R_SUCCESS) {
                printf("contextcreate(%u) returned: %s\n", dst_key_alg(key),
                       isc_result_totext(ret));
@@ -73,8 +73,8 @@ use(dst_key_t *key, isc_mem_t *mctx) {
 
        isc_buffer_forward(&sigbuf, 1);
        isc_buffer_remainingregion(&sigbuf, &sigreg);
-       ret = dst_context_create3(key, mctx,
-                                 DNS_LOGCATEGORY_GENERAL, ISC_FALSE, &ctx);
+       ret = dst_context_create(key, mctx,
+                                DNS_LOGCATEGORY_GENERAL, ISC_FALSE, 0, &ctx);
        if (ret != ISC_R_SUCCESS) {
                printf("contextcreate(%u) returned: %s\n", dst_key_alg(key),
                       isc_result_totext(ret));
@@ -214,7 +214,7 @@ generate(int alg, isc_mem_t *mctx) {
        dst_key_t *key = NULL;
 
        ret = dst_key_generate(dns_rootname, alg, 512, 0, 0, 0,
-                              dns_rdataclass_in, mctx, &key);
+                              dns_rdataclass_in, mctx, &key, NULL);
        printf("generate(%d) returned: %s\n", alg, isc_result_totext(ret));
        if (ret != ISC_R_SUCCESS)
                return;
@@ -254,7 +254,8 @@ main(void) {
        result = isc_entropy_createfilesource(ectx, "randomfile");
        if (result != ISC_R_SUCCESS)
                return (1);
-       dst_lib_init(mctx, ectx, ISC_ENTROPY_BLOCKING|ISC_ENTROPY_GOODONLY);
+       dst_lib_init(mctx, ectx, NULL,
+                    ISC_ENTROPY_BLOCKING|ISC_ENTROPY_GOODONLY);
 
        dns_fixedname_init(&fname);
        name = dns_fixedname_name(&fname);
index 7b4ddddf6b6237a2bddbfd8ccee6ed9a858301a4..74ff400808cc073e38b46b45357ac57b5ee1704d 100644 (file)
@@ -475,7 +475,7 @@ main(int argc, char *argv[]) {
        RUNCHECK(isc_entropy_create(mctx, &ectx));
        RUNCHECK(isc_entropy_createfilesource(ectx, "/dev/urandom"));
 
-       RUNCHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_GOODONLY));
+       RUNCHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_GOODONLY));
 
        taskmgr = NULL;
        RUNCHECK(isc_taskmgr_create(mctx, 1, 0, &taskmgr));
index b64e1ada2d8cfef1f0c32a7b48dc6a78fb831cf1..a8db222ba43c23bace50d90cbcfe3b8c69ca3c05 100644 (file)
@@ -225,7 +225,7 @@ main(int argc, char *argv[]) {
        }
 
        RUNTIME_CHECK(isc_entropy_create(mctx, &ent) == ISC_R_SUCCESS);
-       RUNTIME_CHECK(dst_lib_init(mctx, ent, 0) == ISC_R_SUCCESS);
+       RUNTIME_CHECK(dst_lib_init(mctx, ent, NULL, 0) == ISC_R_SUCCESS);
 
        dns_result_register();
        dst_result_register();
index c6801d32be74c1e224292c6fcb8273ef05fb72ee..0a042fbcf5cc61603da62fdc60f3406aba8170ff 100644 (file)
@@ -286,7 +286,7 @@ main(int argc, char *argv[]) {
        if (randomfile != NULL)
                RUNCHECK(isc_entropy_createfilesource(ectx, randomfile));
 
-       RUNCHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_GOODONLY));
+       RUNCHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_GOODONLY));
 
        taskmgr = NULL;
        RUNCHECK(isc_taskmgr_create(mctx, 1, 0, &taskmgr));
index ba87fbeca2de89aeb9649cc8ce0466755037b87d..0100c39f0af5260d38cc12cba2aceb27732e0d2a 100644 (file)
@@ -192,7 +192,7 @@ main(int argc, char **argv) {
                                        "../random.data",
                                        ISC_ENTROPY_KEYBOARDNO),
              "isc_entropy_usebestsource(\"../random.data\")");
-       CHECK(dst_lib_init2(mctx, ectx, NULL, 0), "dst_lib_init2()");
+       CHECK(dst_lib_init(mctx, ectx, NULL, 0), "dst_lib_init()");
        CHECK(isc_log_create(mctx, &log_, &logconfig), "isc_log_create()");
        isc_log_setcontext(log_);
        dns_log_init(log_);
index ea41bc3bdf5402ca7634004643eca76f02a37706..4bbc721aa176fda1b455fab1fb0f546451702c1a 100644 (file)
@@ -264,7 +264,7 @@ main(int argc, char *argv[]) {
        logconfig = NULL;
        RUNCHECK(isc_log_create(mctx, &log, &logconfig));
 
-       RUNCHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_GOODONLY));
+       RUNCHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_GOODONLY));
 
        taskmgr = NULL;
        RUNCHECK(isc_taskmgr_create(mctx, 1, 0, &taskmgr));
index 931e77f4969a0853376b254fe3279be89ddac143..b49c1711fb45201bcd64deed967e515ea2be63bb 100644 (file)
@@ -195,7 +195,7 @@ main(int argc, char **argv) {
        logconfig = NULL;
        RUNCHECK(isc_log_create(mctx, &log, &logconfig));
 
-       RUNCHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_GOODONLY));
+       RUNCHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_GOODONLY));
 
        taskmgr = NULL;
        RUNCHECK(isc_taskmgr_create(mctx, 1, 0, &taskmgr));
index e615243ca9daaa5078449d83c95fdade15d251d0..82bb5c2d58c9d436f8ffa32e3d09b9c21dbddb82 100644 (file)
@@ -1934,7 +1934,7 @@ main(int argc, char *argv[]) {
 
        ectx = NULL;
        RUNCHECK(isc_entropy_create(mctx, &ectx));
-       RUNCHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_GOODONLY));
+       RUNCHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_GOODONLY));
        RUNCHECK(isc_entropy_getdata(ectx, cookie_secret,
                                     sizeof(cookie_secret), NULL, 0));
 
index dd1e75bdef2c93b34d0d683844326544ec4b6271..6628177f6319f1b6f20d86120b2d0afda55d6591 100644 (file)
@@ -268,8 +268,8 @@ dns_dnssec_sign(const dns_name_t *name, dns_rdataset_t *set, dst_key_t *key,
        if (ret != ISC_R_SUCCESS)
                goto cleanup_databuf;
 
-       ret = dst_context_create3(key, mctx,
-                                 DNS_LOGCATEGORY_DNSSEC, ISC_TRUE, &ctx);
+       ret = dst_context_create(key, mctx,
+                                DNS_LOGCATEGORY_DNSSEC, ISC_TRUE, 0, &ctx);
        if (ret != ISC_R_SUCCESS)
                goto cleanup_databuf;
 
@@ -455,8 +455,8 @@ dns_dnssec_verify(const dns_name_t *name, dns_rdataset_t *set, dst_key_t *key,
        }
 
  again:
-       ret = dst_context_create4(key, mctx, DNS_LOGCATEGORY_DNSSEC,
-                                 ISC_FALSE, maxbits, &ctx);
+       ret = dst_context_create(key, mctx, DNS_LOGCATEGORY_DNSSEC,
+                                ISC_FALSE, maxbits, &ctx);
        if (ret != ISC_R_SUCCESS)
                goto cleanup_struct;
 
@@ -919,8 +919,8 @@ dns_dnssec_signmessage(dns_message_t *msg, dst_key_t *key) {
 
        isc_buffer_init(&databuf, data, sizeof(data));
 
-       RETERR(dst_context_create3(key, mctx,
-                                  DNS_LOGCATEGORY_DNSSEC, ISC_TRUE, &ctx));
+       RETERR(dst_context_create(key, mctx,
+                                 DNS_LOGCATEGORY_DNSSEC, ISC_TRUE, 0, &ctx));
 
        /*
         * Digest the fields of the SIG - we can cheat and use
@@ -1068,8 +1068,8 @@ dns_dnssec_verifymessage(isc_buffer_t *source, dns_message_t *msg,
                goto failure;
        }
 
-       RETERR(dst_context_create3(key, mctx,
-                                  DNS_LOGCATEGORY_DNSSEC, ISC_FALSE, &ctx));
+       RETERR(dst_context_create(key, mctx,
+                                 DNS_LOGCATEGORY_DNSSEC, ISC_FALSE, 0, &ctx));
 
        /*
         * Digest the SIG(0) record, except for the signature.
index eb304157ec6b97951dd598dfbe31bf24ad38d2f3..97340bcad2ba3be1d836561c8f80b88747c6eef8 100644 (file)
@@ -148,13 +148,8 @@ default_memfree(void *arg, void *ptr) {
 #endif
 
 isc_result_t
-dst_lib_init(isc_mem_t *mctx, isc_entropy_t *ectx, unsigned int eflags) {
-       return (dst_lib_init2(mctx, ectx, NULL, eflags));
-}
-
-isc_result_t
-dst_lib_init2(isc_mem_t *mctx, isc_entropy_t *ectx,
-             const char *engine, unsigned int eflags) {
+dst_lib_init(isc_mem_t *mctx, isc_entropy_t *ectx,
+            const char *engine, unsigned int eflags) {
        isc_result_t result;
 
        REQUIRE(mctx != NULL);
@@ -339,31 +334,9 @@ dst_ds_digest_supported(unsigned int digest_type) {
 }
 
 isc_result_t
-dst_context_create(dst_key_t *key, isc_mem_t *mctx, dst_context_t **dctxp) {
-       return (dst_context_create4(key, mctx, DNS_LOGCATEGORY_GENERAL,
-                                   ISC_TRUE, 0, dctxp));
-}
-
-isc_result_t
-dst_context_create2(dst_key_t *key, isc_mem_t *mctx,
-                   isc_logcategory_t *category, dst_context_t **dctxp)
-{
-       return (dst_context_create4(key, mctx, category, ISC_TRUE, 0, dctxp));
-}
-
-isc_result_t
-dst_context_create3(dst_key_t *key, isc_mem_t *mctx,
-                   isc_logcategory_t *category, isc_boolean_t useforsigning,
-                   dst_context_t **dctxp)
-{
-       return (dst_context_create4(key, mctx, category,
-                                   useforsigning, 0, dctxp));
-}
-
-isc_result_t
-dst_context_create4(dst_key_t *key, isc_mem_t *mctx,
-                   isc_logcategory_t *category, isc_boolean_t useforsigning,
-                   int maxbits, dst_context_t **dctxp)
+dst_context_create(dst_key_t *key, isc_mem_t *mctx,
+                  isc_logcategory_t *category, isc_boolean_t useforsigning,
+                  int maxbits, dst_context_t **dctxp)
 {
        dst_context_t *dctx;
        isc_result_t result;
@@ -983,19 +956,8 @@ dst_key_generate(const dns_name_t *name, unsigned int alg,
                 unsigned int bits, unsigned int param,
                 unsigned int flags, unsigned int protocol,
                 dns_rdataclass_t rdclass,
-                isc_mem_t *mctx, dst_key_t **keyp)
-{
-       return (dst_key_generate2(name, alg, bits, param, flags, protocol,
-                                 rdclass, mctx, keyp, NULL));
-}
-
-isc_result_t
-dst_key_generate2(const dns_name_t *name, unsigned int alg,
-                 unsigned int bits, unsigned int param,
-                 unsigned int flags, unsigned int protocol,
-                 dns_rdataclass_t rdclass,
-                 isc_mem_t *mctx, dst_key_t **keyp,
-                 void (*callback)(int))
+                isc_mem_t *mctx, dst_key_t **keyp,
+                void (*callback)(int))
 {
        dst_key_t *key;
        isc_result_t ret;
index 4edc9ffd488ba643fc7ac3082b583bd344c8deb0..18d918cb41a7bfe3ef0b1f37cea1993b98ea7415 100644 (file)
@@ -46,31 +46,14 @@ typedef struct dns_xfrin_ctx dns_xfrin_ctx_t;
 
 ISC_LANG_BEGINDECLS
 
-/*% see dns_xfrin_create2() */
 isc_result_t
 dns_xfrin_create(dns_zone_t *zone, dns_rdatatype_t xfrtype,
-                const isc_sockaddr_t *masteraddr, dns_tsigkey_t *tsigkey,
-                isc_mem_t *mctx, isc_timermgr_t *timermgr,
-                isc_socketmgr_t *socketmgr, isc_task_t *task,
-                dns_xfrindone_t done, dns_xfrin_ctx_t **xfrp);
-
-isc_result_t
-dns_xfrin_create2(dns_zone_t *zone, dns_rdatatype_t xfrtype,
-                 const isc_sockaddr_t *masteraddr,
-                 const isc_sockaddr_t *sourceaddr,
-                 dns_tsigkey_t *tsigkey, isc_mem_t *mctx,
-                 isc_timermgr_t *timermgr, isc_socketmgr_t *socketmgr,
-                 isc_task_t *task, dns_xfrindone_t done,
-                 dns_xfrin_ctx_t **xfrp);
-
-isc_result_t
-dns_xfrin_create3(dns_zone_t *zone, dns_rdatatype_t xfrtype,
-                 const isc_sockaddr_t *masteraddr,
-                 const isc_sockaddr_t *sourceaddr,
-                 isc_dscp_t dscp, dns_tsigkey_t *tsigkey, isc_mem_t *mctx,
-                 isc_timermgr_t *timermgr, isc_socketmgr_t *socketmgr,
-                 isc_task_t *task, dns_xfrindone_t done,
-                 dns_xfrin_ctx_t **xfrp);
+                const isc_sockaddr_t *masteraddr,
+                const isc_sockaddr_t *sourceaddr,
+                isc_dscp_t dscp, dns_tsigkey_t *tsigkey, isc_mem_t *mctx,
+                isc_timermgr_t *timermgr, isc_socketmgr_t *socketmgr,
+                isc_task_t *task, dns_xfrindone_t done,
+                dns_xfrin_ctx_t **xfrp);
 /*%<
  * Attempt to start an incoming zone transfer of 'zone'
  * from 'masteraddr', creating a dns_xfrin_ctx_t object to
index 53baf0bcdf9f27c13e003a43dca4849c3f64265f..2053ff826438ba665ef6509bd845b03237a3ef35 100644 (file)
@@ -128,13 +128,9 @@ typedef struct dst_context         dst_context_t;
 /***
  *** Functions
  ***/
-
-isc_result_t
-dst_lib_init(isc_mem_t *mctx, isc_entropy_t *ectx, unsigned int eflags);
-
 isc_result_t
-dst_lib_init2(isc_mem_t *mctx, isc_entropy_t *ectx,
-             const char *engine, unsigned int eflags);
+dst_lib_init(isc_mem_t *mctx, isc_entropy_t *ectx,
+            const char *engine, unsigned int eflags);
 /*%<
  * Initializes the DST subsystem.
  *
@@ -196,21 +192,9 @@ dst_ds_digest_supported(unsigned int digest_type);
  */
 
 isc_result_t
-dst_context_create(dst_key_t *key, isc_mem_t *mctx, dst_context_t **dctxp);
-
-isc_result_t
-dst_context_create2(dst_key_t *key, isc_mem_t *mctx,
-                   isc_logcategory_t *category, dst_context_t **dctxp);
-
-isc_result_t
-dst_context_create3(dst_key_t *key, isc_mem_t *mctx,
-                   isc_logcategory_t *category, isc_boolean_t useforsigning,
-                   dst_context_t **dctxp);
-
-isc_result_t
-dst_context_create4(dst_key_t *key, isc_mem_t *mctx,
-                   isc_logcategory_t *category, isc_boolean_t useforsigning,
-                   int maxbits, dst_context_t **dctxp);
+dst_context_create(dst_key_t *key, isc_mem_t *mctx,
+                  isc_logcategory_t *category, isc_boolean_t useforsigning,
+                  int maxbits, dst_context_t **dctxp);
 /*%<
  * Creates a context to be used for a sign or verify operation.
  *
@@ -583,15 +567,8 @@ dst_key_generate(const dns_name_t *name, unsigned int alg,
                 unsigned int bits, unsigned int param,
                 unsigned int flags, unsigned int protocol,
                 dns_rdataclass_t rdclass,
-                isc_mem_t *mctx, dst_key_t **keyp);
-
-isc_result_t
-dst_key_generate2(const dns_name_t *name, unsigned int alg,
-                 unsigned int bits, unsigned int param,
-                 unsigned int flags, unsigned int protocol,
-                 dns_rdataclass_t rdclass,
-                 isc_mem_t *mctx, dst_key_t **keyp,
-                 void (*callback)(int));
+                isc_mem_t *mctx, dst_key_t **keyp,
+                void (*callback)(int));
 
 /*%<
  * Generate a DST key (or keypair) with the supplied parameters.  The
index 20aabc9a80f60ea780b96e50cc11910b29b3c482..887851141f05f8c9a73a0e0fbe5a67c3f9e7abb5 100644 (file)
@@ -91,7 +91,7 @@ initialize(void) {
        if (result != ISC_R_SUCCESS)
                goto cleanup_db;
 
-       result = dst_lib_init(dns_g_mctx, ectx, 0);
+       result = dst_lib_init(dns_g_mctx, ectx, NULL, 0);
        if (result != ISC_R_SUCCESS)
                goto cleanup_ectx;
 
index ef6d1844c091c3b16bfab85dab62c3f934b98d96..b672b50f6113550a2d861e5213018cb4418c72d1 100644 (file)
@@ -120,7 +120,7 @@ dns_test_begin(FILE *logfile, isc_boolean_t start_managers) {
        CHECK(isc_mem_create(0, 0, &mctx));
        CHECK(isc_entropy_create(mctx, &ectx));
 
-       CHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_BLOCKING));
+       CHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_BLOCKING));
        dst_active = ISC_TRUE;
 
        if (logfile != NULL) {
index 284249b65fede7d15c57267a13f7ca14ea0f51b2..397381bb60e461e088c2958f9d1925df551fdf99 100644 (file)
@@ -177,8 +177,8 @@ check_sig(const char *datapath, const char *sigpath, const char *keyname,
         */
        isc_buffer_remainingregion(&sigbuf, &sigreg);
 
-       result = dst_context_create3(key, mctx, DNS_LOGCATEGORY_GENERAL,
-                                    ISC_FALSE, &ctx);
+       result = dst_context_create(key, mctx, DNS_LOGCATEGORY_GENERAL,
+                                   ISC_FALSE, 0, &ctx);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
        result = dst_context_adddata(ctx, &datareg);
index 58577153e6ad9cd2eaf4ecd3a4dbd9a4272d61a6..55d1de66a37f727a7b410908d5f3845e4b05fd70 100644 (file)
@@ -48,7 +48,7 @@ ATF_TC_BODY(isc_entropy_getdata, tc) {
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
        result = isc_entropy_create(mctx, &ectx);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
-       result = dst_lib_init(mctx, ectx, 0);
+       result = dst_lib_init(mctx, ectx, NULL, 0);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
 
 #ifdef ISC_PLATFORM_CRYPTORANDOM
index a700b5e18f00e8afd64291837fb1fdaf7d5db104..fb9a035460d1631e830fc79b8f23416abc67b0f7 100644 (file)
@@ -208,8 +208,8 @@ ATF_TC_BODY(isc_rsa_verify, tc) {
 
        /* RSASHA1 */
 
-       ret = dst_context_create3(key, mctx, DNS_LOGCATEGORY_DNSSEC,
-                                 ISC_FALSE, &ctx);
+       ret = dst_context_create(key, mctx, DNS_LOGCATEGORY_DNSSEC,
+                                ISC_FALSE, 0, &ctx);
        ATF_REQUIRE_EQ(ret, ISC_R_SUCCESS);
 
        r.base = d;
@@ -229,8 +229,8 @@ ATF_TC_BODY(isc_rsa_verify, tc) {
 #ifndef PK11_MD5_DISABLE
        key->key_alg = DST_ALG_RSAMD5;
 
-       ret = dst_context_create3(key, mctx, DNS_LOGCATEGORY_DNSSEC,
-                                 ISC_FALSE, &ctx);
+       ret = dst_context_create(key, mctx, DNS_LOGCATEGORY_DNSSEC,
+                                ISC_FALSE, 0, &ctx);
        ATF_REQUIRE_EQ(ret, ISC_R_SUCCESS);
 
        r.base = d;
@@ -250,8 +250,8 @@ ATF_TC_BODY(isc_rsa_verify, tc) {
 
        key->key_alg = DST_ALG_RSASHA256;
 
-       ret = dst_context_create3(key, mctx, DNS_LOGCATEGORY_DNSSEC,
-                                 ISC_FALSE, &ctx);
+       ret = dst_context_create(key, mctx, DNS_LOGCATEGORY_DNSSEC,
+                                ISC_FALSE, 0, &ctx);
        ATF_REQUIRE_EQ(ret, ISC_R_SUCCESS);
 
        r.base = d;
@@ -270,8 +270,8 @@ ATF_TC_BODY(isc_rsa_verify, tc) {
 
        key->key_alg = DST_ALG_RSASHA512;
 
-       ret = dst_context_create3(key, mctx, DNS_LOGCATEGORY_DNSSEC,
-                                 ISC_FALSE, &ctx);
+       ret = dst_context_create(key, mctx, DNS_LOGCATEGORY_DNSSEC,
+                                ISC_FALSE, 0, &ctx);
        ATF_REQUIRE_EQ(ret, ISC_R_SUCCESS);
 
        r.base = d;
index 490eb0b73e9b8e72715c416a5a179241e9c48a0c..7ac73676df5aed0ce96b46f056dcffccd7891ca7 100644 (file)
@@ -357,8 +357,8 @@ ATF_TC_BODY(tsig_tcp, tc) {
        isc_buffer_free(&buf);
        dns_message_destroy(&msg);
 
-       result = dst_context_create3(key->key, mctx, DNS_LOGCATEGORY_DNSSEC,
-                                    ISC_FALSE, &outctx);
+       result = dst_context_create(key->key, mctx, DNS_LOGCATEGORY_DNSSEC,
+                                   ISC_FALSE, 0, &outctx);
        ATF_REQUIRE_EQ(result, ISC_R_SUCCESS);
        ATF_REQUIRE(outctx != NULL);
 
index 0310958e1c93a6286c051f7486d9011b33ac8818..758b81dc547616d4d24bafeb565d03c949729080 100644 (file)
@@ -850,9 +850,9 @@ dns_tsig_sign(dns_message_t *msg) {
                 * has validated at this point. This is why we include a
                 * MAC length > 0 in the reply.
                 */
-               ret = dst_context_create3(key->key, mctx,
-                                         DNS_LOGCATEGORY_DNSSEC,
-                                         ISC_TRUE, &ctx);
+               ret = dst_context_create(key->key, mctx,
+                                        DNS_LOGCATEGORY_DNSSEC,
+                                        ISC_TRUE, 0, &ctx);
                if (ret != ISC_R_SUCCESS)
                        return (ret);
 
@@ -1235,9 +1235,9 @@ dns_tsig_verify(isc_buffer_t *source, dns_message_t *msg,
                sig_r.base = tsig.signature;
                sig_r.length = tsig.siglen;
 
-               ret = dst_context_create3(key, mctx,
-                                         DNS_LOGCATEGORY_DNSSEC,
-                                         ISC_FALSE, &ctx);
+               ret = dst_context_create(key, mctx,
+                                        DNS_LOGCATEGORY_DNSSEC,
+                                        ISC_FALSE, 0, &ctx);
                if (ret != ISC_R_SUCCESS)
                        return (ret);
 
@@ -1531,9 +1531,9 @@ tsig_verify_tcp(isc_buffer_t *source, dns_message_t *msg) {
        }
 
        if (msg->tsigctx == NULL) {
-               ret = dst_context_create3(key, mctx,
-                                         DNS_LOGCATEGORY_DNSSEC,
-                                         ISC_FALSE, &msg->tsigctx);
+               ret = dst_context_create(key, mctx,
+                                        DNS_LOGCATEGORY_DNSSEC,
+                                        ISC_FALSE, 0, &msg->tsigctx);
                if (ret != ISC_R_SUCCESS)
                        goto cleanup_querystruct;
 
index 4314c0154d8f19be2b5a173fd2871c70278e64e1..2ad2be8ce22ad031fe8eb0aa1f3ae9c94bd0ac2a 100644 (file)
@@ -1109,8 +1109,6 @@ dns_viewlist_find
 dns_viewlist_findzone
 dns_xfrin_attach
 dns_xfrin_create
-dns_xfrin_create2
-dns_xfrin_create3
 dns_xfrin_detach
 dns_xfrin_shutdown
 dns_zone_addnsec3chain
@@ -1357,9 +1355,6 @@ dns_zt_unmount
 dst_algorithm_supported
 dst_context_adddata
 dst_context_create
-dst_context_create2
-dst_context_create3
-dst_context_create4
 dst_context_destroy
 dst_context_sign
 dst_context_verify
@@ -1390,7 +1385,6 @@ dst_key_fromgssapi
 dst_key_fromlabel
 dst_key_fromnamedfile
 dst_key_generate
-dst_key_generate2
 dst_key_getbits
 dst_key_getfilename
 dst_key_getgssctx
@@ -1431,7 +1425,6 @@ dst_key_unsetnum
 dst_key_unsettime
 dst_lib_destroy
 dst_lib_init
-dst_lib_init2
 dst_lib_initmsgcat
 @IF PKCS11
 dst_random_getdata
index 162987100e309ee0f6b7404808788fd9968fd031..8e192ace1026614f2aba73df7e6b1cd2e78f230e 100644 (file)
@@ -630,54 +630,12 @@ xfr_rr(dns_xfrin_ctx_t *xfr, dns_name_t *name, isc_uint32_t ttl,
 
 isc_result_t
 dns_xfrin_create(dns_zone_t *zone, dns_rdatatype_t xfrtype,
-                const isc_sockaddr_t *masteraddr, dns_tsigkey_t *tsigkey,
-                isc_mem_t *mctx, isc_timermgr_t *timermgr,
-                isc_socketmgr_t *socketmgr, isc_task_t *task,
-                dns_xfrindone_t done, dns_xfrin_ctx_t **xfrp)
-{
-       isc_sockaddr_t sourceaddr;
-       isc_dscp_t dscp;
-
-       switch (isc_sockaddr_pf(masteraddr)) {
-       case PF_INET:
-               sourceaddr = *dns_zone_getxfrsource4(zone);
-               dscp = dns_zone_getxfrsource4dscp(zone);
-               break;
-       case PF_INET6:
-               sourceaddr = *dns_zone_getxfrsource6(zone);
-               dscp = dns_zone_getxfrsource6dscp(zone);
-               break;
-       default:
-               INSIST(0);
-       }
-
-       return(dns_xfrin_create3(zone, xfrtype, masteraddr, &sourceaddr,
-                                dscp, tsigkey, mctx, timermgr, socketmgr,
-                                task, done, xfrp));
-}
-
-isc_result_t
-dns_xfrin_create2(dns_zone_t *zone, dns_rdatatype_t xfrtype,
-                 const isc_sockaddr_t *masteraddr,
-                 const isc_sockaddr_t *sourceaddr,
-                 dns_tsigkey_t *tsigkey, isc_mem_t *mctx,
-                 isc_timermgr_t *timermgr, isc_socketmgr_t *socketmgr,
-                 isc_task_t *task, dns_xfrindone_t done,
-                 dns_xfrin_ctx_t **xfrp)
-{
-       return (dns_xfrin_create3(zone, xfrtype, masteraddr, sourceaddr, -1,
-                                 tsigkey, mctx, timermgr, socketmgr, task,
-                                 done, xfrp));
-}
-
-isc_result_t
-dns_xfrin_create3(dns_zone_t *zone, dns_rdatatype_t xfrtype,
-                 const isc_sockaddr_t *masteraddr,
-                 const isc_sockaddr_t *sourceaddr,
-                 isc_dscp_t dscp, dns_tsigkey_t *tsigkey, isc_mem_t *mctx,
-                 isc_timermgr_t *timermgr, isc_socketmgr_t *socketmgr,
-                 isc_task_t *task, dns_xfrindone_t done,
-                 dns_xfrin_ctx_t **xfrp)
+                const isc_sockaddr_t *masteraddr,
+                const isc_sockaddr_t *sourceaddr,
+                isc_dscp_t dscp, dns_tsigkey_t *tsigkey, isc_mem_t *mctx,
+                isc_timermgr_t *timermgr, isc_socketmgr_t *socketmgr,
+                isc_task_t *task, dns_xfrindone_t done,
+                dns_xfrin_ctx_t **xfrp)
 {
        dns_name_t *zonename = dns_zone_getorigin(zone);
        dns_xfrin_ctx_t *xfr = NULL;
index 8edc8ae0223c833b6994c78a100468c98a81b1f8..b02da3b3b28be519d428fcfbb47c96f1d356ab57 100644 (file)
@@ -15546,10 +15546,10 @@ got_transfer_quota(isc_task_t *task, isc_event_t *event) {
        };
        UNLOCK_ZONE(zone);
        INSIST(isc_sockaddr_pf(&masteraddr) == isc_sockaddr_pf(&sourceaddr));
-       result = dns_xfrin_create3(zone, xfrtype, &masteraddr, &sourceaddr,
-                                  dscp, zone->tsigkey, zone->mctx,
-                                  zone->zmgr->timermgr, zone->zmgr->socketmgr,
-                                  zone->task, zone_xfrdone, &zone->xfr);
+       result = dns_xfrin_create(zone, xfrtype, &masteraddr, &sourceaddr,
+                                 dscp, zone->tsigkey, zone->mctx,
+                                 zone->zmgr->timermgr, zone->zmgr->socketmgr,
+                                 zone->task, zone_xfrdone, &zone->xfr);
        if (result == ISC_R_SUCCESS) {
                LOCK_ZONE(zone);
                if (xfrtype == dns_rdatatype_axfr) {
index c63cea62b089d325ad56a206668dab88abdbf269..ce43815272139b1faea09feb525bc0e127d9d9db 100644 (file)
@@ -245,7 +245,7 @@ ns_test_begin(FILE *logfile, isc_boolean_t start_managers) {
        CHECK(isc_mem_create(0, 0, &mctx));
        CHECK(isc_entropy_create(mctx, &ectx));
 
-       CHECK(dst_lib_init(mctx, ectx, ISC_ENTROPY_BLOCKING));
+       CHECK(dst_lib_init(mctx, ectx, NULL, ISC_ENTROPY_BLOCKING));
        dst_active = ISC_TRUE;
 
        if (logfile != NULL) {