interfaces: improve bridge_fd handling

and ignore values with a warning that are outside of the kernels
expected range.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

diff --git a/src/PVE/INotify.pm b/src/PVE/INotify.pm
index 4f682be94903e750e8dde622b8bfb563c24407fa..ad45cd2cba2fe40b4fc1038cb06897ef0ca98f95 100644 (file)
--- a/src/PVE/INotify.pm
+++ b/src/PVE/INotify.pm
@@ -1265,8 +1265,10 @@ sub __interface_to_string {
 
        $v = defined($d->{bridge_fd}) ? $d->{bridge_fd} : 0;
        # 0 is only allowed when STP is disabled
-       if ($v || $no_stp) {
+       if ($no_stp || ($v >= 2 && $v <= 30)) {
            $raw .= "\tbridge-fd $v\n";
+       } else {
+           warn "'$iface': not setting 'bridge_fd' to value '$v' outside of allowed range 2-30\n";
        }
        $done->{bridge_fd} = 1;