Re: [PATCH] wifi: p54: fix incorrect length check in p54_find_ie()
From: Wang Yan
Date: Mon Sep 07 2026 - 02:20:58 EST
Hi Christian,
Thank you for the thorough review and for tracing the issue back to the original commit.
I agree with your analysis. The while loop together with the bounds check inside the loop body already ensures that no IE pointer is returned unless it lies within skb->len, making the initial length check redundant. As you also pointed out, the check itself is incorrect because sizeof(mgmt) evaluates to the pointer size rather than the size of struct ieee80211_mgmt.
I have posted a new patch that removes the superfluous check and updates the Fixes tag to e5ea92a7528d ("p54: AP & Ad‑hoc testing") as you suggested.
Thanks again for the review.
Regards,
Wang Yan