[PATCH] app/testpmd: fix error handling in flow API commands

Stephen Hemminger stephen at networkplumber.org
Mon Sep 7 20:24:50 CEST 2026


On Mon, 7 Sep 2026 09:18:31 +0530
<kirankumark at marvell.com> wrote:

> From: Kiran Kumar K <kirankumark at marvell.com>
> 
> Fix two robustness issues in the flow API command paths:
> 
> - port_flow_configure() committed the new queue attributes and job list
>   before rte_flow_configure() succeeded, leaving the port state
>   inconsistent on failure. Build the job list locally, free it on error,
>   and update the port only after configure succeeds.
> 
> - port_flow_create() and port_queue_flow_create() indexed ports[port_id]
>   without validating the port id. Add an rte_eth_dev_is_valid_port()
>   check before accessing the array.
> 
> Fixes: 9ad3a41ab2a1 ("app/testpmd: add flow engine configuration")
> Fixes: 938a184a1870 ("app/testpmd: implement basic support for flow API")
> Fixes: ecdc927b99f2 ("app/testpmd: add async flow create/destroy operations")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Kiran Kumar K <kirankumark at marvell.com>
> ---

Applied to next-net


More information about the stable mailing list