commit 6ea1a95ce831f66793f5db99dbb36e5a11a62618
author Paul Bakker <p.j.bakker@polarssl.org> Tue Dec 31 11:16:03 2013 +0100
committer Paul Bakker <p.j.bakker@polarssl.org> Tue Dec 31 11:17:14 2013 +0100
tree 13dc3cfde696d3a9388e30a1101bafd4200a6126
parent 5bc07a3d302ce403e98b24d667338c7b07230dec

Added missing MPI_CHK() around some statements

ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 62cd00c..3617915 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -30,6 +30,8 @@
    * Potential memory leak in ssl_ticket_keys_init()
    * Memory leak in benchmark application
    * Fixed x509_crt_parse_path() bug on Windows platforms
+   * Added missing MPI_CHK() around some statements in mpi_div_mpi() (found by
+     TrustInSoft)
 
 Security
    * Possible remotely-triggered out-of-bounds memory access fixed (found by

library/bignum.c

diff --git a/library/bignum.c b/library/bignum.c
index dd5c0bf..eb7a822 100644
--- a/library/bignum.c
+++ b/library/bignum.c
@@ -1221,9 +1221,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-- )
     {
@@ -1316,15 +1316,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;