commit ee4ba54d8dc51bbcf409db8f7a0055a8e0427ca6
author Manuel Pégourié-Gonnard <mpg@elzevir.fr> Fri Jan 24 12:11:56 2020 +0100
committer Manuel Pégourié-Gonnard <mpg@elzevir.fr> Fri Jan 24 12:11:56 2020 +0100
tree 5d20430b13f7e56072fe51b19b217917931d2063
parent d27a88438fe785f0906c01e3216044d1c98be875

Fix incrementing pointer instead of value

This was introduced by a hasty search-and-replace that didn't account for C's
operator precedence when changing those variables to pointer types.

library/ecdsa.c

diff --git a/library/ecdsa.c b/library/ecdsa.c
index e9c4315..5acd2d0 100644
--- a/library/ecdsa.c
+++ b/library/ecdsa.c
@@ -298,7 +298,7 @@
     *p_sign_tries = 0;
     do
     {
-        if( *p_sign_tries++ > 10 )
+        if( (*p_sign_tries)++ > 10 )
         {
             ret = MBEDTLS_ERR_ECP_RANDOM_FAILED;
             goto cleanup;
@@ -311,7 +311,7 @@
         *p_key_tries = 0;
         do
         {
-            if( *p_key_tries++ > 10 )
+            if( (*p_key_tries)++ > 10 )
             {
                 ret = MBEDTLS_ERR_ECP_RANDOM_FAILED;
                 goto cleanup;