]> git.ipfire.org Git - thirdparty/gnutls.git/commitdiff
ext/{client,server}_cert_type: use proper types for integers
authorDaiki Ueno <ueno@gnu.org>
Thu, 9 Sep 2021 07:38:21 +0000 (09:38 +0200)
committerDaiki Ueno <ueno@gnu.org>
Thu, 9 Sep 2021 07:38:21 +0000 (09:38 +0200)
Spotted by LGTM.

Signed-off-by: Daiki Ueno <ueno@gnu.org>
lib/ext/client_cert_type.c
lib/ext/server_cert_type.c

index 264c144d553fb8d4edb358068c0f946977709d47..261b56c8001d6600e0a9f8d9e486595e338e9b0f 100644 (file)
@@ -70,7 +70,8 @@ static int _gnutls_client_cert_type_recv_params(gnutls_session_t session,
 {
        int ret;
        gnutls_certificate_type_t cert_type;
-       uint8_t i, found = 0;
+       size_t i;
+       bool found = false;
        const uint8_t* pdata = data;
 
        /* Only activate this extension if we have cert credentials set
@@ -121,7 +122,7 @@ static int _gnutls_client_cert_type_recv_params(gnutls_session_t session,
                // Check whether what we got back is actually offered by us
                for (i = 0; i < sent_cert_types.size; i++) {
                        if (IANA2cert_type(sent_cert_types.data[i]) == cert_type)
-                               found = 1;
+                               found = true;
                }
 
                if (found) {
@@ -174,7 +175,7 @@ static int _gnutls_client_cert_type_recv_params(gnutls_session_t session,
 
                        // Check for support of this cert type
                        if (_gnutls_session_is_cert_type_supported(session, cert_type, false, GNUTLS_CTYPE_CLIENT) == 0) {
-                               found = 1;
+                               found = true;
                                break;
                        }
                }
@@ -213,7 +214,7 @@ static int _gnutls_client_cert_type_send_params(gnutls_session_t session,
 
        if (!IS_SERVER(session)) {      // Client mode
                uint8_t cert_types[GNUTLS_CRT_MAX]; // The list with supported (IANA) cert types. Inv: 0 <= cert type Id < 256
-               uint8_t i = 0, num_cert_types = 0;
+               size_t i, num_cert_types = 0;
                priority_st* cert_priorities;
                gnutls_datum_t tmp_cert_types; // For type conversion
 
index e2221883f81481603f8f429806f78ede5cd55862..6db2a1f92ee1643bd6211dd19c99bb455b5ca77f 100644 (file)
@@ -70,7 +70,8 @@ static int _gnutls_server_cert_type_recv_params(gnutls_session_t session,
 {
        int ret;
        gnutls_certificate_type_t cert_type;
-       uint8_t i, found = 0;
+       size_t i;
+       bool found = false;
        const uint8_t* pdata = data;
 
        /* Only activate this extension if we have cert credentials set
@@ -120,7 +121,7 @@ static int _gnutls_server_cert_type_recv_params(gnutls_session_t session,
                // Check whether what we got back is actually offered by us
                for (i = 0; i < sent_cert_types.size; i++) {
                        if (IANA2cert_type(sent_cert_types.data[i]) == cert_type)
-                               found = 1;
+                               found = true;
                }
 
                if (found) {
@@ -173,7 +174,7 @@ static int _gnutls_server_cert_type_recv_params(gnutls_session_t session,
 
                        // Check for support of this cert type
                        if (_gnutls_session_is_cert_type_supported(session, cert_type, true, GNUTLS_CTYPE_SERVER) == 0) {
-                               found = 1;
+                               found = true;
                                break;
                        }
                }
@@ -209,7 +210,7 @@ static int _gnutls_server_cert_type_send_params(gnutls_session_t session,
 
        if (!IS_SERVER(session)) {      // Client mode
                uint8_t cert_types[GNUTLS_CRT_MAX]; // The list with supported (IANA) cert types. Inv: 0 <= cert type Id < 256
-               uint8_t i = 0, num_cert_types = 0;
+               size_t i, num_cert_types = 0;
                priority_st* cert_priorities;
                gnutls_datum_t tmp_cert_types; // For type conversion