diff options
| author | David Carlier <devnexen@gmail.com> | 2026-08-26 00:17:09 +0100 |
|---|---|---|
| committer | Jeff Chen <jeff.chen_1@nxp.com> | 2026-10-06 21:13:54 +0800 |
| commit | 32d1a000e99612e010335760363936d85591fd69 (patch) | |
| tree | fcaab94ce70ecc6dc235a590585b97416cd6ebcd /tools/perf/scripts/python/task-analyzer.py | |
| parent | 7fa5fb9976c4532007a666624dc241021f74c8a9 (diff) | |
| download | linux-32d1a000e99612e010335760363936d85591fd69.tar.gz linux-32d1a000e99612e010335760363936d85591fd69.zip | |
wifi: nxpwifi: fix inverted check in Tx BA stream entry deletion
nxpwifi_is_tx_ba_stream_ptr_valid() returns true when the entry is still
linked, and every caller passes an entry that is on the list, so the early
return always fires and nothing is ever unlinked or freed. Entries leak on
every teardown and, since nxpwifi_space_avail_for_new_ba_stream() counts
them, Tx aggregation stops being negotiated once the stale count reaches
the maximum.
Changing the original dead && test to || to silence a NULL dereference
report inverted the validity test along with it.
Fixes: 00c786a7581e ("wifi: nxpwifi: fix multiple static analysis errors and warnings")
Assisted-by: Claude:claude-opus-5
Signed-off-by: David Carlier <devnexen@gmail.com>
Signed-off-by: Jeff Chen <jeff.chen_1@nxp.com>
Diffstat (limited to 'tools/perf/scripts/python/task-analyzer.py')
0 files changed, 0 insertions, 0 deletions
