commit e36f5b1c11841ce564c652dc574c252de6265c28
author Mateusz Starzyk <mateusz.starzyk@mobica.com> Thu Jul 22 16:43:35 2021 +0200
committer Mateusz Starzyk <mateusz.starzyk@mobica.com> Tue Aug 10 16:54:21 2021 +0200
tree 3c2fa7dbc8d2920e1649fb0a7a066cd95dc388db
parent 200f58c07e5a1a9d16a97428e22d2bea912fffa5

Fix parantheses on return and sizeof statements.

Used script:
```
import re
import sys

for arg in sys.argv[1:]:
    print(arg)
    with open(arg, 'r') as file:
        content = file.read()
        content = re.sub(r"return\s?\((?!.*\).*\()\s*\n?(.*)\n?\);", r"return \1;", \
            content, flags = re.M)
        content = re.sub(r"sizeof ([^\(][a-zA-Z0-9_\[\]]*)", r"sizeof(\1)",\
            content, flags = re.M)
    with open(arg, 'w') as file:
        file.write(content)

```
Executed with:
` find . -regextype posix-egrep -regex ".*\.([hc]|fmt|function)" | xargs -L1 python script.py`

Signed-off-by: Mateusz Starzyk <mateusz.starzyk@mobica.com>

library/ecdsa.c

diff --git a/library/ecdsa.c b/library/ecdsa.c
index 0b612ce..1364c9e 100644
--- a/library/ecdsa.c
+++ b/library/ecdsa.c
@@ -185,7 +185,7 @@
     {                                                                \
         rs_ctx->SUB = mbedtls_calloc( 1, sizeof( *rs_ctx->SUB ) );   \
         if( rs_ctx->SUB == NULL )                                    \
-            return( MBEDTLS_ERR_ECP_ALLOC_FAILED );                  \
+            return MBEDTLS_ERR_ECP_ALLOC_FAILED ;                  \
                                                                      \
         ecdsa_restart_## SUB ##_init( rs_ctx->SUB );                 \
     }                                                                \
@@ -240,7 +240,7 @@
         MBEDTLS_MPI_CHK( mbedtls_mpi_sub_mpi( x, x, &grp->N ) );
 
 cleanup:
-    return( ret );
+    return ret ;
 }
 #endif /* ECDSA_DETERMINISTIC || !ECDSA_SIGN_ALT || !ECDSA_VERIFY_ALT */
 
@@ -265,11 +265,11 @@
 
     /* Fail cleanly on curves such as Curve25519 that can't be used for ECDSA */
     if( ! mbedtls_ecdsa_can_do( grp->id ) || grp->N.p == NULL )
-        return( MBEDTLS_ERR_ECP_BAD_INPUT_DATA );
+        return MBEDTLS_ERR_ECP_BAD_INPUT_DATA ;
 
     /* Make sure d is in range 1..n-1 */
     if( mbedtls_mpi_cmp_int( d, 1 ) < 0 || mbedtls_mpi_cmp_mpi( d, &grp->N ) >= 0 )
-        return( MBEDTLS_ERR_ECP_INVALID_KEY );
+        return MBEDTLS_ERR_ECP_INVALID_KEY ;
 
     mbedtls_ecp_point_init( &R );
     mbedtls_mpi_init( &k ); mbedtls_mpi_init( &e ); mbedtls_mpi_init( &t );
@@ -380,7 +380,7 @@
 
     ECDSA_RS_LEAVE( sig );
 
-    return( ret );
+    return ret ;
 }
 
 int mbedtls_ecdsa_can_do( mbedtls_ecp_group_id gid )
@@ -441,7 +441,7 @@
     mbedtls_mpi h;
 
     if( ( md_info = mbedtls_md_info_from_type( md_alg ) ) == NULL )
-        return( MBEDTLS_ERR_ECP_BAD_INPUT_DATA );
+        return MBEDTLS_ERR_ECP_BAD_INPUT_DATA ;
 
     mbedtls_mpi_init( &h );
     mbedtls_hmac_drbg_init( &rng_ctx );
@@ -489,7 +489,7 @@
 
     ECDSA_RS_LEAVE( det );
 
-    return( ret );
+    return ret ;
 }
 
 /*
@@ -537,7 +537,7 @@
 
     /* Fail cleanly on curves such as Curve25519 that can't be used for ECDSA */
     if( ! mbedtls_ecdsa_can_do( grp->id ) || grp->N.p == NULL )
-        return( MBEDTLS_ERR_ECP_BAD_INPUT_DATA );
+        return MBEDTLS_ERR_ECP_BAD_INPUT_DATA ;
 
     ECDSA_RS_ENTER( ver );
 
@@ -622,7 +622,7 @@
 
     ECDSA_RS_LEAVE( ver );
 
-    return( ret );
+    return ret ;
 }
 
 /*
@@ -640,7 +640,7 @@
     ECDSA_VALIDATE_RET( s   != NULL );
     ECDSA_VALIDATE_RET( buf != NULL || blen == 0 );
 
-    return( ecdsa_verify_restartable( grp, buf, blen, Q, r, s, NULL ) );
+    return ecdsa_verify_restartable( grp, buf, blen, Q, r, s, NULL ) ;
 }
 #endif /* !MBEDTLS_ECDSA_VERIFY_ALT */
 
@@ -664,12 +664,12 @@
                                        MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_SEQUENCE ) );
 
     if( len > sig_size )
-        return( MBEDTLS_ERR_ECP_BUFFER_TOO_SMALL );
+        return MBEDTLS_ERR_ECP_BUFFER_TOO_SMALL ;
 
     memcpy( sig, p, len );
     *slen = len;
 
-    return( 0 );
+    return 0 ;
 }
 
 /*
@@ -691,7 +691,7 @@
     ECDSA_VALIDATE_RET( slen  != NULL );
 
     if( f_rng == NULL )
-        return( MBEDTLS_ERR_ECP_BAD_INPUT_DATA );
+        return MBEDTLS_ERR_ECP_BAD_INPUT_DATA ;
 
     mbedtls_mpi_init( &r );
     mbedtls_mpi_init( &s );
@@ -722,7 +722,7 @@
     mbedtls_mpi_free( &r );
     mbedtls_mpi_free( &s );
 
-    return( ret );
+    return ret ;
 }
 
 /*
@@ -820,7 +820,7 @@
     mbedtls_mpi_free( &r );
     mbedtls_mpi_free( &s );
 
-    return( ret );
+    return ret ;
 }
 
 #if !defined(MBEDTLS_ECDSA_GENKEY_ALT)
@@ -836,7 +836,7 @@
 
     ret = mbedtls_ecp_group_load( &ctx->grp, gid );
     if( ret != 0 )
-        return( ret );
+        return ret ;
 
    return( mbedtls_ecp_gen_keypair( &ctx->grp, &ctx->d,
                                     &ctx->Q, f_rng, p_rng ) );
@@ -859,7 +859,7 @@
         mbedtls_ecdsa_free( ctx );
     }
 
-    return( ret );
+    return ret ;
 }
 
 /*