2021-04-08 20:48:35

by Muhammad Usama Anjum

[permalink] [raw]
Subject: [PATCH] staging: rtl8723bs: Remove unnecessary code

The same code is executed when the condition "!show_errors" is true or
false. Remove the if condition from here. The original intension of this
condition was to not to report any error logs if condition was true. But
this driver doesn't has those logs added. So this if condition can be
removed.

Addresses-Coverity: ("Same code execution")
Signed-off-by: Muhammad Usama Anjum <[email protected]>
---
drivers/staging/rtl8723bs/core/rtw_ieee80211.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c b/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
index 1b60252545b3..2af66a18200d 100644
--- a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
@@ -1000,8 +1000,6 @@ enum ParseRes rtw_ieee802_11_parse_elems(u8 *start, uint len,
break;
default:
unknown++;
- if (!show_errors)
- break;
break;
}

--
2.25.1