Code Review / vpp.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 092a30a) | patch
bonding: ASSERT fails for bif->active_members in TX path 53/42653/4
authorSteven Luong <sluong@cisco.com>
Thu, 3 Apr 2025 17:22:10 +0000 (10:22 -0700)
committerDamjan Marion <dmarion@0xa5.net>
Tue, 15 Apr 2025 10:14:50 +0000 (10:14 +0000)
commit4b7fde2335dc11f0758c1d786345a6f4806bb146
tree1c5135183578471a35be212a3fd1c4f236b6c300tree | snapshot
parent092a30a53f76108e2bfd69c53712579efd3ef1cecommit | diff
bonding: ASSERT fails for bif->active_members in TX path

Elements in bif->active_members are used without protection in the TX DP path.
However, elements in bif->active_members may freed by another thread asynchronously
and it may cause the ASSERT to fail when accessing elements.
The fix is to take a snapshot of the active_members and use that in the TX DP path.

Type: fix

Change-Id: I081cc81cc4d072807d6cbbcf5d21feb97f500362
Signed-off-by: Steven Luong <sluong@cisco.com>
src/vnet/bonding/device.c diff | blob | history
src/vnet/bonding/node.h diff | blob | history
Vector Packet Processing