Code Review / vpp.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: b4704f2) | patch
bonding: coverity woe in bond_dev_class fuction 18/31518/2
authorSteven Luong <sluong@cisco.com>
Thu, 4 Mar 2021 03:03:38 +0000 (19:03 -0800)
committerDamjan Marion <dmarion@me.com>
Thu, 4 Mar 2021 10:58:35 +0000 (10:58 +0000)
commit8d462190587f78658c8a9e33ed4f35c2d873965f
treef420d3488930fe32d27e59cb8cc1323516170a8ctree | snapshot
parentb4704f2570fb2484b3085c8d25d0c850cc3fc77fcommit | diff
bonding: coverity woe in bond_dev_class fuction

Coverity complans the line
  h = hashes;
uses uninitialized variable if the prior ASSERT statement is hit.

ASSERT is compiled out coverity as well as in release image. So the
complain is legitimate. Change the ASSERT to drop the frame and log
an error instead.

Type: fix

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