Re: [PATCH v4 03/10] net/iavf: fix false success on flow validation

[Bruce Richardson]

On Wed, Oct 21, 2026 at 03:25:43PM +0100, Anatoly Burakov wrote:
> `iavf_parse_engine_validate()` sets an error when validation fails, but
> then returns the engine pointer. `iavf_flow_process_filter()` treats a
> non-NULL engine as success, so `rte_flow_validate()` can accept an
> invalid flow.
> 
> Return `NULL` on validation failure and return the engine only after
> validation succeeds.
> 
> Fixes: ff2d0c345c3b ("net/iavf: support generic flow API")
> Cc: sta...@dpdk.org
> 
> Signed-off-by: Anatoly Burakov <anatoly.bura...@intel.com>
> ---
>  drivers/net/intel/iavf/iavf_generic_flow.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
Acked-by: Bruce Richardson <bruce.richard...@intel.com>