|SUCCESS| [2/2] vdpa/mlx5: fix RQT attributes memory leak
dpdklab at iol.unh.edu
dpdklab at iol.unh.edu
Sun Oct 25 09:53:51 CET 2026
Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/172062
_AI Code Review_
=== Patch Review: series.patch (via Claude) ===
Review date: 2026-10-25
# DPDK Patch Review
## PATCH 1/2: vdpa/mlx5: fix RSS flow leak on steering update failure
### ERRORS
None.
### WARNINGS
None.
### INFO
None.
---
## PATCH 2/2: vdpa/mlx5: fix RQT attributes memory leak
### ERRORS
None.
### WARNINGS
None.
### INFO
None.
---
# Summary
Both patches address resource leaks in error paths:
**Patch 1/2** fixes a leak of RSS steering objects (matcher, TIR, TIR action, flow) when `mlx5_vdpa_rss_flows_create()` fails. The patch adds cleanup by calling `mlx5_vdpa_rss_flows_destroy()` on the error path in `mlx5_vdpa_steer_update()`. Since `mlx5_vdpa_rss_flows_destroy()` checks each object for NULL, this is safe even when called again by `mlx5_vdpa_steer_unset()` on the `steer_setup()` error path.
**Patch 2/2** fixes a leak of RQT attributes allocated with `rte_zmalloc()` when the early-return path (`k == 0`) is taken in `mlx5_vdpa_rqt_prepare()`. The patch adds `rte_free(attr)` before the return.
Both patches correctly identify and fix resource leaks on error paths. The code changes are minimal, targeted, and follow the existing error-handling patterns. The commit messages clearly explain the bug, the impact, and the fix.
Review-Result: CLEAN
More information about the test-report
mailing list