proxmox-mirrors/pve-common
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-common synced 2025-07-25 12:49:36 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fixup: don't use exists boolean

Browse Source 
The previous condition didn't fail tests because it was
always false. The 'exists' property is not actually usable
when writing the interface file. It is merely a hint that
the interface existed in /proc/net/dev while parsing the
interfaces file and is otherwise actually unused here.

Simply check for the existence of the interface in $ifaces.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
This commit is contained in:
Wolfgang Bumiller 2018-07-10 12:00:14 +02:00
parent c27ef07ff5
commit fc158d0d9c
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/PVE/INotify.pm
View File

@ -1385,7 +1385,7 @@ sub __write_etc_network_interfaces {
my $n = $ifaces->{$p};
die "vlan '$iface' - unable to find parent '$p'\n"
if $n->{exists} eq 0;
if !$n;
if ($n->{type} eq 'bridge' && !$n->{bridge_vlan_aware}) {
die "vlan '$iface' - bridge vlan aware is not enabled on parent '$p'\n";