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

fix location in tfm.c that could result in potential cache attack

Browse Source
This commit is contained in:
Jacob Barthelmeh
2017-01-10 15:00:00 -07:00
parent 8954de40ff
commit e3277c19b7
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
wolfcrypt/src/tfm.c
View File

@ -480,12 +480,11 @@ void fp_mul_comba(fp_int *A, fp_int *B, fp_int *C)
pa = FP_SIZE-1; pa = FP_SIZE-1;
} }
if (A == C || B == C) { /* Always take branch to use tmp variable. This avoids a cache attack for
* determining if C equals A */
if (1) {
fp_init(&tmp); fp_init(&tmp);
dst = &tmp; dst = &tmp;
} else {
fp_zero(C);
dst = C;
} }
for (ix = 0; ix < pa; ix++) { for (ix = 0; ix < pa; ix++) {