From c4b8c71ba41b018ecf6c03a71c5526b94806197b Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Sun, 5 Jan 2014 14:18:38 +0200 Subject: [PATCH] EAP-GPSK: Report CSuite negotiation failure properly Setting methodState = DONE for the case where GPSK-1 is found to be invalid or incompatible allows EAP state machine to proceed to FAILURE state instead of remaining stuck until AP times out the connection. Signed-hostap: Jouni Malinen --- src/eap_peer/eap_gpsk.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/eap_peer/eap_gpsk.c b/src/eap_peer/eap_gpsk.c index 6f7b23b2b..5b023c793 100644 --- a/src/eap_peer/eap_gpsk.c +++ b/src/eap_peer/eap_gpsk.c @@ -289,6 +289,7 @@ static struct wpabuf * eap_gpsk_process_gpsk_1(struct eap_sm *sm, pos = eap_gpsk_process_csuite_list(sm, data, &csuite_list, &csuite_list_len, pos, end); if (pos == NULL) { + ret->methodState = METHOD_DONE; eap_gpsk_state(data, FAILURE); return NULL; }