wifi: p54: add missing parentheses in p54_flush()
[ Upstream commitbcfd9d7f68
] The assignment of the value to the variable total in the loop condition must be enclosed in additional parentheses, since otherwise, in accordance with the precedence of the operators, the conjunction will be performed first, and only then the assignment. Due to this error, a warning later in the function after the loop may not occur in the situation when it should. Found by Linux Verification Center (linuxtesting.org) with SVACE. Signed-off-by: Rustam Subkhankulov <subkhankulov@ispras.ru> Fixes:0d4171e215
("p54: implement flush callback") Acked-by: Christian Lamparter <chunkeey@gmail.com> Signed-off-by: Kalle Valo <kvalo@kernel.org> Link: https://lore.kernel.org/r/20220714134831.106004-1-subkhankulov@ispras.ru Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
56924fc19d
commit
a06c98c47e
@ -682,7 +682,7 @@ static void p54_flush(struct ieee80211_hw *dev, struct ieee80211_vif *vif,
|
||||
* queues have already been stopped and no new frames can sneak
|
||||
* up from behind.
|
||||
*/
|
||||
while ((total = p54_flush_count(priv) && i--)) {
|
||||
while ((total = p54_flush_count(priv)) && i--) {
|
||||
/* waste time */
|
||||
msleep(20);
|
||||
}
|
||||
|
Reference in New Issue
Block a user