]> git.ipfire.org Git - thirdparty/gnutls.git/commitdiff
make sure gnutls_assert is present at the cases where GNUTLS_E_INTERNAL_ERROR is...
authorNikos Mavrogiannopoulos <nmav@redhat.com>
Tue, 15 Dec 2015 12:52:20 +0000 (13:52 +0100)
committerNikos Mavrogiannopoulos <nmav@redhat.com>
Tue, 15 Dec 2015 12:52:20 +0000 (13:52 +0100)
lib/session_pack.c
lib/state.c
lib/ui.c

index 19229b70d92ece95ec1133add16c56ac11ff8e39..e49b32c54f3bf9c8746a2a39f505e1cbeb1fc2b9 100644 (file)
@@ -136,7 +136,7 @@ _gnutls_session_pack(gnutls_session_t session,
                }
                break;
        default:
-               return GNUTLS_E_INTERNAL_ERROR;
+               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
        }
 
index 3758ad8d17331ac6ac3c99a8e7f2cc88ece5acf4..93c17dcebe8d31756d93f42acde48d6fa4baf25a 100644 (file)
@@ -521,7 +521,7 @@ int _gnutls_dh_set_peer_public(gnutls_session_t session, bigint_t public)
                        anon_auth_info_t info;
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
@@ -531,7 +531,7 @@ int _gnutls_dh_set_peer_public(gnutls_session_t session, bigint_t public)
                        psk_auth_info_t info;
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
@@ -542,14 +542,13 @@ int _gnutls_dh_set_peer_public(gnutls_session_t session, bigint_t public)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
                }
        default:
-               gnutls_assert();
-               return GNUTLS_E_INTERNAL_ERROR;
+               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
        }
 
        if (dh->public_key.data)
@@ -572,7 +571,7 @@ int _gnutls_dh_set_secret_bits(gnutls_session_t session, unsigned bits)
                        anon_auth_info_t info;
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        info->dh.secret_bits = bits;
                        break;
                }
@@ -581,7 +580,7 @@ int _gnutls_dh_set_secret_bits(gnutls_session_t session, unsigned bits)
                        psk_auth_info_t info;
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        info->dh.secret_bits = bits;
                        break;
                }
@@ -591,13 +590,12 @@ int _gnutls_dh_set_secret_bits(gnutls_session_t session, unsigned bits)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        info->dh.secret_bits = bits;
                        break;
        default:
-                       gnutls_assert();
-                       return GNUTLS_E_INTERNAL_ERROR;
+                       return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                }
        }
 
@@ -619,7 +617,7 @@ _gnutls_dh_set_group(gnutls_session_t session, bigint_t gen,
                        anon_auth_info_t info;
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
@@ -629,7 +627,7 @@ _gnutls_dh_set_group(gnutls_session_t session, bigint_t gen,
                        psk_auth_info_t info;
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
@@ -640,14 +638,13 @@ _gnutls_dh_set_group(gnutls_session_t session, bigint_t gen,
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
                }
        default:
-               gnutls_assert();
-               return GNUTLS_E_INTERNAL_ERROR;
+               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
        }
 
        if (dh->prime.data)
index a5f4c309381792670d07dd01168b9948b7c7d33b..e84feec970f0b5dd9e3cb3721579fb7fb5e6631c 100644 (file)
--- a/lib/ui.c
+++ b/lib/ui.c
@@ -144,19 +144,19 @@ gnutls_dh_get_group(gnutls_session_t session,
        case GNUTLS_CRD_ANON:
                anon_info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                if (anon_info == NULL)
-                       return GNUTLS_E_INTERNAL_ERROR;
+                       return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                dh = &anon_info->dh;
                break;
        case GNUTLS_CRD_PSK:
                psk_info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                if (psk_info == NULL)
-                       return GNUTLS_E_INTERNAL_ERROR;
+                       return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                dh = &psk_info->dh;
                break;
        case GNUTLS_CRD_CERTIFICATE:
                cert_info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                if (cert_info == NULL)
-                       return GNUTLS_E_INTERNAL_ERROR;
+                       return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                dh = &cert_info->dh;
                break;
        default:
@@ -211,7 +211,7 @@ gnutls_dh_get_pubkey(gnutls_session_t session, gnutls_datum_t * raw_key)
                {
                        anon_info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                        if (anon_info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        dh = &anon_info->dh;
                        break;
                }
@@ -219,7 +219,7 @@ gnutls_dh_get_pubkey(gnutls_session_t session, gnutls_datum_t * raw_key)
                {
                        psk_info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                        if (psk_info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        dh = &psk_info->dh;
                        break;
                }
@@ -228,7 +228,7 @@ gnutls_dh_get_pubkey(gnutls_session_t session, gnutls_datum_t * raw_key)
 
                        cert_info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                        if (cert_info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        dh = &cert_info->dh;
                        break;
                }
@@ -261,7 +261,7 @@ int gnutls_dh_get_secret_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        return info->dh.secret_bits;
                }
        case GNUTLS_CRD_PSK:
@@ -270,7 +270,7 @@ int gnutls_dh_get_secret_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        return info->dh.secret_bits;
                }
        case GNUTLS_CRD_CERTIFICATE:
@@ -279,7 +279,7 @@ int gnutls_dh_get_secret_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        return info->dh.secret_bits;
                }
@@ -332,7 +332,7 @@ int gnutls_dh_get_prime_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        dh = &info->dh;
                        break;
                }
@@ -342,7 +342,7 @@ int gnutls_dh_get_prime_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
                        dh = &info->dh;
                        break;
                }
@@ -352,7 +352,7 @@ int gnutls_dh_get_prime_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
@@ -390,7 +390,7 @@ int gnutls_dh_get_peers_public_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_ANON);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
@@ -401,7 +401,7 @@ int gnutls_dh_get_peers_public_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_PSK);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;
@@ -412,7 +412,7 @@ int gnutls_dh_get_peers_public_bits(gnutls_session_t session)
 
                        info = _gnutls_get_auth_info(session, GNUTLS_CRD_CERTIFICATE);
                        if (info == NULL)
-                               return GNUTLS_E_INTERNAL_ERROR;
+                               return gnutls_assert_val(GNUTLS_E_INTERNAL_ERROR);
 
                        dh = &info->dh;
                        break;