Skip to content

Commit f02f6d1

Browse files
authored
Merge pull request #9895 from embhorn/f283-287
Hardening in GeneratePrivateDh186 and wc_DhImportKeyPair
2 parents d4ac953 + 203cce4 commit f02f6d1

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

wolfcrypt/src/dh.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1157,8 +1157,9 @@ static int GeneratePrivateDh186(DhKey* key, WC_RNG* rng, byte* priv,
11571157
if (err == MP_OKAY)
11581158
err = mp_read_unsigned_bin(tmpX, cBuf, cSz);
11591159
if (err != MP_OKAY) {
1160-
mp_clear(tmpX);
1160+
mp_forcezero(tmpX);
11611161
mp_clear(tmpQ);
1162+
ForceZero(cBuf, cSz);
11621163
#if defined(WOLFSSL_SMALL_STACK) && !defined(WOLFSSL_NO_MALLOC)
11631164
XFREE(cBuf, key->heap, DYNAMIC_TYPE_TMP_BUFFER);
11641165
XFREE(tmpQ, key->heap, DYNAMIC_TYPE_DH);
@@ -2403,7 +2404,7 @@ int wc_DhImportKeyPair(DhKey* key, const byte* priv, word32 privSz,
24032404
}
24042405
if (havePriv) {
24052406
if (mp_read_unsigned_bin(&key->priv, priv, privSz) != MP_OKAY) {
2406-
mp_clear(&key->priv);
2407+
mp_forcezero(&key->priv);
24072408
havePriv = 0;
24082409
} else {
24092410
WOLFSSL_MSG("DH Private Key Set");

0 commit comments

Comments
 (0)