Define MODP_CUSTOM constructors as variadic functions

They now match the dh_constructor_t signature.  This is a follow up for
the changes merged with b668bf3f9e and should fix use of MODP_CUSTOM on
Apple's ARM64 platform.
This commit is contained in:
Tobias Brunner
2017-09-18 12:07:26 +02:00
parent f871b341d7
commit 46a62f0126
8 changed files with 28 additions and 22 deletions
+4 -2
View File
@@ -289,11 +289,13 @@ gcrypt_dh_t *gcrypt_dh_create(diffie_hellman_group_t group)
/* /*
* Described in header. * Described in header.
*/ */
gcrypt_dh_t *gcrypt_dh_create_custom(diffie_hellman_group_t group, gcrypt_dh_t *gcrypt_dh_create_custom(diffie_hellman_group_t group, ...)
chunk_t g, chunk_t p)
{ {
if (group == MODP_CUSTOM) if (group == MODP_CUSTOM)
{ {
chunk_t g, p;
VA_ARGS_GET(group, g, p);
return create_generic(group, p.len, g, p); return create_generic(group, p.len, g, p);
} }
return NULL; return NULL;
+2 -4
View File
@@ -48,12 +48,10 @@ gcrypt_dh_t *gcrypt_dh_create(diffie_hellman_group_t group);
* Creates a new gcrypt_dh_t object for MODP_CUSTOM. * Creates a new gcrypt_dh_t object for MODP_CUSTOM.
* *
* @param group MODP_CUSTOM * @param group MODP_CUSTOM
* @param g generator * @param ... expects generator and prime as chunk_t
* @param p prime
* @return gcrypt_dh_t object, NULL if not supported * @return gcrypt_dh_t object, NULL if not supported
*/ */
gcrypt_dh_t *gcrypt_dh_create_custom(diffie_hellman_group_t group, gcrypt_dh_t *gcrypt_dh_create_custom(diffie_hellman_group_t group, ...);
chunk_t g, chunk_t p);
#endif /** GCRYPT_DH_H_ @}*/ #endif /** GCRYPT_DH_H_ @}*/
@@ -272,7 +272,7 @@ static gmp_diffie_hellman_t *create_generic(diffie_hellman_group_t group,
} }
/* /*
* Described in header. * Described in header
*/ */
gmp_diffie_hellman_t *gmp_diffie_hellman_create(diffie_hellman_group_t group) gmp_diffie_hellman_t *gmp_diffie_hellman_create(diffie_hellman_group_t group)
{ {
@@ -287,12 +287,17 @@ gmp_diffie_hellman_t *gmp_diffie_hellman_create(diffie_hellman_group_t group)
params->generator, params->prime); params->generator, params->prime);
} }
/*
* Described in header
*/
gmp_diffie_hellman_t *gmp_diffie_hellman_create_custom( gmp_diffie_hellman_t *gmp_diffie_hellman_create_custom(
diffie_hellman_group_t group, chunk_t g, chunk_t p) diffie_hellman_group_t group, ...)
{ {
if (group == MODP_CUSTOM) if (group == MODP_CUSTOM)
{ {
chunk_t g, p;
VA_ARGS_GET(group, g, p);
return create_generic(MODP_CUSTOM, p.len, g, p); return create_generic(MODP_CUSTOM, p.len, g, p);
} }
return NULL; return NULL;
@@ -49,12 +49,11 @@ gmp_diffie_hellman_t *gmp_diffie_hellman_create(diffie_hellman_group_t group);
* Creates a new gmp_diffie_hellman_t object for MODP_CUSTOM. * Creates a new gmp_diffie_hellman_t object for MODP_CUSTOM.
* *
* @param group MODP_CUSTOM * @param group MODP_CUSTOM
* @param g generator * @param ... expects generator and prime as chunk_t
* @param p prime
* @return gmp_diffie_hellman_t object, NULL if not supported * @return gmp_diffie_hellman_t object, NULL if not supported
*/ */
gmp_diffie_hellman_t *gmp_diffie_hellman_create_custom( gmp_diffie_hellman_t *gmp_diffie_hellman_create_custom(
diffie_hellman_group_t group, chunk_t g, chunk_t p); diffie_hellman_group_t group, ...);
#endif /** GMP_DIFFIE_HELLMAN_H_ @}*/ #endif /** GMP_DIFFIE_HELLMAN_H_ @}*/
@@ -193,7 +193,7 @@ METHOD(diffie_hellman_t, destroy, void,
* Described in header. * Described in header.
*/ */
openssl_diffie_hellman_t *openssl_diffie_hellman_create( openssl_diffie_hellman_t *openssl_diffie_hellman_create(
diffie_hellman_group_t group, chunk_t g, chunk_t p) diffie_hellman_group_t group, ...)
{ {
private_openssl_diffie_hellman_t *this; private_openssl_diffie_hellman_t *this;
const BIGNUM *privkey; const BIGNUM *privkey;
@@ -225,6 +225,9 @@ openssl_diffie_hellman_t *openssl_diffie_hellman_create(
if (group == MODP_CUSTOM) if (group == MODP_CUSTOM)
{ {
chunk_t g, p;
VA_ARGS_GET(group, g, p);
if (!DH_set0_pqg(this->dh, BN_bin2bn(p.ptr, p.len, NULL), NULL, if (!DH_set0_pqg(this->dh, BN_bin2bn(p.ptr, p.len, NULL), NULL,
BN_bin2bn(g.ptr, g.len, NULL))) BN_bin2bn(g.ptr, g.len, NULL)))
{ {
@@ -40,12 +40,11 @@ struct openssl_diffie_hellman_t {
* Creates a new openssl_diffie_hellman_t object. * Creates a new openssl_diffie_hellman_t object.
* *
* @param group Diffie Hellman group number to use * @param group Diffie Hellman group number to use
* @param g custom generator, if MODP_CUSTOM * @param ... expects generator and prime as chunk_t if MODP_CUSTOM
* @param p custom prime, if MODP_CUSTOM
* @return openssl_diffie_hellman_t object, NULL if not supported * @return openssl_diffie_hellman_t object, NULL if not supported
*/ */
openssl_diffie_hellman_t *openssl_diffie_hellman_create( openssl_diffie_hellman_t *openssl_diffie_hellman_create(
diffie_hellman_group_t group, chunk_t g, chunk_t p); diffie_hellman_group_t group, ...);
#endif /** OPENSSL_DIFFIE_HELLMAN_H_ @}*/ #endif /** OPENSSL_DIFFIE_HELLMAN_H_ @}*/
+4 -2
View File
@@ -415,13 +415,15 @@ static chunk_t ecparams_lookup(diffie_hellman_group_t group)
/** /**
* Described in header. * Described in header.
*/ */
pkcs11_dh_t *pkcs11_dh_create(diffie_hellman_group_t group, pkcs11_dh_t *pkcs11_dh_create(diffie_hellman_group_t group, ...)
chunk_t g, chunk_t p)
{ {
switch (group) switch (group)
{ {
case MODP_CUSTOM: case MODP_CUSTOM:
{ {
chunk_t g, p;
VA_ARGS_GET(group, g, p);
return create_modp(group, p.len, g, p); return create_modp(group, p.len, g, p);
} }
case ECP_192_BIT: case ECP_192_BIT:
+2 -4
View File
@@ -40,12 +40,10 @@ struct pkcs11_dh_t {
* Creates a new pkcs11_dh_t object. * Creates a new pkcs11_dh_t object.
* *
* @param group Diffie Hellman group number to use * @param group Diffie Hellman group number to use
* @param g generator in case group is MODP_CUSTOM * @param ... expects generator and prime as chunk_t if MODP_CUSTOM
* @param p prime in case group is MODP_CUSTOM
* @return pkcs11_dh_t object, NULL if not supported * @return pkcs11_dh_t object, NULL if not supported
*/ */
pkcs11_dh_t *pkcs11_dh_create(diffie_hellman_group_t group, pkcs11_dh_t *pkcs11_dh_create(diffie_hellman_group_t group, ...);
chunk_t g, chunk_t p);
#endif /** PKCS11_DH_H_ @}*/ #endif /** PKCS11_DH_H_ @}*/