From be8013a8b20b6e5f21b630be7d2446a916cbf5cf Mon Sep 17 00:00:00 2001
From: teastep <teastep@fbd18981-670d-0410-9b5c-8dc0c1a9a2bb>
Date: Wed, 13 Jun 2007 23:50:26 +0000
Subject: [PATCH] Don't increase MSS

git-svn-id: https://shorewall.svn.sourceforge.net/svnroot/shorewall/trunk@6539 fbd18981-670d-0410-9b5c-8dc0c1a9a2bb
---
 Shorewall-common/lib.config        |  2 +-
 Shorewall-common/releasenotes.txt  |  2 +-
 Shorewall-perl/Shorewall/Chains.pm |  2 +-
 Shorewall-perl/Shorewall/Rules.pm  | 12 ++++++++++--
 4 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/Shorewall-common/lib.config b/Shorewall-common/lib.config
index 0e6f3607d..cd51b4187 100644
--- a/Shorewall-common/lib.config
+++ b/Shorewall-common/lib.config
@@ -413,7 +413,7 @@ setup_ipsec() {
 
 	if [ "$policy" != NONE ]; then
 	    ensurechain $1
-	    run_iptables -I $1 -p tcp --tcp-flags SYN,RST SYN -j TCPMSS --set-mss $2
+	    run_iptables -I $1 -p tcp --tcp-flags SYN,RST SYN -m tcpmss --mss $2: -j TCPMSS --set-mss $2
 	fi
     }
     #
diff --git a/Shorewall-common/releasenotes.txt b/Shorewall-common/releasenotes.txt
index 467d00b34..aba445e30 100644
--- a/Shorewall-common/releasenotes.txt
+++ b/Shorewall-common/releasenotes.txt
@@ -36,7 +36,7 @@ Other changes in Shorewall 4.0.0 Beta 5.
 
     The compiler program is /usr/share/shorewall-perl/compiler.pl:
 
-	compiler.pl [ <options> ] [ <filename> ]
+	compiler.pl [ <option> ... ] [ <filename> ]
 
     If a <filename> is given, then the configuration will be compiled
     output placed in the named file. If <filename> is not given, then
diff --git a/Shorewall-perl/Shorewall/Chains.pm b/Shorewall-perl/Shorewall/Chains.pm
index 5d9c61720..e839f9c93 100644
--- a/Shorewall-perl/Shorewall/Chains.pm
+++ b/Shorewall-perl/Shorewall/Chains.pm
@@ -699,7 +699,7 @@ sub set_mss1( $$ ) {
     my $chainref = ensure_chain 'filter', $chain;
 
     if ( $chainref->{policy} ne 'NONE' ) {
-	insert_rule $chainref, 1, "-p tcp --tcp-flags SYN,RST SYN -j TCPMSS --set-mss $mss"
+	insert_rule $chainref, 1, "-p tcp --tcp-flags SYN,RST SYN -m tcpmss --mss $mss: -j TCPMSS --set-mss $mss"
     }
 }
 
diff --git a/Shorewall-perl/Shorewall/Rules.pm b/Shorewall-perl/Shorewall/Rules.pm
index b1093fb07..c265c7b7b 100644
--- a/Shorewall-perl/Shorewall/Rules.pm
+++ b/Shorewall-perl/Shorewall/Rules.pm
@@ -1777,9 +1777,17 @@ sub generate_matrix() {
 
 sub setup_mss( $ ) {
     my $clampmss = $_[0];
-    my $option = "\L$clampmss" eq 'yes' ? '--clamp-mss-to-pmtu' : '--set-mss $clampmss';
+    my $option;
+    my $match = '';
 
-    add_rule $filter_table->{FORWARD} , "-p tcp --tcp-flags SYN,RST SYN -j TCPMSS $option";
+    if ( "\L$clampmss" eq 'yes' ) {
+	$option = '--clamp-mss-to-pmtu';
+    } else {
+	$match  = "-m tcpmss --mss $clampmss: ";
+	$option = '--set-mss $clampmss';
+    }
+    
+    add_rule $filter_table->{FORWARD} , "-p tcp --tcp-flags SYN,RST SYN ${match}-j TCPMSS $option";
 }
 
 sub dump_rule_chains() {