summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--usr.bin/dig/lib/dns/dst_api.c26
-rw-r--r--usr.bin/dig/lib/dns/dst_internal.h3
-rw-r--r--usr.bin/dig/lib/dns/hmac_link.c37
3 files changed, 4 insertions, 62 deletions
diff --git a/usr.bin/dig/lib/dns/dst_api.c b/usr.bin/dig/lib/dns/dst_api.c
index 9ddcefc662d..0fc5bba7c37 100644
--- a/usr.bin/dig/lib/dns/dst_api.c
+++ b/usr.bin/dig/lib/dns/dst_api.c
@@ -33,7 +33,7 @@
/*
* Principal Author: Brian Wellington
- * $Id: dst_api.c,v 1.8 2020/02/23 08:51:53 florian Exp $
+ * $Id: dst_api.c,v 1.9 2020/02/23 08:52:50 florian Exp $
*/
/*! \file */
@@ -150,8 +150,6 @@ dst_context_create3(dst_key_t *key,
REQUIRE(dst_initialized == ISC_TRUE);
REQUIRE(dctxp != NULL && *dctxp == NULL);
- if (key->func->createctx == NULL)
- return (DST_R_UNSUPPORTEDALG);
if (key->keydata.generic == NULL)
return (DST_R_NULLKEY);
@@ -183,7 +181,6 @@ dst_context_destroy(dst_context_t **dctxp) {
REQUIRE(dctxp != NULL);
dctx = *dctxp;
- INSIST(dctx->key->func->destroyctx != NULL);
dctx->key->func->destroyctx(dctx);
if (dctx->key != NULL)
dst_key_free(&dctx->key);
@@ -194,8 +191,6 @@ dst_context_destroy(dst_context_t **dctxp) {
isc_result_t
dst_context_adddata(dst_context_t *dctx, const isc_region_t *data) {
REQUIRE(data != NULL);
- INSIST(dctx->key->func->adddata != NULL);
-
return (dctx->key->func->adddata(dctx, data));
}
@@ -210,12 +205,6 @@ dst_context_sign(dst_context_t *dctx, isc_buffer_t *sig) {
if (key->keydata.generic == NULL)
return (DST_R_NULLKEY);
- if (key->func->sign == NULL)
- return (DST_R_NOTPRIVATEKEY);
- if (key->func->isprivate == NULL ||
- key->func->isprivate(key) == ISC_FALSE)
- return (DST_R_NOTPRIVATEKEY);
-
return (key->func->sign(dctx, sig));
}
@@ -226,8 +215,6 @@ dst_context_verify(dst_context_t *dctx, isc_region_t *sig) {
CHECKALG(dctx->key->key_alg);
if (dctx->key->keydata.generic == NULL)
return (DST_R_NULLKEY);
- if (dctx->key->func->verify == NULL)
- return (DST_R_NOTPUBLICKEY);
return (dctx->key->func->verify(dctx, sig));
}
@@ -239,9 +226,6 @@ dst_key_todns(const dst_key_t *key, isc_buffer_t *target) {
CHECKALG(key->key_alg);
- if (key->func->todns == NULL)
- return (DST_R_UNSUPPORTEDALG);
-
if (isc_buffer_availablelength(target) < 4)
return (ISC_R_NOSPACE);
isc_buffer_putuint16(target, (uint16_t)(key->key_flags & 0xffff));
@@ -313,10 +297,8 @@ dst_key_free(dst_key_t **keyp) {
return;
isc_refcount_destroy(&key->refs);
- if (key->keydata.generic != NULL) {
- INSIST(key->func->destroy != NULL);
+ if (key->keydata.generic != NULL)
key->func->destroy(key);
- }
if (key->engine != NULL)
free(key->engine);
if (key->label != NULL)
@@ -461,10 +443,6 @@ frombuffer(dns_name_t *name, unsigned int alg, unsigned int flags,
dst_key_free(&key);
return (ret);
}
- if (key->func->fromdns == NULL) {
- dst_key_free(&key);
- return (DST_R_UNSUPPORTEDALG);
- }
ret = key->func->fromdns(key, source);
if (ret != ISC_R_SUCCESS) {
diff --git a/usr.bin/dig/lib/dns/dst_internal.h b/usr.bin/dig/lib/dns/dst_internal.h
index 7799e9951e0..d76ebd8d6de 100644
--- a/usr.bin/dig/lib/dns/dst_internal.h
+++ b/usr.bin/dig/lib/dns/dst_internal.h
@@ -31,7 +31,7 @@
* IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*/
-/* $Id: dst_internal.h,v 1.8 2020/02/23 08:51:53 florian Exp $ */
+/* $Id: dst_internal.h,v 1.9 2020/02/23 08:52:50 florian Exp $ */
#ifndef DST_DST_INTERNAL_H
#define DST_DST_INTERNAL_H 1
@@ -143,7 +143,6 @@ struct dst_func {
*/
isc_result_t (*sign)(dst_context_t *dctx, isc_buffer_t *sig);
isc_result_t (*verify)(dst_context_t *dctx, const isc_region_t *sig);
- isc_boolean_t (*isprivate)(const dst_key_t *key);
void (*destroy)(dst_key_t *key);
/* conversion functions */
diff --git a/usr.bin/dig/lib/dns/hmac_link.c b/usr.bin/dig/lib/dns/hmac_link.c
index 40c911001f6..53d259ef883 100644
--- a/usr.bin/dig/lib/dns/hmac_link.c
+++ b/usr.bin/dig/lib/dns/hmac_link.c
@@ -33,7 +33,7 @@
/*
* Principal Author: Brian Wellington
- * $Id: hmac_link.c,v 1.5 2020/02/23 08:51:53 florian Exp $
+ * $Id: hmac_link.c,v 1.6 2020/02/23 08:52:50 florian Exp $
*/
#include <string.h>
@@ -113,12 +113,6 @@ hmacsha1_verify(dst_context_t *dctx, const isc_region_t *sig) {
return (DST_R_VERIFYFAILURE);
}
-static isc_boolean_t
-hmacsha1_isprivate(const dst_key_t *key) {
- UNUSED(key);
- return (ISC_TRUE);
-}
-
static void
hmacsha1_destroy(dst_key_t *key) {
dst_hmacsha1_key_t *hkey = key->keydata.hmacsha1;
@@ -186,7 +180,6 @@ static dst_func_t hmacsha1_functions = {
hmacsha1_adddata,
hmacsha1_sign,
hmacsha1_verify,
- hmacsha1_isprivate,
hmacsha1_destroy,
hmacsha1_todns,
hmacsha1_fromdns,
@@ -265,12 +258,6 @@ hmacsha224_verify(dst_context_t *dctx, const isc_region_t *sig) {
return (DST_R_VERIFYFAILURE);
}
-static isc_boolean_t
-hmacsha224_isprivate(const dst_key_t *key) {
- UNUSED(key);
- return (ISC_TRUE);
-}
-
static void
hmacsha224_destroy(dst_key_t *key) {
dst_hmacsha224_key_t *hkey = key->keydata.hmacsha224;
@@ -338,7 +325,6 @@ static dst_func_t hmacsha224_functions = {
hmacsha224_adddata,
hmacsha224_sign,
hmacsha224_verify,
- hmacsha224_isprivate,
hmacsha224_destroy,
hmacsha224_todns,
hmacsha224_fromdns,
@@ -417,12 +403,6 @@ hmacsha256_verify(dst_context_t *dctx, const isc_region_t *sig) {
return (DST_R_VERIFYFAILURE);
}
-static isc_boolean_t
-hmacsha256_isprivate(const dst_key_t *key) {
- UNUSED(key);
- return (ISC_TRUE);
-}
-
static void
hmacsha256_destroy(dst_key_t *key) {
dst_hmacsha256_key_t *hkey = key->keydata.hmacsha256;
@@ -490,7 +470,6 @@ static dst_func_t hmacsha256_functions = {
hmacsha256_adddata,
hmacsha256_sign,
hmacsha256_verify,
- hmacsha256_isprivate,
hmacsha256_destroy,
hmacsha256_todns,
hmacsha256_fromdns,
@@ -569,12 +548,6 @@ hmacsha384_verify(dst_context_t *dctx, const isc_region_t *sig) {
return (DST_R_VERIFYFAILURE);
}
-static isc_boolean_t
-hmacsha384_isprivate(const dst_key_t *key) {
- UNUSED(key);
- return (ISC_TRUE);
-}
-
static void
hmacsha384_destroy(dst_key_t *key) {
dst_hmacsha384_key_t *hkey = key->keydata.hmacsha384;
@@ -642,7 +615,6 @@ static dst_func_t hmacsha384_functions = {
hmacsha384_adddata,
hmacsha384_sign,
hmacsha384_verify,
- hmacsha384_isprivate,
hmacsha384_destroy,
hmacsha384_todns,
hmacsha384_fromdns,
@@ -721,12 +693,6 @@ hmacsha512_verify(dst_context_t *dctx, const isc_region_t *sig) {
return (DST_R_VERIFYFAILURE);
}
-static isc_boolean_t
-hmacsha512_isprivate(const dst_key_t *key) {
- UNUSED(key);
- return (ISC_TRUE);
-}
-
static void
hmacsha512_destroy(dst_key_t *key) {
dst_hmacsha512_key_t *hkey = key->keydata.hmacsha512;
@@ -794,7 +760,6 @@ static dst_func_t hmacsha512_functions = {
hmacsha512_adddata,
hmacsha512_sign,
hmacsha512_verify,
- hmacsha512_isprivate,
hmacsha512_destroy,
hmacsha512_todns,
hmacsha512_fromdns,