yuzu-emu/mbedtls
1
0
Fork 0
mirror of https://github.com/yuzu-emu/mbedtls.git synced 2024-11-22 10:45:36 +01:00
Code Issues Packages Projects Releases Wiki Activity

Make mbedtls_mpi_lt_mpi_ct more portable

Browse Source 
The code relied on the assumptions that CHAR_BIT is 8 and that unsigned
does not have padding bits.

In the Bignum module we already assume that the sign of an MPI is either
-1 or 1. Using this, we eliminate the above mentioned dependency.
This commit is contained in:
Janos Follath 2019-10-28 12:12:15 +00:00
parent aac48d1b3d
commit 58525180fb
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
library/bignum.c
View File

@ -1123,12 +1123,11 @@ int mbedtls_mpi_lt_mpi_ct( const mbedtls_mpi *X, const mbedtls_mpi *Y,
return MBEDTLS_ERR_MPI_BAD_INPUT_DATA;
/*
* Get sign bits of the signs.
* Set sign_N to 1 if N >= 0, 0 if N < 0.
* We know that N->s == 1 if N >= 0 and N->s == -1 if N < 0.
*/
sign_X = X->s;
sign_X = sign_X >> ( sizeof( unsigned ) * 8 - 1 );
sign_Y = Y->s;
sign_Y = sign_Y >> ( sizeof( unsigned ) * 8 - 1 );
sign_X = ( X->s & 2 ) >> 1;
sign_Y = ( Y->s & 2 ) >> 1;
/*
* If the signs are different, then the positive operand is the bigger.