git: 8704d29c6cc8 - main - ixgbe: Avoid a signed multicast bitmap shift

Kevin Bowling <[email protected]>
Newsgroups gmane.os.freebsd.devel.cvs.src
Message-ID <6a7b8d51.38ba9.28129eb0__15768.7161977933$1786482128$gmane$org@gitrepo.freebsd.org>
The branch main has been updated by kbowling:

URL: https://cgit.FreeBSD.org/src/commit/?id=8704d29c6cc86f0780dff3d3d17d744106776ad3

commit 8704d29c6cc86f0780dff3d3d17d744106776ad3
Author:     Kevin Bowling <[email protected]>
AuthorDate: 2026-08-11 19:27:45 +0000
Commit:     Kevin Bowling <[email protected]>
CommitDate: 2026-08-11 20:59:32 +0000

    ixgbe: Avoid a signed multicast bitmap shift
    
    The multicast vector bit can be 31.  Use an unsigned value so setting
    the bit cannot shift a signed integer into its sign bit.
    
    MFC after:      2 weeks
---
 sys/dev/ixgbe/ixgbe_common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/dev/ixgbe/ixgbe_common.c b/sys/dev/ixgbe/ixgbe_common.c
index 3ee6885dcad9..458452a8492e 100644
--- a/sys/dev/ixgbe/ixgbe_common.c
+++ b/sys/dev/ixgbe/ixgbe_common.c
@@ -2729,7 +2729,7 @@ void ixgbe_set_mta(struct ixgbe_hw *hw, u8 *mc_addr)
 	 */
 	vector_reg = (vector >> 5) & 0x7F;
 	vector_bit = vector & 0x1F;
-	hw->mac.mta_shadow[vector_reg] |= (1 << vector_bit);
+	hw->mac.mta_shadow[vector_reg] |= 1U << vector_bit;
 }
 
 /**
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.