aboutsummaryrefslogtreecommitdiffstats
path: root/src/eap_server/eap_server_sim.c
diff options
context:
space:
mode:
authorJouni Malinen <jouni@qca.qualcomm.com>2016-01-13 21:35:53 (GMT)
committerJouni Malinen <j@w1.fi>2016-01-13 21:35:53 (GMT)
commit814f43cff5e366f3a6a6452666629d73c83161a1 (patch)
tree1383c7153508bb6120ec754142a0551022ed0086 /src/eap_server/eap_server_sim.c
parent49a26bb3e37ef7d3c9b8b55a5a331ae64a1cd0f3 (diff)
downloadhostap-814f43cff5e366f3a6a6452666629d73c83161a1.zip
hostap-814f43cff5e366f3a6a6452666629d73c83161a1.tar.gz
hostap-814f43cff5e366f3a6a6452666629d73c83161a1.tar.bz2
EAP server: Simplify EAP method registration call
Free the allocated structure in error cases to remove need for each EAP method to handle the error cases separately. Each registration function can simply do "return eap_server_method_register(eap);" in the end of the function. Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
Diffstat (limited to 'src/eap_server/eap_server_sim.c')
-rw-r--r--src/eap_server/eap_server_sim.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/src/eap_server/eap_server_sim.c b/src/eap_server/eap_server_sim.c
index ddfb71c..3a6ed79 100644
--- a/src/eap_server/eap_server_sim.c
+++ b/src/eap_server/eap_server_sim.c
@@ -846,7 +846,6 @@ static u8 * eap_sim_get_session_id(struct eap_sm *sm, void *priv, size_t *len)
int eap_server_sim_register(void)
{
struct eap_method *eap;
- int ret;
eap = eap_server_method_alloc(EAP_SERVER_METHOD_INTERFACE_VERSION,
EAP_VENDOR_IETF, EAP_TYPE_SIM, "SIM");
@@ -864,8 +863,5 @@ int eap_server_sim_register(void)
eap->get_emsk = eap_sim_get_emsk;
eap->getSessionId = eap_sim_get_session_id;
- ret = eap_server_method_register(eap);
- if (ret)
- eap_server_method_free(eap);
- return ret;
+ return eap_server_method_register(eap);
}