aboutsummaryrefslogtreecommitdiffstats
path: root/src/eap_peer/eap_sim.c
diff options
context:
space:
mode:
authorJouni Malinen <jouni.malinen@atheros.com>2009-06-29 18:42:29 (GMT)
committerJouni Malinen <j@w1.fi>2009-06-29 18:42:29 (GMT)
commitf141be0cafeb327ecec374de52fef3d216af5014 (patch)
tree55b3502d49aaa71dddbaef329e18452589d0b230 /src/eap_peer/eap_sim.c
parent2b16c01c4e9ae01f2d09457ac8aaeed99e57f276 (diff)
downloadhostap-f141be0cafeb327ecec374de52fef3d216af5014.zip
hostap-f141be0cafeb327ecec374de52fef3d216af5014.tar.gz
hostap-f141be0cafeb327ecec374de52fef3d216af5014.tar.bz2
EAP-SIM peer: Remove AT_NOTIFICATION from Notification response
This attribute is not supposed to be used in the response frame (i.e., it is only in the EAP-Request/SIM-Notification frame) per RFC 4186 chapters 10.1 and 9.9. This is a minor bug since the server is required to ignore the contents of the EAP-Response/SIM-Notification during protected result indication per chapter 6.2. EAP-AKA peer was already following the similar specification in RFC 4187, but this was somehow missed in the EAP-SIM peer implementation.
Diffstat (limited to 'src/eap_peer/eap_sim.c')
-rw-r--r--src/eap_peer/eap_sim.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/eap_peer/eap_sim.c b/src/eap_peer/eap_sim.c
index a7e49f8..5e30d1f 100644
--- a/src/eap_peer/eap_sim.c
+++ b/src/eap_peer/eap_sim.c
@@ -468,8 +468,6 @@ static struct wpabuf * eap_sim_response_notification(struct eap_sim_data *data,
wpa_printf(MSG_DEBUG, "Generating EAP-SIM Notification (id=%d)", id);
msg = eap_sim_msg_init(EAP_CODE_RESPONSE, id,
EAP_TYPE_SIM, EAP_SIM_SUBTYPE_NOTIFICATION);
- wpa_printf(MSG_DEBUG, " AT_NOTIFICATION");
- eap_sim_msg_add(msg, EAP_SIM_AT_NOTIFICATION, notification, NULL, 0);
if (k_aut && data->reauth) {
wpa_printf(MSG_DEBUG, " AT_IV");
wpa_printf(MSG_DEBUG, " AT_ENCR_DATA");