On 5/30/11 8:17 AM, Steven Jan Springl wrote:

> 
> I confirm the patch has fixed the issue.
> 
> The following message is now being produced from the config.:
> 
> Use of uninitialized value $disposition in pattern match (m//) 
> at /usr/share/shorewall/Shorewall/Misc.pm line 802.

I managed to overlook that in the output when I tested. The attached
removes this second defect.

-Tom
-- 
Tom Eastep        \ When I die, I want to go like my Grandfather who
Shoreline,         \ died peacefully in his sleep. Not screaming like
Washington, USA     \ all of the passengers in his car
http://shorewall.net \________________________________________________
diff --git a/Shorewall/Perl/Shorewall/Config.pm 
b/Shorewall/Perl/Shorewall/Config.pm
index f2ef1bd..7694827 100644
--- a/Shorewall/Perl/Shorewall/Config.pm
+++ b/Shorewall/Perl/Shorewall/Config.pm
@@ -3403,7 +3403,8 @@ sub get_configuration( $ ) {
 
        require_capability 'AUDIT_TARGET' , "MACLIST_DISPOSITION=$val", 's' if 
$val =~ /^A_/;
     } else {
-       $globals{MACLIST_TARGET} = 'reject';
+       $config{MACLIST_DISPOSITION}  = 'REJECT';
+       $globals{MACLIST_TARGET}      = 'reject';
     }
 
     if ( $val = $config{MACLIST_TABLE} ) {

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
vRanger cuts backup time in half-while increasing security.
With the market-leading solution for virtual backup and recovery, 
you get blazing-fast, flexible, and affordable data protection.
Download your free trial now. 
http://p.sf.net/sfu/quest-d2dcopy1
_______________________________________________
Shorewall-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/shorewall-devel

Reply via email to