]> git.ipfire.org Git - thirdparty/hostap.git/commitdiff
EAP-GPSK server: Fix memory freeing on error path
authorJouni Malinen <j@w1.fi>
Fri, 19 Apr 2019 14:08:27 +0000 (17:08 +0300)
committerJouni Malinen <j@w1.fi>
Fri, 19 Apr 2019 14:08:27 +0000 (17:08 +0300)
Allocated struct wpabuf was freed with incorrect freeing function. Fix
this by using the appropriate function.

Signed-off-by: Jouni Malinen <j@w1.fi>
src/eap_server/eap_server_gpsk.c

index fb3d11748c8c2751020b8e38fe72fb57b8972a07..bebb17f40aaa59c214546bdb4ee17d2b18f05190 100644 (file)
@@ -181,7 +181,7 @@ static struct wpabuf * eap_gpsk_build_gpsk_3(struct eap_sm *sm,
        if (eap_gpsk_compute_mic(data->sk, data->sk_len, data->vendor,
                                 data->specifier, start, pos - start, pos) < 0)
        {
-               os_free(req);
+               wpabuf_free(req);
                eap_gpsk_state(data, FAILURE);
                return NULL;
        }
@@ -379,7 +379,7 @@ static void eap_gpsk_process_gpsk_2(struct eap_sm *sm,
        data->specifier = WPA_GET_BE16(csuite->specifier);
        wpa_printf(MSG_DEBUG, "EAP-GPSK: CSuite_Sel %d:%d",
                   data->vendor, data->specifier);
-       pos += sizeof(*csuite); 
+       pos += sizeof(*csuite);
 
        if (end - pos < 2) {
                wpa_printf(MSG_DEBUG, "EAP-GPSK: Too short message for "