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

DH SP Fix

Browse Source 
When doing the sp_2048_mod_exp_*() is processed, one of the index
variables is masked with 0x1f when it was updated. Added the masking to
the initial set of the index.
This commit is contained in:
John Safranek
2018-08-01 16:32:22 -07:00
parent 244d931e75
commit 6527c2a724
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
wolfcrypt/src/sp_c32.c
View File

@ -2083,7 +2083,7 @@ static int sp_2048_mod_exp_45(sp_digit* r, sp_digit* a, sp_digit* e, int bits,
n |= e[i--] << (9 - c); n |= e[i--] << (9 - c);
c += 23; c += 23;
} }
y = n >> 27; y = (n >> 27) & 0x1f;
n <<= 5; n <<= 5;
c -= 5; c -= 5;
XMEMCPY(rt, t[y], sizeof(rt)); XMEMCPY(rt, t[y], sizeof(rt));
@ -3016,7 +3016,7 @@ static int sp_2048_mod_exp_90(sp_digit* r, sp_digit* a, sp_digit* e, int bits,
n |= e[i--] << (9 - c); n |= e[i--] << (9 - c);
c += 23; c += 23;
} }
y = n >> 27; y = (n >> 27) & 0x1f;
n <<= 5; n <<= 5;
c -= 5; c -= 5;
XMEMCPY(rt, t[y], sizeof(rt)); XMEMCPY(rt, t[y], sizeof(rt));

4
wolfcrypt/src/sp_c64.c
View File

@ -1593,7 +1593,7 @@ static int sp_2048_mod_exp_18(sp_digit* r, sp_digit* a, sp_digit* e, int bits,
n |= e[i--] << (7 - c); n |= e[i--] << (7 - c);
c += 57; c += 57;
} }
y = n >> 59; y = (n >> 59) & 0x1f;
n <<= 5; n <<= 5;
c -= 5; c -= 5;
XMEMCPY(rt, t[y], sizeof(rt)); XMEMCPY(rt, t[y], sizeof(rt));
@ -2442,7 +2442,7 @@ static int sp_2048_mod_exp_36(sp_digit* r, sp_digit* a, sp_digit* e, int bits,
n |= e[i--] << (7 - c); n |= e[i--] << (7 - c);
c += 57; c += 57;
} }
y = n >> 59; y = (n >> 59) & 0x1f;
n <<= 5; n <<= 5;
c -= 5; c -= 5;
XMEMCPY(rt, t[y], sizeof(rt)); XMEMCPY(rt, t[y], sizeof(rt));