[RADIATOR] Bug in SessSQL.pm

[Mike McCauley]

Hi Eddie,

thanks for reporting this.
It has now been fixed in the latest patch set.

Cheers.

On Tuesday, February 14, 2012 03:53:04 PM Eddie Stassen wrote:
> UpdateQuery <SessionDatabase SQL> crashes Radiator when the query
> contains %{Quote:...}.  This is due to $self not being passed as the
> third parameter to Radius::Util::format_special(). The patch below
> fixes it.
> 
> Regards,
> Eddie Stassen
> 
> --- SessSQL.pm.ORIG     2012-02-14 15:32:12.000000000 +0200
> +++ SessSQL.pm  2012-02-14 15:44:42.000000000 +0200
> @@ -132,7 +132,9 @@
>      $self->log($main::LOG_DEBUG,
>                "$self->{Identifier} Updating session for $name,
> $nas_id, $nas_port", $p);
>      # Now add the new one
> -    $self->do(&Radius::Util::format_special($self->{UpdateQuery}, $p));
> +    $self->do(&$self->{UpdateQuery}, $p, $self,
> +            $self->quote($name), $nas_id, $nas_port+0,
> +           
> $self->quote($p->getAttrByNum($Radius::Radius::ACCT_SESSION_ID)))); }
> _______________________________________________
> radiator mailing list
> radiator at open.com.au
> http://www.open.com.au/mailman/listinfo/radiator
-- 
Mike McCauley                               mikem at open.com.au
Open System Consultants Pty. Ltd
9 Bulbul Place Currumbin Waters QLD 4223 Australia   http://www.open.com.au
Phone +61 7 5598-7474                       Fax   +61 7 5598-7070

Radiator: the most portable, flexible and configurable RADIUS server 
anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald, 
Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS, 
TTLS, PEAP, TNC, WiMAX, RSA, Vasco, Yubikey, MOTP, HOTP, TOTP,
DIAMETER etc. Full source on Unix, Windows, MacOSX, Solaris, VMS, NetWare etc.