diff options
author | shub-al <aleksandrshubnikov97@gmail.com> | 2023-08-17 16:21:26 +0300 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2023-08-20 13:26:46 +0200 |
commit | d561fe5a0aabb4d0a9400b5086441bb4f4b4dca4 (patch) | |
tree | 137e1dea715ed150cc69546842da9d637c3acfa2 | |
parent | 6404d064b8012a2c353603a3b3effa6289313d61 (diff) |
ssl_get_min_max_version(): Remove unused variable single
CLA: trivial
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/21768)
-rw-r--r-- | ssl/statem/statem_lib.c | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/ssl/statem/statem_lib.c b/ssl/statem/statem_lib.c index 406788bbfc..e94b2f76cd 100644 --- a/ssl/statem/statem_lib.c +++ b/ssl/statem/statem_lib.c @@ -2418,7 +2418,6 @@ int ssl_get_min_max_version(const SSL_CONNECTION *s, int *min_version, { int version, tmp_real_max; int hole; - const SSL_METHOD *single = NULL; const SSL_METHOD *method; const version_info *table; const version_info *vent; @@ -2460,13 +2459,12 @@ int ssl_get_min_max_version(const SSL_CONNECTION *s, int *min_version, * the valid protocol entries) and we don't have a selected version yet. * * Whenever "hole == 1", and we hit an enabled method, its version becomes - * the selected version, and the method becomes a candidate "single" - * method. We're no longer in a hole, so "hole" becomes 0. + * the selected version. We're no longer in a hole, so "hole" becomes 0. * - * If "hole == 0" and we hit an enabled method, then "single" is cleared, - * as we support a contiguous range of at least two methods. If we hit - * a disabled method, then hole becomes true again, but nothing else - * changes yet, because all the remaining methods may be disabled too. + * If "hole == 0" and we hit an enabled method, we support a contiguous + * range of at least two methods. If we hit a disabled method, + * then hole becomes true again, but nothing else changes yet, + * because all the remaining methods may be disabled too. * If we again hit an enabled method after the new hole, it becomes * selected, as we start from scratch. */ @@ -2493,12 +2491,11 @@ int ssl_get_min_max_version(const SSL_CONNECTION *s, int *min_version, if (ssl_method_error(s, method) != 0) { hole = 1; } else if (!hole) { - single = NULL; *min_version = method->version; } else { if (real_max != NULL && tmp_real_max != 0) *real_max = tmp_real_max; - version = (single = method)->version; + version = method->version; *min_version = version; hole = 0; } |