Correct evaluation for "Acct-Interim-Interval" from RADIUS



  • There is a patch submitted for review that correct the evaluation for "Acct-Interim-Interval" from RADIUS.
    BUG: Setting "Acct-Interim-Interval :=600" in FreeRadius2 evaluates to 'random' number with PfSense 2.1.
    Possibly a bug related to:
    https://forum.pfsense.org/index.php?topic=60079.0
    https://forum.pfsense.org/index.php?topic=60262.0

    The github pull request is https://github.com/pfsense/pfsense/pull/1285

    I'm using this patched version of radius.inc in production environment successfully.

    Thank you_

    
    --- /etc/inc/radius.inc.orig    2014-09-19 10:16:10.000000000 +0200
    +++ /etc/inc/radius.inc 2014-09-19 10:17:00.000000000 +0200
    @@ -686,7 +686,7 @@
                     break;
    
                 case 85: /* Acct-Interim-Interval: RFC 2869 */
    -                $this->attributes['interim_interval'] = radius_cvt_int($datav[1]);
    +                $this->attributes['interim_interval'] = radius_cvt_int($data);
                     break;
                 }
             }
    
    


  • It has been merged already


Log in to reply