From add0068afc3b732c27c9cc116d7ec331f9a32735 Mon Sep 17 00:00:00 2001 From: "Alan T. DeKok" Date: Sun, 24 May 2009 18:19:43 +0200 Subject: [PATCH] Check for misconfigured systems --- src/modules/rlm_eap/rlm_eap.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/modules/rlm_eap/rlm_eap.c b/src/modules/rlm_eap/rlm_eap.c index 44ab2f3..8dc12f7 100644 --- a/src/modules/rlm_eap/rlm_eap.c +++ b/src/modules/rlm_eap/rlm_eap.c @@ -511,6 +511,11 @@ static int eap_post_proxy(void *inst, REQUEST *request) EAP_HANDLER *handler; /* + * Just in case the admin lists EAP in post-proxy-type Fail. + */ + if (!request->proxy_reply) return RLM_MODULE_NOOP; + + /* * If there was a handler associated with this request, * then it's a tunneled request which was proxied... */ @@ -548,7 +553,7 @@ static int eap_post_proxy(void *inst, REQUEST *request) * We are done, wrap the EAP-request in RADIUS to send * with all other required radius attributes */ - rcode = eap_compose(handler); + eap_compose(handler); /* * Add to the list only if it is EAP-Request, OR if @@ -592,7 +597,6 @@ static int eap_post_proxy(void *inst, REQUEST *request) RDEBUG2("No pre-existing handler found"); } - /* * There may be more than one Cisco-AVPair. * Ensure we find the one with the LEAP attribute. -- 2.1.4