]> git.ipfire.org Git - people/ms/ipfire-2.x.git/commitdiff
firewall: Avoid creating a rule that permits all traffic on invalid source firewall-uninitialized
authorMichael Tremer <michael.tremer@ipfire.org>
Tue, 17 Jan 2023 13:23:58 +0000 (14:23 +0100)
committerMichael Tremer <michael.tremer@ipfire.org>
Tue, 17 Jan 2023 13:23:58 +0000 (14:23 +0100)
The firewall engine generated rules that did not have any traffic
selectors due to an improperly initialized variable in the source.

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
config/firewall/rules.pl

index 30ad20df1d9a31ff86a08784364daf3c24e192dc..1b628df169948f7771005c4a430334ef3af572a4 100644 (file)
@@ -401,6 +401,9 @@ sub buildrules {
                                        $source = "";
                                }
 
+                               # Make sure that $source is properly defined
+                               next unless (defined $source);
+
                                my $source_intf = @$src[1];
 
                                foreach my $dst (@destinations) {