On 2/14/11 9:59 AM, Tom Eastep wrote:

> 
> This one-liner seems to remove the bug.
> 

Unfortunately, there was a bug in the one-liner.  Fix attached.

-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/Chains.pm 
b/Shorewall/Perl/Shorewall/Chains.pm
index 75f1265..6c2c1f8 100644
--- a/Shorewall/Perl/Shorewall/Chains.pm
+++ b/Shorewall/Perl/Shorewall/Chains.pm
@@ -1896,7 +1896,7 @@ sub optimize_level4( $$ ) {
                    #
                    # Chain has a single rule
                    #
-                   if ( $firstrule =~ /^-A -[jg] ([^\s])(\s+-m comment 
.*)?\s*$/ ) {
+                   if ( $firstrule =~ /^-A -[jg] ([^\s]+)(\s+-m comment 
.*)?\s*$/ ) {
                        #
                        # Easy case -- the rule is a simple jump
                        #

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb
_______________________________________________
Shorewall-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/shorewall-devel

Reply via email to