fix ipset remove order
authorDietmar Maurer <dietmar@proxmox.com>
Fri, 28 Nov 2014 11:43:31 +0000 (12:43 +0100)
committerDietmar Maurer <dietmar@proxmox.com>
Fri, 28 Nov 2014 11:44:18 +0000 (12:44 +0100)
debian/changelog
src/PVE/Firewall.pm

index 1ea7a4c..df0a283 100644 (file)
@@ -2,6 +2,8 @@ pve-firewall (1.0-12) unstable; urgency=low
 
   * add preinst script to clear ipset from older installation (because 
     sets cannot be swapped if there type does not match.
+  
+  * fix ipset remove order
 
  -- Proxmox Support Team <support@proxmox.com>  Fri, 28 Nov 2014 08:59:38 +0100
 
index f4b199b..20759b6 100644 (file)
@@ -3381,7 +3381,7 @@ sub get_ipset_cmdlist {
     # remove unused list chains first
     foreach my $chain (keys %$statushash) {
        next if $statushash->{$chain}->{action} ne 'delete';
-       next if $chain !~ m/-v[46]$/;
+       next if $chain =~ m/-v[46]$/;
 
        $delete_cmdlist .= "flush $chain\n";
        $delete_cmdlist .= "destroy $chain\n";
@@ -3390,7 +3390,7 @@ sub get_ipset_cmdlist {
     # the remove unused -v4 -v6 chains
     foreach my $chain (keys %$statushash) {
        next if $statushash->{$chain}->{action} ne 'delete';
-       next if $chain =~ m/-v[46]$/;
+       next if $chain !~ m/-v[46]$/;
 
        $delete_cmdlist .= "flush $chain\n";
        $delete_cmdlist .= "destroy $chain\n";