aboutsummaryrefslogtreecommitdiffstats
path: root/hostapd
diff options
context:
space:
mode:
authorJouni Malinen <j@w1.fi>2014-03-09 22:44:26 (GMT)
committerJouni Malinen <j@w1.fi>2014-03-09 22:44:26 (GMT)
commit4fb363c62b0c1fd5e7ba537cd8e419cad831ba70 (patch)
treedd3d2873dc244ade0982885a2e03eb05ac5648cb /hostapd
parent8b56743ef7346d5743b67776e167d6dd54acfd9b (diff)
downloadhostap-4fb363c62b0c1fd5e7ba537cd8e419cad831ba70.zip
hostap-4fb363c62b0c1fd5e7ba537cd8e419cad831ba70.tar.gz
hostap-4fb363c62b0c1fd5e7ba537cd8e419cad831ba70.tar.bz2
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>
Diffstat (limited to 'hostapd')
-rw-r--r--hostapd/config_file.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/hostapd/config_file.c b/hostapd/config_file.c
index 949a9d1..b7d1f2f 100644
--- 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) {