feedc0de/wolfssl
1
0
Fork 0
forked from wolfSSL/wolfssl
Code Pull Requests Activity

Fix issue with wc_ecc_make_key where state failure can occur if the wc_ecc_init hasn’t been called on key prior. Now wc_ecc_make_key and wc_ecc_import_private_key (and _ex versions) can be called without having to call wc_ecc_init first. This keeps backwards compatibility. If async or static memory is desired then wc_ecc_init_ex must be called first.

Browse Source
This commit is contained in:
David Garske
2017-04-11 15:57:09 -07:00
parent 85bef98331
commit d0f31d4a30
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
wolfcrypt/src/ecc.c
View File

@@ -3005,6 +3005,11 @@ int wc_ecc_make_key_ex(WC_RNG* rng, int keysize, ecc_key* key, int curve_id)
return BAD_FUNC_ARG;
}
/* make sure required key variables are reset */
key->state = ECC_STATE_NONE;
key->idx = 0;
key->dp = NULL;
err = wc_ecc_set_curve(key, keysize, curve_id);
if (err != 0) {
return err;
@@ -3255,6 +3260,7 @@ int wc_ecc_init_ex(ecc_key* key, void* heap, int devId)
#endif
XMEMSET(key, 0, sizeof(ecc_key));
key->state = ECC_STATE_NONE;
#ifdef WOLFSSL_ATECC508A
key->slot = atmel_ecc_alloc();
@@ -5083,7 +5089,6 @@ int wc_ecc_import_private_key_ex(const byte* priv, word32 privSz,
int curve_id)
{
int ret;
void* heap;
/* public optional, NULL if only importing private */
if (pub != NULL) {
@@ -5095,15 +5100,10 @@ int wc_ecc_import_private_key_ex(const byte* priv, word32 privSz,
if (key == NULL || priv == NULL)
return BAD_FUNC_ARG;
/* init key */
heap = key->heap;
ret = wc_ecc_init_ex(key, NULL, INVALID_DEVID);
key->heap = heap;
/* make sure required key variables are reset */
key->state = ECC_STATE_NONE;
if (ret != 0)
return ret;
key->idx = 0;
key->dp = NULL;
/* set key size */
ret = wc_ecc_set_curve(key, privSz, curve_id);