Added missing MPI_CHK() around some statements

This commit is contained in:
Paul Bakker 2013-12-31 11:16:03 +01:00
parent b5296b7f78
commit 78e819698b
2 changed files with 7 additions and 5 deletions

View file

@ -9,6 +9,8 @@ Bugfix
* SSL now gracefully handles missing RNG * SSL now gracefully handles missing RNG
* crypt_and_hash app checks MAC before final decryption * crypt_and_hash app checks MAC before final decryption
* Fixed x509_crt_parse_path() bug on Windows platforms * Fixed x509_crt_parse_path() bug on Windows platforms
* Added missing MPI_CHK() around some statements in mpi_div_mpi() (found by
TrustInSoft)
= Version 1.2.10 released 2013-10-07 = Version 1.2.10 released 2013-10-07
Changes Changes

View file

@ -1104,9 +1104,9 @@ int mpi_div_mpi( mpi *Q, mpi *R, const mpi *A, const mpi *B )
while( mpi_cmp_mpi( &X, &Y ) >= 0 ) while( mpi_cmp_mpi( &X, &Y ) >= 0 )
{ {
Z.p[n - t]++; Z.p[n - t]++;
mpi_sub_mpi( &X, &X, &Y ); MPI_CHK( mpi_sub_mpi( &X, &X, &Y ) );
} }
mpi_shift_r( &Y, biL * (n - t) ); MPI_CHK( mpi_shift_r( &Y, biL * (n - t) ) );
for( i = n; i > t ; i-- ) for( i = n; i > t ; i-- )
{ {
@ -1199,15 +1199,15 @@ int mpi_div_mpi( mpi *Q, mpi *R, const mpi *A, const mpi *B )
if( Q != NULL ) if( Q != NULL )
{ {
mpi_copy( Q, &Z ); MPI_CHK( mpi_copy( Q, &Z ) );
Q->s = A->s * B->s; Q->s = A->s * B->s;
} }
if( R != NULL ) if( R != NULL )
{ {
mpi_shift_r( &X, k ); MPI_CHK( mpi_shift_r( &X, k ) );
X.s = A->s; X.s = A->s;
mpi_copy( R, &X ); MPI_CHK( mpi_copy( R, &X ) );
if( mpi_cmp_int( R, 0 ) == 0 ) if( mpi_cmp_int( R, 0 ) == 0 )
R->s = 1; R->s = 1;