From dd1885de11bd2192037f0faf35164426e1559b35 Mon Sep 17 00:00:00 2001 From: James Prestwood Date: Tue, 14 Aug 2018 11:16:10 -0700 Subject: [PATCH] sae: remove double sc increment The send confirm counter is incremented before calling sae_send_confirm in all cases, but the function itself was also incrementing sc after sending the packet. This isn't critical to the successful execution of SAE as the AP just uses the sc value in the packet but it did violate the 802.11 spec. --- src/sae.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/sae.c b/src/sae.c index 5777510d..ed945f26 100644 --- a/src/sae.c +++ b/src/sae.c @@ -468,8 +468,6 @@ static void sae_send_confirm(struct sae_sm *sm) memcpy(ptr, confirm, 32); ptr += 32; - sm->sc++; - sm->state = SAE_STATE_CONFIRMED; sm->tx(sm->handshake->aa, body, 38, sm->user_data);