From b1b2aa910ed0bcb79c0498b7dc9322a92a2bcb30 Mon Sep 17 00:00:00 2001
From: Tom Eastep <teastep@shorewall.net>
Date: Sun, 27 Jan 2013 10:08:02 -0800
Subject: [PATCH] Correct section handling:

- Correct typo (' INVALID' -> 'INVALID' )
- Don't jump to non-existent target in finish_chain_section()

Signed-off-by: Tom Eastep <teastep@shorewall.net>
---
 Shorewall/Perl/Shorewall/Rules.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Shorewall/Perl/Shorewall/Rules.pm b/Shorewall/Perl/Shorewall/Rules.pm
index 3e81c1501..644e347e1 100644
--- a/Shorewall/Perl/Shorewall/Rules.pm
+++ b/Shorewall/Perl/Shorewall/Rules.pm
@@ -896,7 +896,7 @@ sub finish_chain_section ($$$) {
 		}
 
 		if ( $twochains ) {
-		    add_ijump $chainref, g => $target;
+		    add_ijump $chainref, g => $target if $target;
 		    %state = ();
 		    last;
 		}
@@ -2613,7 +2613,7 @@ sub process_section ($) {
 	finish_section ( 'ESTABLISHED,RELATED' );
     } elsif ( $sect eq 'UNTRACKED' ) {
 	@sections{'ALL','ESTABLISHED','RELATED', 'INVALID' } = ( 1, 1, 1, 1 );
-	finish_section ( 'ESTABLISHED,RELATED, INVALID' );
+	finish_section ( 'ESTABLISHED,RELATED,INVALID' );
     } elsif ( $sect eq 'NEW' ) {
 	@sections{'ALL','ESTABLISHED','RELATED','INVALID','UNTRACKED', 'NEW'} = ( 1, 1, 1, 1, 1, 1 );
 	finish_section ( 'ESTABLISHED,RELATED,INVALID,UNTRACKED' );