From 68dd6726af4cc47056353f51ebb1c5e92eb68326 Mon Sep 17 00:00:00 2001
From: Christian Schmidt <christian.schmidt@ipfire.org>
Date: Tue, 12 Oct 2010 17:38:40 +0200
Subject: [PATCH] Fixed the feature for deactivating outgoingfw rules.

---
 config/outgoingfw/outgoingfw.pl          | 2 +-
 config/rootfiles/core/41/filelists/files | 6 ++++--
 html/cgi-bin/outgoingfw.cgi              | 8 ++++++--
 3 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/config/outgoingfw/outgoingfw.pl b/config/outgoingfw/outgoingfw.pl
index 8d4d27d313..6709ee603f 100644
--- a/config/outgoingfw/outgoingfw.pl
+++ b/config/outgoingfw/outgoingfw.pl
@@ -187,7 +187,7 @@ foreach $configentry (sort @configs)
 			foreach $SOURCE (@SOURCE) {
 				$SOURCE =~ s/\s//gi;
 
-				if ( $SOURCE eq "" ){next;}
+				if ( $SOURCE eq "" || $configline[1] eq "" ){next;}
 
 				if ( ( $configline[6] ne "" || $configline[2] eq 'mac' ) && $configline[2] ne 'all'){
 					$SOURCE =~ s/[^a-zA-Z0-9]/:/gi;
diff --git a/config/rootfiles/core/41/filelists/files b/config/rootfiles/core/41/filelists/files
index 59e2f539f5..068f2ba68a 100644
--- a/config/rootfiles/core/41/filelists/files
+++ b/config/rootfiles/core/41/filelists/files
@@ -1,8 +1,10 @@
 etc/system-release
 srv/web/ipfire/cgi-bin/outgoinggrp.cgi
+srv/web/ipfire/cgi-bin/outgoingfw.cgi
+srv/web/ipfire/cgi-bin/urlfilter.cgi
+srv/web/ipfire/cgi-bin/vpnmain.cgi
 var/ipfire/langs/en.pl
 var/ipfire/langs/de.pl
 var/ipfire/langs/es.pl
 var/ipfire/langs/fr.pl
-srv/web/ipfire/cgi-bin/urlfilter.cgi
-srv/web/ipfire/cgi-bin/vpnmain.cgi
+var/ipfire/outgoingfw/bin/outgoingfw.pl
diff --git a/html/cgi-bin/outgoingfw.cgi b/html/cgi-bin/outgoingfw.cgi
index 27a8927e6c..a15b243c05 100644
--- a/html/cgi-bin/outgoingfw.cgi
+++ b/html/cgi-bin/outgoingfw.cgi
@@ -408,8 +408,12 @@ END
 				if ($outfwsettings{'STATE'} eq 'DENY'){ $outfwsettings{'DISPLAY_STATE'} = "<img src='/images/stock_stop.png' alt='DENY' />"; }
 				if ($outfwsettings{'STATE'} eq 'ALLOW'){ $outfwsettings{'DISPLAY_STATE'} = "<img src='/images/stock_ok.png' alt='ALLOW' />"; }
 				if ((($outfwsettings{'POLICY'} eq 'MODE1') && ($outfwsettings{'STATE'} eq 'ALLOW')) || (($outfwsettings{'POLICY'} eq 'MODE2') && ($outfwsettings{'STATE'} eq 'DENY'))){
-			  		print <<END
-					<tr bgcolor='$color{'color20'}'>
+				if ( $outfwsettings{'ENABLED'} eq "on" ){
+					print "<tr bgcolor='$color{'color20'}'>";
+				} else {
+					print "<tr bgcolor='$color{'color18'}'>";
+				}
+					print <<END
 					    <td align='center'>$outfwsettings{'PROT'}
 					    <td align='center'>$outfwsettings{'SNET'}
 					    <td align='center'>$outfwsettings{'DISPLAY_DIP'}:$outfwsettings{'DISPLAY_DPORT'}
-- 
2.39.2