From a27a3af0c29de8e0c4ea307fd23b98b0b8ac489b Mon Sep 17 00:00:00 2001
From: Luke Howard <lukeh@padl.com>
Date: Tue, 10 Aug 2021 20:11:35 +1000
Subject: [PATCH] gss: store_spnego_context(): remove redundant overflow check

ctx->negoex_step is now a uint8_t, so it can never be bigger than 0xff. Remove
redundant overflow check.
---
 lib/gssapi/spnego/context_storage.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/lib/gssapi/spnego/context_storage.c b/lib/gssapi/spnego/context_storage.c
index 3628488e9..d1ea927e8 100644
--- a/lib/gssapi/spnego/context_storage.c
+++ b/lib/gssapi/spnego/context_storage.c
@@ -296,10 +296,6 @@ store_spnego_context(krb5_storage *sp, gssspnego_ctx ctx)
         uint32_t nschemes;
         struct negoex_auth_mech *mech;
 
-        if (ctx->negoex_step > 0xff) {
-            ret = ERANGE;
-            goto fail;
-        }
         CHECK(ret, krb5_store_uint8(sp, ctx->negoex_step));
 
         if (ctx->negoex_transcript) {