Merge pull request #244 from libtom/warn-unused-result

add warn_unused_result, found one missing check!

Author: sjaeckel

bn_mp_mod_2d.c

@@ -16,8 +16,7 @@ int mp_mod_2d(const mp_int *a, int b, mp_int *c)
 
    /* if the modulus is larger than the value than return */
    if (b >= (a->used * MP_DIGIT_BIT)) {
-      res = mp_copy(a, c);
-      return res;
+      return mp_copy(a, c);
    }
 
    /* copy */

bn_s_mp_toom_mul.c

@@ -52,7 +52,9 @@ int s_mp_toom_mul(const mp_int *a, const mp_int *b, mp_int *c)
       goto LBL_ERR;
    }
    mp_rshd(&b1, B);
-   (void)mp_mod_2d(&b1, MP_DIGIT_BIT * B, &b1);
+   if ((res = mp_mod_2d(&b1, MP_DIGIT_BIT * B, &b1)) != MP_OKAY) {
+      goto LBL_ERR;
+   }
 
    if ((res = mp_copy(b, &b2)) != MP_OKAY) {
       goto LBL_ERR;

demo/shared.h

@@ -24,6 +24,7 @@
 #define LTM_DEMO_RAND_SEED  23
 #endif
 
+#define MP_WUR /* TODO: result checks disabled for now */
 #include "tommath.h"
 
 extern void ndraw(mp_int* a, const char* name);