Patch "net: bridge: fix uninitialized variables when BRIDGE_CFM is disabled" has been added to the 5.14-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    net: bridge: fix uninitialized variables when BRIDGE_CFM is disabled

to the 5.14-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     net-bridge-fix-uninitialized-variables-when-bridge_c.patch
and it can be found in the queue-5.14 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit a8c1b374a728c0b80848af298ac221be938a8f90
Author: Ivan Vecera <ivecera@xxxxxxxxxx>
Date:   Thu Oct 28 17:58:35 2021 +0200

    net: bridge: fix uninitialized variables when BRIDGE_CFM is disabled
    
    [ Upstream commit 829e050eea69c7442441b714b6f5b339b5b8c367 ]
    
    Function br_get_link_af_size_filtered() calls br_cfm_{,peer}_mep_count()
    that return a count. When BRIDGE_CFM is not enabled these functions
    simply return -EOPNOTSUPP but do not modify count parameter and
    calling function then works with uninitialized variables.
    Modify these inline functions to return zero in count parameter.
    
    Fixes: b6d0425b816e ("bridge: cfm: Netlink Notifications.")
    Cc: Henrik Bjoernlund <henrik.bjoernlund@xxxxxxxxxxxxx>
    Signed-off-by: Ivan Vecera <ivecera@xxxxxxxxxx>
    Acked-by: Nikolay Aleksandrov <nikolay@xxxxxxxxxx>
    Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h
index 7d3155283af93..f538fe3902da9 100644
--- a/net/bridge/br_private.h
+++ b/net/bridge/br_private.h
@@ -1594,11 +1594,13 @@ static inline int br_cfm_status_fill_info(struct sk_buff *skb,
 
 static inline int br_cfm_mep_count(struct net_bridge *br, u32 *count)
 {
+	*count = 0;
 	return -EOPNOTSUPP;
 }
 
 static inline int br_cfm_peer_mep_count(struct net_bridge *br, u32 *count)
 {
+	*count = 0;
 	return -EOPNOTSUPP;
 }
 #endif



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux