denozord/strongswan-ext
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

keymat_v2: Properly wipe DH secret during IKE_SA rekeying

Browse Source 
While `secret` is wiped explicitly, it wasn't when concatenating with
the nonces.
This commit is contained in:
Tobias Brunner
2021-09-20 16:10:03 +02:00
parent 91896264b2
commit a5662e8f6e
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/libcharon/sa/ikev2/keymat_v2.c
+1 -1
View File
@@ -387,7 +387,7 @@ METHOD(keymat_v2_t, derive_ike_keys, bool,
chunk_clear(&prf_plus_seed); chunk_clear(&prf_plus_seed);
return FALSE; return FALSE;
} }
secret = chunk_cat("mc", secret, full_nonce); secret = chunk_cat("sc", secret, full_nonce);
if (rekey_prf->set_key(rekey_prf, rekey_skd) && if (rekey_prf->set_key(rekey_prf, rekey_skd) &&
rekey_prf->allocate_bytes(rekey_prf, secret, &skeyseed) && rekey_prf->allocate_bytes(rekey_prf, secret, &skeyseed) &&
rekey_prf->set_key(rekey_prf, skeyseed)) rekey_prf->set_key(rekey_prf, skeyseed))