Code Review / vpp.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 60c3977) | patch
l2: coverity woe in l2_rw_mod_entry 95/31495/2
authorSteven Luong <sluong@cisco.com>
Thu, 4 Mar 2021 00:56:19 +0000 (16:56 -0800)
committerDamjan Marion <dmarion@me.com>
Fri, 5 Mar 2021 10:50:48 +0000 (10:50 +0000)
commitd9360174483d3716886fa42f1b7041fc0cc79ef9
treeecb7c4201b12c1c4d6792ed4f04c8b915241e7ectree | snapshot
parent60c397711a4d0f694491945ccb64d3da3049557acommit | diff
l2: coverity woe in l2_rw_mod_entry

Coverity complains that the statement
 if (!e)
   return -1;
is never true and is logically dead code in the subject function. It is
right. e is assigned in both the if and else statementes immediately above
and can never be null.

Type: fix

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