remove unnecessary function, and fix error message

FreeRADIUS · Mar 23, 2018 · 81fcb84 · 81fcb84
1 parent 174fa66
commit 81fcb84
Showing 1 changed file with 7 additions and 15 deletions.
diff --git a/src/modules/rlm_eap/types/rlm_eap_sim/rlm_eap_sim.c b/src/modules/rlm_eap/types/rlm_eap_sim/rlm_eap_sim.c
@@ -43,20 +43,6 @@ typedef struct eap_sim_server_state {
 	int  sim_id;
 } eap_sim_state_t;
 
-/*
- *	build a reply to be sent.
- */
-static void eap_sim_compose(REQUEST *request, eap_handler_t *handler)
-{
-	/* we will set the ID on requests, since we have to HMAC it */
-	handler->eap_ds->set_request_id = 1;
-
-	if (!map_eapsim_basictypes(handler->request->reply,
-				   handler->eap_ds->request)) {
-		REDEBUG("Failed decoding EAP-SIM packet: %s", fr_strerror());
-	}
-}
-
 static int eap_sim_sendstart(eap_handler_t *handler)
 {
 	VALUE_PAIR **vps, *newvp;
@@ -454,7 +440,13 @@ static void eap_sim_state_enter(REQUEST *request, eap_handler_t *handler,
 	ess->state = newstate;
 
 	/* build the target packet */
-	eap_sim_compose(request, handler);
+	/* we will set the ID on requests, since we have to HMAC it */
+	handler->eap_ds->set_request_id = 1;
+
+	if (!map_eapsim_basictypes(handler->request->reply,
+				   handler->eap_ds->request)) {
+		REDEBUG("Failed encoding EAP-SIM packet");
+	}
 }
 
 /*