From 662f9f14d59377554bcad381b944bea6d81a79f1 Mon Sep 17 00:00:00 2001
From: Gris Ge <fge@redhat.com>
Date: Thu, 1 Feb 2024 16:00:42 +0800
Subject: [PATCH] bridge: Change InfoBridge::VlanFiltering from u8 to bool

The kernel code `net/bridge/br_vlan.c` function
`br_vlan_filter_toggle()` indicate the `IFLA_BR_VLAN_FILTERING` is
holding boolean.

Signed-off-by: Gris Ge <fge@redhat.com>
---
 src/link/link_info/bridge.rs | 12 ++++++------
 src/link/tests/bridge.rs     |  2 +-
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/link/link_info/bridge.rs b/src/link/link_info/bridge.rs
index b09430ae..3eb9f12e 100644
--- a/src/link/link_info/bridge.rs
+++ b/src/link/link_info/bridge.rs
@@ -94,7 +94,7 @@ pub enum InfoBridge {
     BridgeId(BridgeId),
     RootPort(u16),
     VlanDefaultPvid(u16),
-    VlanFiltering(u8),
+    VlanFiltering(bool),
     TopologyChange(u8),
     TopologyChangeDetected(u8),
     MulticastRouter(u8),
@@ -149,8 +149,8 @@ impl Nla for InfoBridge {
 
             Self::GroupAddr(_) => 6,
 
-            Self::VlanFiltering(_)
-            | Self::TopologyChange(_)
+            Self::VlanFiltering(_) => 1,
+            Self::TopologyChange(_)
             | Self::TopologyChangeDetected(_)
             | Self::MulticastRouter(_)
             | Self::MulticastSnooping(_)
@@ -217,8 +217,8 @@ impl Nla for InfoBridge {
 
             Self::GroupAddr(value) => buffer.copy_from_slice(&value[..]),
 
-            Self::VlanFiltering(value)
-            | Self::TopologyChange(value)
+            Self::VlanFiltering(value) => buffer[0] = (*value).into(),
+            Self::TopologyChange(value)
             | Self::TopologyChangeDetected(value)
             | Self::MulticastRouter(value)
             | Self::MulticastSnooping(value)
@@ -435,7 +435,7 @@ impl<'a, T: AsRef<[u8]> + ?Sized> Parseable<NlaBuffer<&'a T>> for InfoBridge {
             ),
             IFLA_BR_VLAN_FILTERING => Self::VlanFiltering(
                 parse_u8(payload)
-                    .context("invalid IFLA_BR_VLAN_FILTERING value")?,
+                    .context("invalid IFLA_BR_VLAN_FILTERING value")? > 0,
             ),
             IFLA_BR_TOPOLOGY_CHANGE => Self::TopologyChange(
                 parse_u8(payload)
diff --git a/src/link/tests/bridge.rs b/src/link/tests/bridge.rs
index ab40b6cd..a80aeaff 100644
--- a/src/link/tests/bridge.rs
+++ b/src/link/tests/bridge.rs
@@ -259,7 +259,7 @@ fn test_parse_link_bridge_no_extention_mask() {
                     InfoBridge::AgeingTime(29999),
                     InfoBridge::StpState(0),
                     InfoBridge::Priority(32768),
-                    InfoBridge::VlanFiltering(0),
+                    InfoBridge::VlanFiltering(false),
                     InfoBridge::GroupFwdMask(0),
                     InfoBridge::BridgeId(BridgeId {
                         priority: 0x8000,