commit 78e819698b80d0c5b0c8114c2521808f308a994d
author Paul Bakker <p.j.bakker@polarssl.org> Tue Dec 31 11:16:03 2013 +0100
committer Paul Bakker <p.j.bakker@polarssl.org> Mon Jul 07 16:01:10 2014 +0200
tree 77e080d49d27e0949fdeb5b8c690d85097085edb
parent b5296b7f78630ecbdc5e05d7b0d253460d1a3e5e

Added missing MPI_CHK() around some statements

ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 3236632..3bc8e9b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -9,6 +9,8 @@
    * SSL now gracefully handles missing RNG
    * crypt_and_hash app checks MAC before final decryption
    * 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
 Changes

library/bignum.c

diff --git a/library/bignum.c b/library/bignum.c
index 504f51b..23a3f68 100644
--- a/library/bignum.c
+++ b/library/bignum.c
@@ -1104,9 +1104,9 @@
     while( mpi_cmp_mpi( &X, &Y ) >= 0 )
     {
         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-- )
     {
@@ -1199,15 +1199,15 @@
 
     if( Q != NULL )
     {
-        mpi_copy( Q, &Z );
+        MPI_CHK( mpi_copy( Q, &Z ) );
         Q->s = A->s * B->s;
     }
 
     if( R != NULL )
     {
-        mpi_shift_r( &X, k );
+        MPI_CHK( mpi_shift_r( &X, k ) );
         X.s = A->s;
-        mpi_copy( R, &X );
+        MPI_CHK( mpi_copy( R, &X ) );
 
         if( mpi_cmp_int( R, 0 ) == 0 )
             R->s = 1;