cheng/libsodium
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

aes256-ctr: do not left shift more than 16 bits on a signed int.

Browse Source 
Spotted by John Regehr and clang -fsigned-integer-overflow
This commit is contained in:
Frank Denis 2013-11-12 09:47:36 -08:00
parent 5a566c041c
commit f553bb4bf2
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/libsodium/crypto_stream/aes256estream/hongjun/aes256-ctr.c
View File

@ -27,7 +27,7 @@ ECRYPT_keysetup(ECRYPT_ctx* ctx, const u8* key, u32 keysize, u32 ivsize)
w[i] = key[(i << 2)];
w[i] |= key[(i << 2)+1] << 8;
w[i] |= key[(i << 2)+2] << 16;
w[i] |= key[(i << 2)+3] << 24;
w[i] |= (unsigned int) key[(i << 2)+3] << 24;
}
i = Nk;
@ -35,7 +35,7 @@ ECRYPT_keysetup(ECRYPT_ctx* ctx, const u8* key, u32 keysize, u32 ivsize)
while( i < Nb*(Nr+1) ) {
temp = w[i-1];
temp = Sbox[ temp & 0xFF] << 24 ^
temp = (unsigned int) Sbox[temp & 0xFF] << 24 ^
Sbox[(temp >> 8) & 0xFF] ^
(Sbox[(temp >> 16) & 0xFF] << 8 ) ^
(Sbox[(temp >> 24) & 0xFF] << 16) ^
@ -59,7 +59,7 @@ ECRYPT_keysetup(ECRYPT_ctx* ctx, const u8* key, u32 keysize, u32 ivsize)
temp = Sbox[temp & 0xFF] ^
Sbox[(temp >> 8) & 0xFF] << 8 ^
(Sbox[(temp >> 16) & 0xFF] << 16) ^
(Sbox[(temp >> 24) & 0xFF] << 24);
((unsigned int) Sbox[(temp >> 24) & 0xFF] << 24);
w[i] = w[i-Nk] ^ temp;
i++;