dont use undefined var $ifcount
authorDietmar Maurer <dietmar@proxmox.com>
Wed, 19 Mar 2014 05:50:47 +0000 (06:50 +0100)
committerDietmar Maurer <dietmar@proxmox.com>
Wed, 19 Mar 2014 05:50:47 +0000 (06:50 +0100)
data/PVE/Network.pm

index ce06a96..4677bf9 100644 (file)
@@ -168,10 +168,10 @@ sub activate_bridge_vlan {
     my @ifaces = ();
     my $dir = "/sys/class/net/$bridge/brif";
     PVE::Tools::dir_glob_foreach($dir, '((eth|bond)\d+)', sub {
-        push(@ifaces, $_[0]);
+        push @ifaces, $_[0];
     });
 
-    die "no physical interface on bridge '$bridge'\n" if $ifcount == 0;
+    die "no physical interface on bridge '$bridge'\n" if scalar(@ifaces) == 0;
 
     # add bridgevlan if it doesn't already exist
     if (! -d "/sys/class/net/$bridgevlan") {