]> git.proxmox.com Git - pve-common.git/commitdiff
fix #5623: ovs other_config set to 0 not saved in network config
authorTiomet Pelston <tiometpelston@gmail.com>
Wed, 23 Oct 2024 16:48:49 +0000 (18:48 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Wed, 30 Oct 2024 08:51:34 +0000 (09:51 +0100)
When configuring an OVS network device via web interface,
any OVS option set to value=0 is ignored upon saving. This happens
because value=0 is evaluated as false in $parse_ovs_option.

Signed-off-by: Tiomet Pelston <tiometpelston@gmail.com>
Reviewed-By: Aaron Lauterer <a.lauterer@proxmox.com>
Tested-By: Aaron Lauterer <a.lauterer@proxmox.com>
src/PVE/INotify.pm

index 8a4a810f51fbbfa5905a4973841b5bb037f4df9f..1b3babd950d85591e1b321b5a5151c4417a36381 100644 (file)
@@ -748,7 +748,7 @@ my $parse_ovs_option = sub {
     my $opts = {};
     foreach my $kv (split (/\s+/, $data || '')) {
        my ($k, $v) = split('=', $kv, 2);
-       $opts->{$k} = $v if $k && $v;
+       $opts->{$k} = $v if $k && defined($v);
     }
     return $opts;
 };