use warnings;
use Data::Dumper;
use PVE::Firewall;
+use Getopt::Long;
+use File::Basename;
+use Net::IP;
my $mark;
my $trace;
-sub rule_match {
- my ($chain, $rule, $pkg) = @_;
+my $debug = 0;
- $rule =~ s/^-A $chain // || die "got strange rule: $rule";
+sub print_usage_and_exit {
+ die "usage: $0 [--debug] [testfile [testid]]\n";
+}
- if ($rule =~ s/^-m conntrack\s*//) {
- return undef; # simply ignore
- }
+if (!GetOptions ('debug' => \$debug)) {
+ print_usage_and_exit();
+}
- if ($rule =~ s/^-m addrtype\s*//) {
- return undef; # simply ignore
- }
+my $testfilename = shift;
+my $testid = shift;
- if ($rule =~ s/^-i (\S+)\s*//) {
- die "missing iface_in" if !$pkg->{iface_in};
- return undef if $pkg->{iface_in} ne $1; # no match
- }
- if ($rule =~ s/^-o (\S+)\s*//) {
- die "missing iface_out" if !$pkg->{iface_out};
- return undef if $pkg->{iface_out} ne $1; # no match
- }
+sub add_trace {
+ my ($text) = @_;
- if ($rule =~ s/^-p (tcp|udp)\s*//) {
- die "missing proto" if !$pkg->{proto};
- return undef if $pkg->{proto} ne $1; # no match
+ if ($debug) {
+ print $text;
+ } else {
+ $trace .= $text;
}
+}
- if ($rule =~ s/^--dport (\d+):(\d+)\s*//) {
- die "missing dport" if !$pkg->{dport};
- return undef if ($pkg->{dport} < $1) || ($pkg->{dport} > $2); # no match
- }
+sub nf_dev_match {
+ my ($devre, $dev) = @_;
- if ($rule =~ s/^--dport (\d+)\s*//) {
- die "missing dport" if !$pkg->{dport};
- return undef if $pkg->{dport} != $1; # no match
- }
+ $devre =~ s/\+$/\.\*/;
+ return ($dev =~ m/^${devre}$/) ? 1 : 0;
+}
+
+sub ipset_match {
+ my ($ipsetname, $ipset, $ipaddr) = @_;
+
+ my $ip = Net::IP->new($ipaddr);
- if ($rule =~ s/^-s (\S+)\s*//) {
- die "missing source" if !$pkg->{source};
- return undef if $pkg->{source} ne $1; # no match
+ foreach my $entry (@$ipset) {
+ next if $entry =~ m/^create/; # simply ignore
+ if ($entry =~ m/add \S+ (\S+)$/) {
+ my $test = Net::IP->new($1);
+ if ($test->overlaps($ip)) {
+ add_trace("IPSET $ipsetname match $ipaddr\n");
+ return 1;
+ }
+ } else {
+ die "implement me";
+ }
}
+
+ return 0;
+}
+
+sub rule_match {
+ my ($ipset_ruleset, $chain, $rule, $pkg) = @_;
+
+ $rule =~ s/^-A $chain // || die "got strange rule: $rule";
+
+ while (length($rule)) {
+
+ if ($rule =~ s/^-m conntrack\s*//) {
+ return undef; # simply ignore
+ }
+
+ if ($rule =~ s/^-m addrtype\s*//) {
+ return undef; # simply ignore
+ }
+
+ if ($rule =~ s/^-i (\S+)\s*//) {
+ my $devre = $1;
+ die "missing iface_in" if !$pkg->{iface_in};
+ return undef if !nf_dev_match($devre, $pkg->{iface_in});
+ next;
+ }
+
+ if ($rule =~ s/^-o (\S+)\s*//) {
+ my $devre = $1;
+ die "missing iface_out" if !$pkg->{iface_out};
+ return undef if !nf_dev_match($devre, $pkg->{iface_out});
+ next;
+ }
+
+ if ($rule =~ s/^-p (tcp|udp)\s*//) {
+ die "missing proto" if !$pkg->{proto};
+ return undef if $pkg->{proto} ne $1; # no match
+ next;
+ }
+
+ if ($rule =~ s/^--dport (\d+):(\d+)\s*//) {
+ die "missing dport" if !$pkg->{dport};
+ return undef if ($pkg->{dport} < $1) || ($pkg->{dport} > $2); # no match
+ next;
+ }
+
+ if ($rule =~ s/^--dport (\d+)\s*//) {
+ die "missing dport" if !$pkg->{dport};
+ return undef if $pkg->{dport} != $1; # no match
+ next;
+ }
+
+ if ($rule =~ s/^-s (\S+)\s*//) {
+ die "missing source" if !$pkg->{source};
+ my $ip = Net::IP->new($1);
+ return undef if !$ip->overlaps(Net::IP->new($pkg->{source})); # no match
+ next;
+ }
- if ($rule =~ s/^-d (\S+)\s*//) {
- die "missing destination" if !$pkg->{dest};
- return undef if $pkg->{dest} ne $1; # no match
- }
+ if ($rule =~ s/^-d (\S+)\s*//) {
+ die "missing destination" if !$pkg->{dest};
+ my $ip = Net::IP->new($1);
+ return undef if !$ip->overlaps(Net::IP->new($pkg->{dest})); # no match
+ next;
+ }
- if ($rule =~ s/^-m mac ! --mac-source (\S+)\s*//) {
- die "missing source mac" if !$pkg->{mac_source};
- return undef if $pkg->{mac_source} eq $1; # no match
- }
+ if ($rule =~ s/^-m set --match-set (\S+) src\s*//) {
+ die "missing source" if !$pkg->{source};
+ my $ipset = $ipset_ruleset->{$1};
+ die "no such ip set '$1'" if !$ipset;
+ return undef if !ipset_match($1, $ipset, $pkg->{source});
+ next;
+ }
- if ($rule =~ s/^-m physdev --physdev-is-bridged --physdev-in (\S+)\s*//) {
- my $devre = $1;
- $devre =~ s/\+/\.\*/;
- return undef if !$pkg->{physdev_in};
- return undef if $pkg->{physdev_in} !~ m/^${devre}$/;
- }
+ if ($rule =~ s/^-m set --match-set (\S+) dst\s*//) {
+ die "missing destination" if !$pkg->{dest};
+ my $ipset = $ipset_ruleset->{$1};
+ die "no such ip set '$1'" if !$ipset;
+ return undef if !ipset_match($1, $ipset, $pkg->{dest});
+ next;
+ }
- if ($rule =~ s/^-m physdev --physdev-is-bridged --physdev-out (\S+)\s*//) {
- my $devre = $1;
- $devre =~ s/\+/\.\*/;
- return undef if !$pkg->{physdev_out};
- return undef if $pkg->{physdev_out} !~ m/^${devre}$/;
- }
+ if ($rule =~ s/^-m mac ! --mac-source (\S+)\s*//) {
+ die "missing source mac" if !$pkg->{mac_source};
+ return undef if $pkg->{mac_source} eq $1; # no match
+ next;
+ }
- if ($rule =~ s/^-j MARK --set-mark (\d+)\s*$//) {
- $mark = $1;
- return undef;
- }
+ if ($rule =~ s/^-m physdev --physdev-is-bridged --physdev-in (\S+)\s*//) {
+ my $devre = $1;
+ return undef if !$pkg->{physdev_in};
+ return undef if !nf_dev_match($devre, $pkg->{physdev_in});
+ next;
+ }
- if ($rule =~ s/^-j (\S+)\s*$//) {
- return (0, $1);
- }
+ if ($rule =~ s/^-m physdev --physdev-is-bridged --physdev-out (\S+)\s*//) {
+ my $devre = $1;
+ return undef if !$pkg->{physdev_out};
+ return undef if !nf_dev_match($devre, $pkg->{physdev_out});
+ next;
+ }
- if ($rule =~ s/^-g (\S+)\s*$//) {
- return (1, $1);
+ if ($rule =~ s/^-m mark --mark (\d+)\s*//) {
+ return undef if !defined($mark) || $mark != $1;
+ next;
+ }
+
+ # final actions
+
+ if ($rule =~ s/^-j MARK --set-mark (\d+)\s*$//) {
+ $mark = $1;
+ return undef;
+ }
+
+ if ($rule =~ s/^-j (\S+)\s*$//) {
+ return (0, $1);
+ }
+
+ if ($rule =~ s/^-g (\S+)\s*$//) {
+ return (1, $1);
+ }
+
+ if ($rule =~ s/^-j NFLOG --nflog-prefix \"[^\"]+\"$//) {
+ return undef;
+ }
+
+ last;
}
die "unable to parse rule: $rule";
}
sub ruleset_simulate_chain {
- my ($ruleset, $chain, $pkg) = @_;
+ my ($ruleset, $ipset_ruleset, $chain, $pkg) = @_;
- $trace .= "ENTER chain $chain\n";
+ add_trace("ENTER chain $chain\n");
+ my $counter = 0;
+
if ($chain eq 'PVEFW-Drop') {
- $trace .= "LEAVE chain $chain\n";
- return 'DROP';
+ add_trace("LEAVE chain $chain\n");
+ return ('DROP', $counter);
}
if ($chain eq 'PVEFW-reject') {
- $trace .= "LEAVE chain $chain\n";
- return 'REJECT';
+ add_trace("LEAVE chain $chain\n");
+ return ('REJECT', $counter);
}
if ($chain eq 'PVEFW-tcpflags') {
- $trace .= "LEAVE chain $chain\n";
- return undef;
+ add_trace("LEAVE chain $chain\n");
+ return (undef, $counter);
}
my $rules = $ruleset->{$chain} ||
die "no such chain '$chain'";
foreach my $rule (@$rules) {
- my ($goto, $action) = rule_match($chain, $rule, $pkg);
+ $counter++;
+ my ($goto, $action) = rule_match($ipset_ruleset, $chain, $rule, $pkg);
if (!defined($action)) {
- $trace .= "SKIP: $rule\n";
+ add_trace("SKIP: $rule\n");
next;
}
- $trace .= "MATCH: $rule\n";
+ add_trace("MATCH: $rule\n");
if ($action eq 'ACCEPT' || $action eq 'DROP' || $action eq 'REJECT') {
- $trace .= "TERMINATE chain $chain: $action\n";
- return $action;
+ add_trace("TERMINATE chain $chain: $action\n");
+ return ($action, $counter);
} elsif ($action eq 'RETURN') {
- $trace .= "RETURN FROM chain $chain\n";
+ add_trace("RETURN FROM chain $chain\n");
last;
} else {
if ($goto) {
- $trace .= "LEAVE chain $chain - goto $action\n";
- return ruleset_simulate_chain($ruleset, $action, $pkg)
+ add_trace("LEAVE chain $chain - goto $action\n");
+ return ruleset_simulate_chain($ruleset, $ipset_ruleset, $action, $pkg)
#$chain = $action;
#$rules = $ruleset->{$chain} || die "no such chain '$chain'";
} else {
- if ($action = ruleset_simulate_chain($ruleset, $action, $pkg)) {
- return $action;
- }
- $trace .= "CONTINUE chain $chain\n";
+ my ($act, $ctr) = ruleset_simulate_chain($ruleset, $ipset_ruleset, $action, $pkg);
+ $counter += $ctr;
+ return ($act, $counter) if $act;
+ add_trace("CONTINUE chain $chain\n");
}
}
}
- $trace .= "LEAVE chain $chain\n";
+ add_trace("LEAVE chain $chain\n");
if ($chain =~ m/^PVEFW-(INPUT|OUTPUT|FORWARD)$/) {
- return 'ACCEPT'; # default policy
+ return ('ACCEPT', $counter); # default policy
}
- return undef;
+ return (undef, $counter);
}
sub copy_packet {
return $res;
}
+# Try to simulate packet traversal inside kernel. This invokes iptable
+# checks several times.
+sub route_packet {
+ my ($ruleset, $ipset_ruleset, $pkg, $from_info, $target, $start_state) = @_;
+
+ my $route_state = $start_state;
+
+ my $physdev_in;
+
+ my $ipt_invocation_counter = 0;
+ my $rule_check_counter = 0;
+
+ while ($route_state ne $target->{iface}) {
+
+ my $chain;
+ my $next_route_state;
+ my $next_physdev_in;
+
+ $pkg->{iface_in} = $pkg->{iface_out} = undef;
+ $pkg->{physdev_in} = $pkg->{physdev_out} = undef;
+
+ if ($route_state eq 'from-bport') {
+ $next_route_state = $from_info->{bridge} || die 'internal error';
+ $next_physdev_in = $from_info->{iface} || die 'internal error';
+ } elsif ($route_state eq 'host') {
+
+ if ($target->{type} eq 'bport') {
+ $pkg->{iface_in} = 'lo';
+ $pkg->{iface_out} = $target->{bridge} || die 'internal error';
+ $chain = 'PVEFW-OUTPUT';
+ $next_route_state = $target->{iface} || die 'internal error';
+ } elsif ($target->{type} eq 'ct') {
+ $pkg->{iface_in} = 'lo';
+ $pkg->{iface_out} = 'venet0';
+ $chain = 'PVEFW-OUTPUT';
+ $next_route_state = 'venet-in';
+ } elsif ($target->{type} eq 'vm') {
+ $pkg->{iface_in} = 'lo';
+ $pkg->{iface_out} = $target->{bridge} || die 'internal error';
+ $chain = 'PVEFW-OUTPUT';
+ $next_route_state = 'fwbr-in';
+ } else {
+ die "implement me";
+ }
+
+ } elsif ($route_state eq 'venet-out') {
+
+ if ($target->{type} eq 'host') {
+
+ $chain = 'PVEFW-INPUT';
+ $pkg->{iface_in} = 'venet0';
+ $pkg->{iface_out} = 'lo';
+ $next_route_state = 'host';
+
+ } elsif ($target->{type} eq 'bport') {
+
+ $chain = 'PVEFW-FORWARD';
+ $pkg->{iface_in} = 'venet0';
+ $pkg->{iface_out} = $target->{bridge} || die 'internal error';
+ $next_route_state = $target->{iface} || die 'internal error';
+
+ } elsif ($target->{type} eq 'vm') {
+
+ $chain = 'PVEFW-FORWARD';
+ $pkg->{iface_in} = 'venet0';
+ $pkg->{iface_out} = $target->{bridge} || die 'internal error';
+ $next_route_state = 'fwbr-in';
+
+ } elsif ($target->{type} eq 'ct') {
+
+ $chain = 'PVEFW-FORWARD';
+ $pkg->{iface_in} = 'venet0';
+ $pkg->{iface_out} = 'venet0';
+ $next_route_state = 'venet-in';
+
+ } else {
+ die "implement me";
+ }
+
+ } elsif ($route_state eq 'fwbr-out') {
+
+ $chain = 'PVEFW-FORWARD';
+ $next_route_state = $from_info->{bridge} || die 'internal error';
+ $next_physdev_in = $from_info->{fwpr} || die 'internal error';
+ $pkg->{iface_in} = $from_info->{fwbr} || die 'internal error';
+ $pkg->{iface_out} = $from_info->{fwbr} || die 'internal error';
+ $pkg->{physdev_in} = $from_info->{tapdev} || die 'internal error';
+ $pkg->{physdev_out} = $from_info->{fwln} || die 'internal error';
+
+ } elsif ($route_state eq 'fwbr-in') {
+
+ $chain = 'PVEFW-FORWARD';
+ $next_route_state = $target->{tapdev};
+ $pkg->{iface_in} = $target->{fwbr} || die 'internal error';
+ $pkg->{iface_out} = $target->{fwbr} || die 'internal error';
+ $pkg->{physdev_in} = $target->{fwln} || die 'internal error';
+ $pkg->{physdev_out} = $target->{tapdev} || die 'internal error';
+
+ } elsif ($route_state =~ m/^vmbr\d+$/) {
+
+ die "missing physdev_in - internal error?" if !$physdev_in;
+ $pkg->{physdev_in} = $physdev_in;
+
+ if ($target->{type} eq 'host') {
+
+ $chain = 'PVEFW-INPUT';
+ $pkg->{iface_in} = $route_state;
+ $pkg->{iface_out} = 'lo';
+ $next_route_state = 'host';
+
+ } elsif ($target->{type} eq 'bport') {
+
+ $chain = 'PVEFW-FORWARD';
+ $pkg->{iface_in} = $route_state;
+ $pkg->{iface_out} = $target->{bridge} || die 'internal error';
+ # conditionally set physdev_out (same behavior as kernel)
+ if ($route_state eq $target->{bridge}) {
+ $pkg->{physdev_out} = $target->{iface} || die 'internal error';
+ }
+ $next_route_state = $target->{iface};
+
+ } elsif ($target->{type} eq 'ct') {
+
+ $chain = 'PVEFW-FORWARD';
+ $pkg->{iface_in} = $route_state;
+ $pkg->{iface_out} = 'venet0';
+ $next_route_state = 'venet-in';
+
+ } elsif ($target->{type} eq 'vm') {
+
+ $chain = 'PVEFW-FORWARD';
+ $pkg->{iface_in} = $route_state;
+ $pkg->{iface_out} = $target->{bridge};
+ # conditionally set physdev_out (same behavior as kernel)
+ if ($route_state eq $target->{bridge}) {
+ $pkg->{physdev_out} = $target->{fwpr} || die 'internal error';
+ }
+ $next_route_state = 'fwbr-in';
+
+ } else {
+ die "implement me";
+ }
+
+ } else {
+ die "implement me $route_state";
+ }
+
+ die "internal error" if !defined($next_route_state);
+
+ if ($chain) {
+ add_trace("IPT check at $route_state (chain $chain)\n");
+ add_trace(Dumper($pkg));
+ $ipt_invocation_counter++;
+ my ($res, $ctr) = ruleset_simulate_chain($ruleset, $ipset_ruleset, $chain, $pkg);
+ $rule_check_counter += $ctr;
+ return ($res, $ipt_invocation_counter, $rule_check_counter) if $res ne 'ACCEPT';
+ }
+
+ $route_state = $next_route_state;
+
+ $physdev_in = $next_physdev_in;
+ }
+
+ return ('ACCEPT', $ipt_invocation_counter, $rule_check_counter);
+}
+
+sub extract_ct_info {
+ my ($vmdata, $vmid) = @_;
+
+ my $info = { type => 'ct', vmid => $vmid };
+
+ my $conf = $vmdata->{openvz}->{$vmid} || die "no such CT '$vmid'";
+ if ($conf->{ip_address}) {
+ $info->{ip_address} = $conf->{ip_address}->{value};
+ } else {
+ die "implement me";
+ }
+ return $info;
+}
+
+sub extract_vm_info {
+ my ($vmdata, $vmid) = @_;
+
+ my $info = { type => 'vm', vmid => $vmid };
+
+ my $conf = $vmdata->{qemu}->{$vmid} || die "no such VM '$vmid'";
+ my $net = PVE::QemuServer::parse_net($conf->{net0});
+ $info->{macaddr} = $net->{macaddr} || die "unable to get mac address";
+ $info->{bridge} = $net->{bridge} || die "unable to get bridge";
+ $info->{fwbr} = "fwbr${vmid}i0";
+ $info->{tapdev} = "tap${vmid}i0";
+ $info->{fwln} = "fwln${vmid}i0";
+ $info->{fwpr} = "fwpr${vmid}p0";
+
+ return $info;
+}
sub simulate_firewall {
my ($ruleset, $ipset_ruleset, $vmdata, $test) = @_;
- my $from = delete $test->{from} || die "missing 'from' field";
- my $to = delete $test->{to} || die "missing 'to' field";
- my $action = delete $test->{action} || die "missing 'action'";
-
+ my $from = $test->{from} || die "missing 'from' field";
+ my $to = $test->{to} || die "missing 'to' field";
+ my $action = $test->{action} || die "missing 'action'";
+
+ my $testid = $test->{id};
+
die "from/to needs to be different" if $from eq $to;
my $pkg = {
- iface_in => 'lo',
- iface_out => 'lo',
proto => 'tcp',
- sport => '1234',
- dport => '4321',
- source => '10.11.12.13',
- dest => '10.11.12.14',
+ sport => undef,
+ dport => undef,
+ source => undef,
+ dest => undef,
};
while (my ($k,$v) = each %$test) {
+ next if $k eq 'from';
+ next if $k eq 'to';
+ next if $k eq 'action';
+ next if $k eq 'id';
+ die "unknown attribute '$k'\n" if !exists($pkg->{$k});
$pkg->{$k} = $v;
}
- my $pre_test;
+ my $from_info = {};
+
+ my $start_state;
if ($from eq 'host') {
- $pre_test = ['PVEFW-OUTPUT', $pkg];
+ $from_info->{type} = 'host';
+ $start_state = 'host';
+ } elsif ($from =~ m|^(vmbr\d+)/(\S+)$|) {
+ $from_info->{type} = 'bport';
+ $from_info->{bridge} = $1;
+ $from_info->{iface} = $2;
+ $start_state = 'from-bport';
+ } elsif ($from eq 'outside') {
+ $from_info->{type} = 'bport';
+ $from_info->{bridge} = 'vmbr0';
+ $from_info->{iface} = 'eth0';
+ $start_state = 'from-bport';
+ } elsif ($from eq 'nfvm') {
+ $from_info->{type} = 'bport';
+ $from_info->{bridge} = 'vmbr0';
+ $from_info->{iface} = 'tapXYZ';
+ $start_state = 'from-bport';
} elsif ($from =~ m/^ct(\d+)$/) {
my $vmid = $1;
- my $conf = $vmdata->{openvz}->{$vmid} || die "no such CT '$vmid'";
- if ($conf->{ip_address}) {
- $pkg->{source} = $conf->{ip_address}->{value};
- $pkg->{iface_in} = 'venet0';
+ $from_info = extract_ct_info($vmdata, $vmid);
+ if ($from_info->{ip_address}) {
+ $pkg->{source} = $from_info->{ip_address} if !defined($pkg->{source});
+ $start_state = 'venet-out';
} else {
die "implement me";
}
- $pre_test = ['PVEFW-FORWARD', $pkg];
} elsif ($from =~ m/^vm(\d+)$/) {
my $vmid = $1;
- my $conf = $vmdata->{qemu}->{$vmid} || die "no such VM '$vmid'";
- my $net = PVE::QemuServer::parse_net($conf->{net0});
- my $macaddr = $net->{macaddr} || die "unable to get mac address";
- $pkg->{iface_in} = $net->{bridge} || die "unable to get bridge";
- $pkg->{mac_source} = $macaddr;
- my $brpkg = copy_packet($pkg);
- $brpkg->{physdev_in} = "tap${vmid}i0";
- $brpkg->{physdev_out} = "fwln${vmid}i0";
- $brpkg->{iface_in} = $brpkg->{iface_out} = "fwbr${vmid}i0";
- $pre_test = ['PVEFW-FORWARD', $brpkg];
+ $from_info = extract_vm_info($vmdata, $vmid);
+ $start_state = 'fwbr-out';
+ $pkg->{mac_source} = $from_info->{macaddr};
} else {
- die "implement me";
+ die "unable to parse \"from => '$from'\"\n";
}
- my $post_test;
+ my $target;
+
if ($to eq 'host') {
- $post_test = ['PVEFW-INPUT', $pkg];
+ $target->{type} = 'host';
+ $target->{iface} = 'host';
+ } elsif ($to =~ m|^(vmbr\d+)/(\S+)$|) {
+ $target->{type} = 'bport';
+ $target->{bridge} = $1;
+ $target->{iface} = $2;
+ } elsif ($to eq 'outside') {
+ $target->{type} = 'bport';
+ $target->{bridge} = 'vmbr0';
+ $target->{iface} = 'eth0';
+ } elsif ($to eq 'nfvm') {
+ $target->{type} = 'bport';
+ $target->{bridge} = 'vmbr0';
+ $target->{iface} = 'tapXYZ';
} elsif ($to =~ m/^ct(\d+)$/) {
my $vmid = $1;
- my $conf = $vmdata->{openvz}->{$vmid} || die "no such CT '$vmid'";
- if ($conf->{ip_address}) {
- $pkg->{dest} = $conf->{ip_address}->{value};
- $pkg->{iface_out} = 'venet0';
+ $target = extract_ct_info($vmdata, $vmid);
+ $target->{iface} = 'venet-in';
+
+ if ($target->{ip_address}) {
+ $pkg->{dest} = $target->{ip_address};
} else {
die "implement me";
}
- $post_test = ['PVEFW-FORWARD', $pkg];
} elsif ($to =~ m/^vm(\d+)$/) {
my $vmid = $1;
- my $conf = $vmdata->{qemu}->{$vmid} || die "no such VM '$vmid'";
- my $net = PVE::QemuServer::parse_net($conf->{net0});
- $pkg->{iface_out} = $net->{bridge} || die "unable to get bridge";
- my $brpkg = copy_packet($pkg);
- $brpkg->{physdev_out} = "tap${vmid}i0";
- $brpkg->{physdev_in} = "fwln${vmid}i0";
- $brpkg->{iface_in} = $brpkg->{iface_out} = "fwbr${vmid}i0";
- $post_test = ['PVEFW-FORWARD', $brpkg];
+ $target = extract_vm_info($vmdata, $vmid);
+ $target->{iface} = $target->{tapdev};
} else {
- die "implement me";
+ die "unable to parse \"to => '$to'\"\n";
}
- my $res = 'UNKNOWN';
- if ($pre_test) {
- my ($chain, $testpkg) = @$pre_test;
- $trace .= "PRE TEST $chain: " . Dumper($testpkg);
- $res = ruleset_simulate_chain($ruleset, $chain, $testpkg);
- if ($res ne 'ACCEPT') {
- die "test failed ($res != $action)\n" if $action ne $res;
- return undef; # sucess
- }
- }
-
- if ($post_test) {
- my ($chain, $testpkg) = @$post_test;
- $trace .= "POST TEST $chain: " . Dumper($testpkg);
- $res = ruleset_simulate_chain($ruleset, $chain, $testpkg);
- if ($res ne 'ACCEPT') {
- die "test failed ($res != $action)\n" if $action ne $res;
- return undef; # sucess
- }
- }
+ my ($res, $ic, $rc) = route_packet($ruleset, $ipset_ruleset, $pkg,
+ $from_info, $target, $start_state);
- die "test failed ($res != $action)\n" if $action ne $res; # fixme: remove
+ add_trace("IPT statistics: invocation = $ic, checks = $rc\n");
+
+ die "test failed ($res != $action)\n" if $action ne $res;
- return undef;
+ return undef;
}
sub run_tests {
- my ($vmdata, $testdir) = @_;
+ my ($vmdata, $testdir, $testfile, $testid) = @_;
+
+ $testfile = 'tests' if !$testfile;
$vmdata->{testdir} = $testdir;
my ($ruleset, $ipset_ruleset) =
PVE::Firewall::compile(undef, undef, $vmdata);
- my $testfile = "$testdir/tests";
- my $fh = IO::File->new($testfile) ||
- die "unable to open '$testfile' - $!\n";
+ my $filename = "$testdir/$testfile";
+ my $fh = IO::File->new($filename) ||
+ die "unable to open '$filename' - $!\n";
+ my $testcount = 0;
while (defined(my $line = <$fh>)) {
next if $line =~ m/^\s*$/;
next if $line =~ m/^#.*$/;
if ($line =~ m/^\{.*\}\s*$/) {
my $test = eval $line;
die $@ if $@;
+ next if defined($testid) && (!defined($test->{id}) || ($testid ne $test->{id}));
$trace = '';
- eval { simulate_firewall($ruleset, $ipset_ruleset, $vmdata, $test); };
+ print Dumper($ruleset) if $debug;
+ $testcount++;
+ eval {
+ my @test_zones = qw(host outside nfvm vm100 ct200);
+ if (!defined($test->{from}) && !defined($test->{to})) {
+ die "missing zone speification (from, to)\n";
+ } elsif (!defined($test->{to})) {
+ foreach my $zone (@test_zones) {
+ next if $zone eq $test->{from};
+ $test->{to} = $zone;
+ add_trace("Set Zone: to => '$zone'\n");
+ simulate_firewall($ruleset, $ipset_ruleset, $vmdata, $test);
+ }
+ } elsif (!defined($test->{from})) {
+ foreach my $zone (@test_zones) {
+ next if $zone eq $test->{to};
+ $test->{from} = $zone;
+ add_trace("Set Zone: from => '$zone'\n");
+ simulate_firewall($ruleset, $ipset_ruleset, $vmdata, $test);
+ }
+ } else {
+ simulate_firewall($ruleset, $ipset_ruleset, $vmdata, $test);
+ }
+ };
if (my $err = $@) {
- print Dumper($ruleset);
+ print Dumper($ruleset) if !$debug;
- print "$trace\n";
+ print "$trace\n" if !$debug;
- print "$testfile line $.: $line";
+ print "$filename line $.: $line";
print "test failed: $err\n";
}
}
- print "PASS: $testfile\n";
+ die "no tests found\n" if $testcount <= 0;
+
+ print "PASS: $filename\n";
return undef;
}
my $vmdata = {
qemu => {
100 => {
- net0 => "e1000=0E:0B:38:B8:B3:21,bridge=vmbr0,firewall=0",
+ net0 => "e1000=0E:0B:38:B8:B3:21,bridge=vmbr0,firewall=1",
+ },
+ 101 => {
+ net0 => "e1000=0E:0B:38:B8:B3:22,bridge=vmbr0,firewall=1",
+ },
+ # on bridge vmbr1
+ 110 => {
+ net0 => "e1000=0E:0B:38:B8:B4:21,bridge=vmbr1,firewall=1",
},
},
openvz => {
200 => {
ip_address => { value => '10.0.200.1' },
},
+ 201 => {
+ ip_address => { value => '10.0.200.2' },
+ },
},
};
-foreach my $dir (<test-*>) {
- next if ! -d $dir;
- run_tests($vmdata, $dir);
+if ($testfilename) {
+ my $testfile;
+ my $dir;
+
+ if (-d $testfilename) {
+ $dir = $testfilename;
+ } elsif (-f $testfilename) {
+ $dir = dirname($testfilename);
+ $testfile = basename($testfilename);
+ } else {
+ die "no such file/dir '$testfilename'\n";
+ }
+
+ run_tests($vmdata, $dir, $testfile, $testid);
+
+} else {
+ foreach my $dir (<test-*>) {
+ next if ! -d $dir;
+ run_tests($vmdata, $dir);
+ }
}
print "OK - all tests passed\n";