From 6f577c5e4549a4f2d90d93349a1fff4d9dcbc11e Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Sat, 11 Oct 2014 18:11:14 +0300 Subject: [PATCH] EAP-FAST server: Remove unused assignment Commit e8c08c9a363340c45baf8e13c758c99078bc0d8b ('EAP-FAST server: Fix potential read-after-buffer (by one byte)') changed the while loop design in a way that does not require the pos variable to be updated anymore. Remove that unneeded code to clean up static analyzer warnings about unused assignments. Signed-off-by: Jouni Malinen --- src/eap_server/eap_server_fast.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/eap_server/eap_server_fast.c b/src/eap_server/eap_server_fast.c index d5bee2203..c6bf6de8f 100644 --- a/src/eap_server/eap_server_fast.c +++ b/src/eap_server/eap_server_fast.c @@ -186,7 +186,6 @@ static int eap_fast_session_ticket_cb(void *ctx, const u8 *ticket, size_t len, switch (*pos) { case PAC_OPAQUE_TYPE_PAD: - pos = end; goto done; case PAC_OPAQUE_TYPE_KEY: if (pos[1] != EAP_FAST_PAC_KEY_LEN) {