Use correct return codes and variable types
authorAlan T. DeKok <aland@freeradius.org>
Sat, 2 Mar 2013 20:55:01 +0000 (12:55 -0800)
committerAlan T. DeKok <aland@freeradius.org>
Sat, 2 Mar 2013 20:56:15 +0000 (12:56 -0800)
src/modules/rlm_eap/eap.c
src/modules/rlm_eap/rlm_eap.c
src/modules/rlm_eap/rlm_eap.h

index 9a02fa3..e2855ef 100644 (file)
@@ -429,7 +429,7 @@ int eaptype_select(rlm_eap_t *inst, EAP_HANDLER *handler)
  *     Set the RADIUS reply codes based on EAP request codes.  Append
  *     any additonal VPs to RADIUS reply
  */
-int eap_compose(EAP_HANDLER *handler)
+rlm_rcode_t eap_compose(EAP_HANDLER *handler)
 {
        VALUE_PAIR *vp;
        eap_packet_t *eap_packet;
index 1eb2807..e213124 100644 (file)
@@ -272,7 +272,8 @@ static rlm_rcode_t eap_authenticate(void *instance, REQUEST *request)
        rlm_eap_t       *inst;
        EAP_HANDLER     *handler;
        eap_packet_t    *eap_packet;
-       rlm_rcode_t     rcode;
+       int             rcode;
+       rlm_rcode_t     module_rcode;
 
        inst = (rlm_eap_t *) instance;
 
@@ -392,7 +393,7 @@ static rlm_rcode_t eap_authenticate(void *instance, REQUEST *request)
         *      We are done, wrap the EAP-request in RADIUS to send
         *      with all other required radius attributes
         */
-       rcode = eap_compose(handler);
+       module_rcode = eap_compose(handler);
 
        /*
         *      Add to the list only if it is EAP-Request, OR if
@@ -471,7 +472,7 @@ static rlm_rcode_t eap_authenticate(void *instance, REQUEST *request)
                }
        }
 
-       return rcode;
+       return module_rcode;
 }
 
 /*
index 6c3eb4c..73649a9 100644 (file)
@@ -95,7 +95,7 @@ int           eaptype_select(rlm_eap_t *inst, EAP_HANDLER *h);
 int            eap_start(rlm_eap_t *inst, REQUEST *request);
 void           eap_fail(EAP_HANDLER *handler);
 void           eap_success(EAP_HANDLER *handler);
-int            eap_compose(EAP_HANDLER *handler);
+rlm_rcode_t    eap_compose(EAP_HANDLER *handler);
 EAP_HANDLER    *eap_handler(rlm_eap_t *inst, eap_packet_t **eap_msg, REQUEST *request);
 
 /* Memory Management */