code cleanup

[pve-firewall.git] / PVE / Firewall.pm

diff --git a/PVE/Firewall.pm b/PVE/Firewall.pm
index 6565212d1f3bcb0d470f866b25b38d9079f4ef97..c10aab3e7a26d7d51ebd18326275391a5c635b87 100644 (file)
--- a/PVE/Firewall.pm
+++ b/PVE/Firewall.pm
@@ -3,22 +3,59 @@ package PVE::Firewall;
 use warnings;
 use strict;
 use Data::Dumper;
-
+use PVE::Tools;
 use PVE::QemuServer;
 
 # we need complete VM configuration of all VMs (openvz/qemu)
 # in vmdata
 
 sub compile {
-    my ($vmdata) = @_;
+    my ($targetdir, $vmdata) = @_;
 
     my $netinfo;
 
-    my $bridges = {};
     my $zoneinfo = {
        fw => { type => 'firewall' },
     };
 
+    my $register_bridge;
+
+    $register_bridge = sub {
+       my ($bridge, $vlan) = @_;
+
+       my $zone =  'z' . $bridge;
+
+       return $zone if $zoneinfo->{$zone};
+
+       $zoneinfo->{$zone} = {
+           type => 'bridge',
+           bridge => $bridge,
+       };
+
+       return &$register_bridge("${bridge}v${vlan}") if defined($vlan);
+       
+       return $zone;
+    };
+
+    my $register_bridge_port = sub {
+       my ($bridge, $vlan, $vmzone, $tap) = @_;
+
+       my $bridge_zone = &$register_bridge($bridge, $vlan);
+       my $zone = $bridge_zone . $vmzone;
+
+       if (!$zoneinfo->{$zone}) {
+           $zoneinfo->{$zone} = {
+               type => 'bport',
+               bridge_zone => $bridge_zone,
+               ifaces => {},
+           };
+       }
+
+       $zoneinfo->{$zone}->{ifaces}->{$tap} = 1;
+       
+       return $zone;
+    };
+
     foreach my $vmid (keys %{$vmdata->{qemu}}) {
        $netinfo->{$vmid} = {};
        my $conf = $vmdata->{qemu}->{$vmid};
@@ -28,62 +65,93 @@ sub compile {
            my $net = PVE::QemuServer::parse_net($conf->{$opt});
            next if !$net;
            die "implement me" if !$net->{bridge};
-           my $bridge = $net->{bridge};
-           $bridges->{$bridge} = 1;
-           $zoneinfo->{$bridge}->{type} = 'ipv4';
-           $zoneinfo->{$bridge}->{ifaces}->{$bridge} = 1;
-           if (defined($net->{tag})) {
-               $bridge = $bridge .= "v$net->{tag}";
-               $bridges->{$bridge} = 1;
-               $zoneinfo->{$bridge}->{type} = 'ipv4';
-               $zoneinfo->{$bridge}->{ifaces}->{$bridge} = 1;
-           }
 
-           my $zone = $bridge . ($conf->{zone} || "vm$vmid");
-           $net->{zone} = $zone;
-           $zoneinfo->{$zone}->{type} = 'bport';
-           $zoneinfo->{$zone}->{bridge} = $bridge;
-           $zoneinfo->{$zone}->{ifaces}->{"tap${vmid}i${netid}"} = 1;
+           my $vmzone = $conf->{zone} || "vm$vmid";
+           $net->{zone} = &$register_bridge_port($net->{bridge}, $net->{tag}, $vmzone, "tap${vmid}i${netid}");
            $netinfo->{$vmid}->{$netid} = $net;
        }
     }
 
     #print Dumper($netinfo);
 
-    # TODO: zone names have length limit, so we need to
+    # NOTE: zone names have length limit, so we need to
     # translate them into shorter names 
 
+    my $zoneid = 0;
+    my $zonemap = { fw => 'fw' };
+
+    my $lookup_zonename = sub {
+       my ($zone) = @_;
+
+       return $zonemap->{$zone} if defined($zonemap->{$zone});
+       $zonemap->{$zone} = 'z' . $zoneid++;
+
+       return $zonemap->{$zone};
+    };
+
     # dump zone file
 
-    print "DUMP: zones\n";
-    my $format = "%-15s %-10s %s\n";
-    printf($format, '#ZONE', 'TYPE', 'OPTIONS');
+    my $out;
 
+    my $format = "%-15s %-10s %s\n";
+    $out = sprintf($format, '#ZONE', 'TYPE', 'OPTIONS');
+    
     foreach my $z (sort keys %$zoneinfo) {
-       printf($format, $z, $zoneinfo->{$z}->{type}, '');
+       my $zid = &$lookup_zonename($z);
+       if ($zoneinfo->{$z}->{type} eq 'firewall') {
+           $out .= sprintf($format, $zid, $zoneinfo->{$z}->{type}, '');
+       } elsif ($zoneinfo->{$z}->{type} eq 'bridge') {
+           $out .= sprintf($format, &$lookup_zonename($z), 'ipv4', '');
+       } elsif ($zoneinfo->{$z}->{type} eq 'bport') {
+           my $bridge_zone = $zoneinfo->{$z}->{bridge_zone} || die "internal error";
+           my $bzid = &$lookup_zonename($bridge_zone);
+           $out .= sprintf($format, "$zid:$bzid", 'bport', '');
+       } else {
+           die "internal error";
+       }
     }
 
-    print "#LAST LINE - ADD YOUR ENTRIES ABOVE THIS ONE - DO NOT REMOVE\n";
+    $out .= sprintf("#LAST LINE - ADD YOUR ENTRIES ABOVE THIS ONE - DO NOT REMOVE\n");
+
+    PVE::Tools::file_set_contents("$targetdir/zones", $out);
 
-    print "\n";
-    print "DUMP: interfaces\n";
+    # dump interfaces
 
     $format = "%-15s %-20s %-10s %s\n";
-    printf($format, '#ZONE', 'INTERFACE', 'BROADCAST', 'OPTIONS');
+    $out = sprintf($format, '#ZONE', 'INTERFACE', 'BROADCAST', 'OPTIONS');
+
     foreach my $z (sort keys %$zoneinfo) {
-       my $ifaces = $zoneinfo->{$z}->{ifaces};
-       foreach my $iface (sort keys %$ifaces) {
-           my $broadcast =  $zoneinfo->{$z}->{type} eq 'ipv4' ? 'detect' : '';
-           my $options =  $bridges->{$iface} ? 'bridge' : '';
-           my $bridge = $zoneinfo->{$z}->{bridge} || '';
-           my $iftxt = $zoneinfo->{$z}->{bridge} ? "$zoneinfo->{$z}->{bridge}:$iface" : $iface;
-           printf($format, $z, $iftxt, $broadcast, $options);
+       my $zid = &$lookup_zonename($z);
+       if ($zoneinfo->{$z}->{type} eq 'firewall') {
+           # do nothing;
+       } elsif ($zoneinfo->{$z}->{type} eq 'bridge') {
+           my $bridge = $zoneinfo->{$z}->{bridge} || die "internal error";
+           $out .= sprintf($format, $zid, $bridge, 'detect', 'bridge');
+
+       } elsif ($zoneinfo->{$z}->{type} eq 'bport') {
+           my $ifaces = $zoneinfo->{$z}->{ifaces};
+           foreach my $iface (sort keys %$ifaces) {
+               my $bridge_zone = $zoneinfo->{$z}->{bridge_zone} || die "internal error";
+               my $bridge = $zoneinfo->{$bridge_zone}->{bridge} || die "internal error";
+               my $iftxt = "$bridge:$iface";
+               $out .= sprintf($format, $zid, $iftxt, '', '');
+           }
+       } else {
+           die "internal error";
        }
     }
 
-    print "#LAST LINE - ADD YOUR ENTRIES ABOVE THIS ONE - DO NOT REMOVE\n";
+    $out .= sprintf("#LAST LINE - ADD YOUR ENTRIES ABOVE THIS ONE - DO NOT REMOVE\n");
+
+    PVE::Tools::file_set_contents("$targetdir/interfaces", $out);
+
+    # dump policy
+
+    $format = "%-15s %-15s %-15s %s\n";
+    $out = sprintf($format, '#SOURCE', 'DEST', 'POLICY', 'LOG');
+    $out .= sprintf($format, 'all', 'all', 'REJECT', 'info');
 
-    print "\n";
+    PVE::Tools::file_set_contents("$targetdir/policy", $out);
 
 
 }