[PATCH] rules.pl: fix missing blank in 'if'-condition

[PATCH] rules.pl: fix missing blank in 'if'-condition

[Matthias Fischer]

[-- Attachment #1: Type: text/plain, Size: 715 bytes --]

Triggered by https://forum.ipfire.org/viewtopic.php?f=22&t=19304

Best,
Matthias

Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
---
 config/firewall/rules.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/firewall/rules.pl b/config/firewall/rules.pl
index 8b0c6ddc8..efbe9cfef 100644
--- a/config/firewall/rules.pl
+++ b/config/firewall/rules.pl
@@ -608,7 +608,7 @@ sub geoipblock {
 	# create iptables rules, if blocking this country
 	# is enabled.
 	foreach my $location (@locations) {
-		if($geoipsettings{$location} eq "on") {
+		if ($geoipsettings{$location} eq "on") {
 			run("$IPTABLES -A GEOIPBLOCK -m geoip --src-cc $location -j DROP");
 		}
 	}
-- 
2.14.1

Re: [PATCH] rules.pl: fix missing blank in 'if'-condition

[Michael Tremer]

[-- Attachment #1: Type: text/plain, Size: 879 bytes --]

Hi,

this has been set to deferred on Patchwork. Why?

Best,
-Michael

On Sat, 2017-08-19 at 13:30 +0200, Matthias Fischer wrote:
> Triggered by https://forum.ipfire.org/viewtopic.php?f=22&t=19304
> 
> Best,
> Matthias
> 
> Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
> ---
>  config/firewall/rules.pl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/config/firewall/rules.pl b/config/firewall/rules.pl
> index 8b0c6ddc8..efbe9cfef 100644
> --- a/config/firewall/rules.pl
> +++ b/config/firewall/rules.pl
> @@ -608,7 +608,7 @@ sub geoipblock {
>  	# create iptables rules, if blocking this country
>  	# is enabled.
>  	foreach my $location (@locations) {
> -		if($geoipsettings{$location} eq "on") {
> +		if ($geoipsettings{$location} eq "on") {
>  			run("$IPTABLES -A GEOIPBLOCK -m geoip --src-cc $location -j DROP");
>  		}
>  	}

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] rules.pl: fix missing blank in 'if'-condition

[Matthias Fischer]

[-- Attachment #1: Type: text/plain, Size: 1273 bytes --]

Hi,

For unknown reasons I couldn't reproduce this error even with the
original file. So I deferred this patch - I'm not sure anymore that this
is really a bug!?

The whole story (german) can be found here:
https://forum.ipfire.org/viewtopic.php?f=22&t=19304

I fear I was a bit too fast...

Best,
Matthias

On 21.08.2017 13:17, Michael Tremer wrote:
> Hi,
> 
> this has been set to deferred on Patchwork. Why?
> 
> Best,
> -Michael
> 
> On Sat, 2017-08-19 at 13:30 +0200, Matthias Fischer wrote:
>> Triggered by https://forum.ipfire.org/viewtopic.php?f=22&t=19304
>> 
>> Best,
>> Matthias
>> 
>> Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
>> ---
>>  config/firewall/rules.pl | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/config/firewall/rules.pl b/config/firewall/rules.pl
>> index 8b0c6ddc8..efbe9cfef 100644
>> --- a/config/firewall/rules.pl
>> +++ b/config/firewall/rules.pl
>> @@ -608,7 +608,7 @@ sub geoipblock {
>>  	# create iptables rules, if blocking this country
>>  	# is enabled.
>>  	foreach my $location (@locations) {
>> -		if($geoipsettings{$location} eq "on") {
>> +		if ($geoipsettings{$location} eq "on") {
>>  			run("$IPTABLES -A GEOIPBLOCK -m geoip --src-cc $location -j DROP");
>>  		}
>>  	}
>