On Fri, Jan 13, 2023 at 10:35:43AM +0300, Dan Carpenter wrote: > This code checks if (attrs[DEVLINK_ATTR_TRAP_POLICER_ID]) twice. Once > at the start of the function and then a couple lines later. Delete the > second check since that one must be true. > > Because the second condition is always true, it means the: > > policer_item = group_item->policer_item; > > assignment is immediately over-written. Delete that as well. > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> Reviewed-by: Ido Schimmel <idosch@xxxxxxxxxx>
