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/x509_create.c

diff --git a/library/x509_create.c b/library/x509_create.c
index 056bbaa..af157e8 100644
--- a/library/x509_create.c
+++ b/library/x509_create.c
@@ -115,9 +115,9 @@
             break;
 
     if ( cur->name == NULL )
-        return( NULL );
+        return NULL ;
 
-    return( cur );
+    return cur ;
 }
 
 int mbedtls_x509_string_to_names( mbedtls_asn1_named_data **head, const char *name )
@@ -170,7 +170,7 @@
 
             if(cur == NULL )
             {
-                return( MBEDTLS_ERR_X509_ALLOC_FAILED );
+                return MBEDTLS_ERR_X509_ALLOC_FAILED ;
             }
 
             // set tagType
@@ -199,7 +199,7 @@
 
 exit:
 
-    return( ret );
+    return ret ;
 }
 
 /* The first byte of the value in the mbedtls_asn1_named_data structure is reserved
@@ -213,13 +213,13 @@
     if( ( cur = mbedtls_asn1_store_named_data( head, oid, oid_len,
                                        NULL, val_len + 1 ) ) == NULL )
     {
-        return( MBEDTLS_ERR_X509_ALLOC_FAILED );
+        return MBEDTLS_ERR_X509_ALLOC_FAILED ;
     }
 
     cur->val.p[0] = critical;
     memcpy( cur->val.p + 1, val, val_len );
 
-    return( 0 );
+    return 0 ;
 }
 
 /*
@@ -263,7 +263,7 @@
                                                  MBEDTLS_ASN1_CONSTRUCTED |
                                                  MBEDTLS_ASN1_SET ) );
 
-    return( (int) len );
+    return (int) len ;
 }
 
 int mbedtls_x509_write_names( unsigned char **p, unsigned char *start,
@@ -283,7 +283,7 @@
     MBEDTLS_ASN1_CHK_ADD( len, mbedtls_asn1_write_tag( p, start, MBEDTLS_ASN1_CONSTRUCTED |
                                                  MBEDTLS_ASN1_SEQUENCE ) );
 
-    return( (int) len );
+    return (int) len ;
 }
 
 int mbedtls_x509_write_sig( unsigned char **p, unsigned char *start,
@@ -294,14 +294,14 @@
     size_t len = 0;
 
     if( *p < start || (size_t)( *p - start ) < size )
-        return( MBEDTLS_ERR_ASN1_BUF_TOO_SMALL );
+        return MBEDTLS_ERR_ASN1_BUF_TOO_SMALL ;
 
     len = size;
     (*p) -= len;
     memcpy( *p, sig, len );
 
     if( *p - start < 1 )
-        return( MBEDTLS_ERR_ASN1_BUF_TOO_SMALL );
+        return MBEDTLS_ERR_ASN1_BUF_TOO_SMALL ;
 
     *--(*p) = 0;
     len += 1;
@@ -314,7 +314,7 @@
     MBEDTLS_ASN1_CHK_ADD( len, mbedtls_asn1_write_algorithm_identifier( p, start, oid,
                                                         oid_len, 0 ) );
 
-    return( (int) len );
+    return (int) len ;
 }
 
 static int x509_write_extension( unsigned char **p, unsigned char *start,
@@ -342,7 +342,7 @@
     MBEDTLS_ASN1_CHK_ADD( len, mbedtls_asn1_write_tag( p, start, MBEDTLS_ASN1_CONSTRUCTED |
                                                  MBEDTLS_ASN1_SEQUENCE ) );
 
-    return( (int) len );
+    return (int) len ;
 }
 
 /*
@@ -368,7 +368,7 @@
         cur_ext = cur_ext->next;
     }
 
-    return( (int) len );
+    return (int) len ;
 }
 
 #endif /* MBEDTLS_X509_CREATE_C */