Fix error path handling on radius_accept_attr

This error path must not try to free the user entry since that entry was
already added to the BSS list and will be freed when BSS is freed.

Signed-off-by: Jouni Malinen <j@w1.fi>

diff --git a/hostapd/config_file.c b/hostapd/config_file.c
index 949a9d171d6db543a394d07a315d827b5dc33f20..b7d1f2f1432bb62d19807a1690608aecd7c254ff 100644 (file)
--- a/hostapd/config_file.c
+++ b/hostapd/config_file.c
@@ -257,6 +257,7 @@ static int hostapd_config_read_eap_user(const char *fname,
                        if (attr == NULL) {
                                wpa_printf(MSG_ERROR, "Invalid radius_auth_req_attr: %s",
                                           buf + 19);
+                               user = NULL; /* already in the BSS list */
                                goto failed;
                        }
                        if (user->accept_attr == NULL) {