On Thu, 26 Feb 2026 15:33:32 +0100
Maxime Leroy <[email protected]> wrote:
> Fix three bugs in dump_err_pkts():
>
> 1. NULL pointer dereference: mbuf was checked for NULL but then
> dereferenced unconditionally on the next line.
>
> 2. Memory leak on multi-segment packets: the while loop walked
> mbuf to NULL, so the subsequent rte_pktmbuf_free(mbuf) was a
> no-op and the packet was never freed. Use a separate iterator
> variable and free the original mbuf pointer.
>
> 3. Segment index not reset between packets: variable i was
> initialized once at function scope and never reset inside the
> do/while loop, so hexdump titles had wrong segment numbers.
> Make it local to the multi-segment block.
>
> Not tested, found by code review.
>
> Fixes: 8b49427e73 ("net/dpaa2: add Rx error queue support")
> Cc: [email protected]
You need to use longer (12 character) SHAs on Fixes tags now.
Same problem in patch 14
