Again, problems with RADIUS CoA affecting only Aruba and AeroHIVE this time.

The fix is trivial, see attached patch.

A regression test was added to prevent problems like this to re-occur in
the future.
-- 
Olivier Bilodeau
obilod...@inverse.ca  ::  +1.514.447.4918 *115  ::  www.inverse.ca
Inverse inc. :: Leaders behind SOGo (www.sogo.nu) and PacketFence
(www.packetfence.org)
--- a/lib/pf/SNMP.pm
+++ b/lib/pf/SNMP.pm
@@ -2651,6 +2651,9 @@ sub radiusDisconnect {
     my ($self, $mac, $add_attributes_ref) = @_;
     my $logger = Log::Log4perl::get_logger( ref($self) );
 
+    # initialize
+    $add_attributes_ref = {} if (!defined($add_attributes_ref));
+
     if (!defined($self->{'_radiusSecret'})) {
         $logger->warn(
             "Unable to perform RADIUS Disconnect-Request on $self->{'_ip'}: RADIUS Shared Secret not configured"
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
PacketFence-devel mailing list
PacketFence-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/packetfence-devel

Reply via email to