* [PATCH 0/2] eventdev: fix eth dev ID truncation in telemetry
@ 2026-03-04 0:34 Sergei Iashin
2026-03-04 0:34 ` [PATCH 1/2] " Sergei Iashin
2026-03-04 0:34 ` [PATCH 2/2] mailmap: add Sergei Iashin Sergei Iashin
0 siblings, 2 replies; 4+ messages in thread
From: Sergei Iashin @ 2026-03-04 0:34 UTC (permalink / raw)
Cc: dev, Sergei Iashin
This series fixes an out-of-bounds access in eventdev rx adapter telemetry
handlers caused by eth dev ID truncation.
Patches:
- eventdev: fix eth dev ID truncation in telemetry
- mailmap: add Sergei Iashin
Sergei Iashin (2):
eventdev: fix eth dev ID truncation in telemetry
mailmap: add Sergei Iashin
.mailmap | 1 +
lib/eventdev/rte_event_eth_rx_adapter.c | 12 ++++++++----
2 files changed, 9 insertions(+), 4 deletions(-)
--
2.39.5
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/2] eventdev: fix eth dev ID truncation in telemetry
2026-03-04 0:34 [PATCH 0/2] eventdev: fix eth dev ID truncation in telemetry Sergei Iashin
@ 2026-03-04 0:34 ` Sergei Iashin
2026-03-04 0:34 ` [PATCH 2/2] mailmap: add Sergei Iashin Sergei Iashin
1 sibling, 0 replies; 4+ messages in thread
From: Sergei Iashin @ 2026-03-04 0:34 UTC (permalink / raw)
To: Naga Harish K S V, Jerin Jacob, Jay Jayatheerthan,
Ganapati Kundapura, Weiguo Li
Cc: dev, Sergei Iashin, liwg06, stable
eth_dev_id is declared as int in handle_rxa_get_queue_conf(),
handle_rxa_get_queue_stats(), handle_rxa_queue_stats_reset(), and
handle_rxa_instance_get(), but is implicitly narrowed to uint16_t when
passed to rte_eth_dev_is_valid_port() via
RTE_EVENT_ETH_RX_ADAPTER_PORTID_VALID_OR_GOTO_ERR_RET().
A value like 65535 + N passes validation as N, but is then used as-is as
an array index, causing out-of-bounds access and SIGSEGV.
To reproduce (requires telemetry enabled):
echo "/eventdev/rxa_queue_conf,0,65536,0" | \
socat - UNIX-CONNECT:/var/run/dpdk/vpp/dpdk_telemetry.v2,type=5
Result:
SIGSEGV at handle_rxa_get_queue_conf + 0x34a, faulting address 0x50
This patch changes eth_dev_id from int to uint16_t to avoid truncation.
Fixes: 74b034ff8172 ("eventdev/eth_rx: fix parameters parsing memory leak")
Cc: liwg06@foxmail.com
Cc: stable@dpdk.org
Signed-off-by: Sergei Iashin <yashin.sergey@gmail.com>
---
lib/eventdev/rte_event_eth_rx_adapter.c | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c b/lib/eventdev/rte_event_eth_rx_adapter.c
index d564e14b72..2183adce6f 100644
--- a/lib/eventdev/rte_event_eth_rx_adapter.c
+++ b/lib/eventdev/rte_event_eth_rx_adapter.c
@@ -3827,7 +3827,8 @@ handle_rxa_get_queue_conf(const char *cmd __rte_unused,
{
uint8_t rx_adapter_id;
uint16_t rx_queue_id;
- int eth_dev_id, ret = -1;
+ uint16_t eth_dev_id;
+ int ret = -1;
char *token, *l_params;
struct rte_event_eth_rx_adapter_queue_conf queue_conf;
@@ -3899,7 +3900,8 @@ handle_rxa_get_queue_stats(const char *cmd __rte_unused,
{
uint8_t rx_adapter_id;
uint16_t rx_queue_id;
- int eth_dev_id, ret = -1;
+ uint16_t eth_dev_id;
+ int ret = -1;
char *token, *l_params;
struct rte_event_eth_rx_adapter_queue_stats q_stats;
@@ -3970,7 +3972,8 @@ handle_rxa_queue_stats_reset(const char *cmd __rte_unused,
{
uint8_t rx_adapter_id;
uint16_t rx_queue_id;
- int eth_dev_id, ret = -1;
+ uint16_t eth_dev_id;
+ int ret = -1;
char *token, *l_params;
if (params == NULL || strlen(params) == 0 || !isdigit(*params))
@@ -4031,7 +4034,8 @@ handle_rxa_instance_get(const char *cmd __rte_unused,
{
uint8_t instance_id;
uint16_t rx_queue_id;
- int eth_dev_id, ret = -1;
+ uint16_t eth_dev_id;
+ int ret = -1;
char *token, *l_params;
if (params == NULL || strlen(params) == 0 || !isdigit(*params))
--
2.39.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/2] mailmap: add Sergei Iashin
2026-03-04 0:34 [PATCH 0/2] eventdev: fix eth dev ID truncation in telemetry Sergei Iashin
2026-03-04 0:34 ` [PATCH 1/2] " Sergei Iashin
@ 2026-03-04 0:34 ` Sergei Iashin
2026-03-24 17:34 ` Jerin Jacob
1 sibling, 1 reply; 4+ messages in thread
From: Sergei Iashin @ 2026-03-04 0:34 UTC (permalink / raw)
To: Thomas Monjalon; +Cc: dev, Sergei Iashin
Add my name and email mapping to .mailmap
Signed-off-by: Sergei Iashin <yashin.sergey@gmail.com>
---
.mailmap | 1 +
1 file changed, 1 insertion(+)
diff --git a/.mailmap b/.mailmap
index 6c4c977dde..85db3c11af 100644
--- a/.mailmap
+++ b/.mailmap
@@ -1936,3 +1936,4 @@ Zoltan Kiss <zoltan.kiss@schaman.hu> <zoltan.kiss@linaro.org>
Zorik Machulsky <zorik@amazon.com>
Zyta Szpak <zyta@marvell.com> <zr@semihalf.com>
Zyta Szpak <zyta@marvell.com> <zyta.szpak@semihalf.com>
+Sergei Iashin <yashin.sergey@gmail.com>
--
2.39.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] mailmap: add Sergei Iashin
2026-03-04 0:34 ` [PATCH 2/2] mailmap: add Sergei Iashin Sergei Iashin
@ 2026-03-24 17:34 ` Jerin Jacob
0 siblings, 0 replies; 4+ messages in thread
From: Jerin Jacob @ 2026-03-24 17:34 UTC (permalink / raw)
To: Sergei Iashin; +Cc: Thomas Monjalon, dev
On Wed, Mar 4, 2026 at 8:46 PM Sergei Iashin <yashin.sergey@gmail.com> wrote:
>
> Add my name and email mapping to .mailmap
>
> Signed-off-by: Sergei Iashin <yashin.sergey@gmail.com>
> ---
> .mailmap | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.mailmap b/.mailmap
> index 6c4c977dde..85db3c11af 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -1936,3 +1936,4 @@ Zoltan Kiss <zoltan.kiss@schaman.hu> <zoltan.kiss@linaro.org>
> Zorik Machulsky <zorik@amazon.com>
> Zyta Szpak <zyta@marvell.com> <zr@semihalf.com>
> Zyta Szpak <zyta@marvell.com> <zyta.szpak@semihalf.com>
> +Sergei Iashin <yashin.sergey@gmail.com>
Fixed the name order by making it as alphabetical
Squashed with 1/2 patch
Applied to dpdk-next-eventdev/for-main. Thanks
> --
> 2.39.5
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-03-24 17:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-04 0:34 [PATCH 0/2] eventdev: fix eth dev ID truncation in telemetry Sergei Iashin
2026-03-04 0:34 ` [PATCH 1/2] " Sergei Iashin
2026-03-04 0:34 ` [PATCH 2/2] mailmap: add Sergei Iashin Sergei Iashin
2026-03-24 17:34 ` Jerin Jacob
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox