rlm_eap: RERROR type debugs so Module-Failure-Message gets set

diff --git a/doc/ChangeLog b/doc/ChangeLog
index ca9a61d..3e81d23 100644 (file)
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -39,6 +39,7 @@ FreeRADIUS 3.0.13 Thur 29 Sep 2016 13:00:00 EDT urgency=medium
        * Fix OpenSSL fixes.  Fixes #1876.
        * Finish SQL select queries even when SELECT returns no rows.
          Fixes #1879.
+       * Set Module-Failure-Message for more EAP errors.
 
 FreeRADIUS 3.0.12 Thur 29 Sep 2016 13:00:00 EDT urgency=medium
        Feature improvements

diff --git a/src/modules/rlm_eap/mem.c b/src/modules/rlm_eap/mem.c
index c0899f0..7dc907f 100644 (file)
--- a/src/modules/rlm_eap/mem.c
+++ b/src/modules/rlm_eap/mem.c
@@ -436,7 +436,7 @@ eap_handler_t *eaplist_find(rlm_eap_t *inst, REQUEST *request,
         *      Might not have been there.
         */
        if (!handler) {
-               ERROR("rlm_eap (%s): No EAP session matching state "
+               RERROR("rlm_eap (%s): No EAP session matching state "
                       "0x%02x%02x%02x%02x%02x%02x%02x%02x",
                       inst->xlat_name,
                       state->vp_octets[0], state->vp_octets[1],
@@ -447,7 +447,7 @@ eap_handler_t *eaplist_find(rlm_eap_t *inst, REQUEST *request,
        }
 
        if (handler->trips >= 50) {
-               ERROR("rlm_eap (%s): Aborting! More than 50 roundtrips "
+               RERROR("rlm_eap (%s): Aborting! More than 50 roundtrips "
                       "made in session with state "
                       "0x%02x%02x%02x%02x%02x%02x%02x%02x",
                       inst->xlat_name,

diff --git a/src/modules/rlm_eap/types/rlm_eap_peap/peap.c b/src/modules/rlm_eap/types/rlm_eap_peap/peap.c
index 6c885da..cc0927e 100644 (file)
--- a/src/modules/rlm_eap/types/rlm_eap_peap/peap.c
+++ b/src/modules/rlm_eap/types/rlm_eap_peap/peap.c
@@ -876,7 +876,7 @@ rlm_rcode_t eappeap_process(eap_handler_t *handler, tls_session_t *tls_session,
         */
        case PEAP_STATUS_SENT_TLV_FAILURE:
                RINDENT();
-               RIDEBUG("The users session was previously rejected: returning reject (again.)");
+               REDEBUG("The users session was previously rejected: returning reject (again.)");
                RIDEBUG("This means you need to read the PREVIOUS messages in the debug output");
                RIDEBUG("to find out the reason why the user was rejected");
                RIDEBUG("Look for \"reject\" or \"fail\".  Those earlier messages will tell you");