diff options
author | Ahmed ElArabawy <arabawy@google.com> | 2018-03-15 16:00:10 (GMT) |
---|---|---|
committer | Jouni Malinen <j@w1.fi> | 2018-03-31 08:57:33 (GMT) |
commit | 45f7574d31b543ef4f65551d095be2b42a32ac77 (patch) | |
tree | 2577288e516d0902ee428794f785df37a4145970 /src/eapol_supp | |
parent | c9aba19bd34af3f04517ca6bf37891d0ecb4d59b (diff) | |
download | hostap-45f7574d31b543ef4f65551d095be2b42a32ac77.zip hostap-45f7574d31b543ef4f65551d095be2b42a32ac77.tar.gz hostap-45f7574d31b543ef4f65551d095be2b42a32ac77.tar.bz2 |
Propagate the EAP method error code
In the current implementation, upon an EAP method failure, followed by
an EAP failure, the EAP Status is propagated up in wpa_supplicant with a
general failure parameter string "failure". This parameter is used for a
notification on the dbus.
This commit reports the EAP method failure error code in a separate
callback.
The solution in this commit is generic to all EAP methods, and can be
used by any method that need to pass its error code. However, this
commit only implements the reporting for EAP-SIM and EAP-AKA methods
where the Notification Code (in AT_NOTIFICATION) is used as the method
specific error code value.
Signed-off-by: Ahmed ElArabawy <arabawy@google.com>
Diffstat (limited to 'src/eapol_supp')
-rw-r--r-- | src/eapol_supp/eapol_supp_sm.c | 10 | ||||
-rw-r--r-- | src/eapol_supp/eapol_supp_sm.h | 7 |
2 files changed, 17 insertions, 0 deletions
diff --git a/src/eapol_supp/eapol_supp_sm.c b/src/eapol_supp/eapol_supp_sm.c index 8e4f0e4..bfbc995 100644 --- a/src/eapol_supp/eapol_supp_sm.c +++ b/src/eapol_supp/eapol_supp_sm.c @@ -2015,6 +2015,15 @@ static void eapol_sm_notify_status(void *ctx, const char *status, } +static void eapol_sm_notify_eap_error(void *ctx, int error_code) +{ + struct eapol_sm *sm = ctx; + + if (sm->ctx->eap_error_cb) + sm->ctx->eap_error_cb(sm->ctx->ctx, error_code); +} + + #ifdef CONFIG_EAP_PROXY static void eapol_sm_eap_proxy_cb(void *ctx) @@ -2062,6 +2071,7 @@ static const struct eapol_callbacks eapol_cb = eapol_sm_eap_param_needed, eapol_sm_notify_cert, eapol_sm_notify_status, + eapol_sm_notify_eap_error, #ifdef CONFIG_EAP_PROXY eapol_sm_eap_proxy_cb, eapol_sm_eap_proxy_notify_sim_status, diff --git a/src/eapol_supp/eapol_supp_sm.h b/src/eapol_supp/eapol_supp_sm.h index a25c799..74f40bb 100644 --- a/src/eapol_supp/eapol_supp_sm.h +++ b/src/eapol_supp/eapol_supp_sm.h @@ -271,6 +271,13 @@ struct eapol_ctx { void (*status_cb)(void *ctx, const char *status, const char *parameter); + /** + * eap_error_cb - Notification of EAP method error + * @ctx: Callback context (ctx) + * @error_code: EAP method error code + */ + void (*eap_error_cb)(void *ctx, int error_code); + #ifdef CONFIG_EAP_PROXY /** * eap_proxy_cb - Callback signifying any updates from eap_proxy |