On Sat,  9 Oct 2021 21:07:55 -0700
Ajit Khaparde <ajit.khapa...@broadcom.com> wrote:

> +             if ((uint32_t)item->type >=
> +                 (uint32_t)BNXT_RTE_FLOW_ITEM_TYPE_END) {


This doesn't look right, you are casting away your troubles, not addressing
the root cause.

item->type is type rte_flow_item_type
BNXT_FLOW_ITEM_TYPE_END is the first item in that enum.

So yes the warning is valid and the code is wrong.

Reply via email to