diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-09-29 03:52:34 -0400 |
---|---|---|
committer | Kalle Valo <kvalo@codeaurora.org> | 2017-10-10 04:19:26 -0400 |
commit | 64e79426c2041b931c053a11743277a4f7f279c4 (patch) | |
tree | c50447f9af5416fe6a8e9f1dd3a2839b84cf1331 /drivers/net/wireless/realtek | |
parent | fc09785de0a364427a5df63d703bae9a306ed116 (diff) |
rtlwifi: silence underflow warning
My static checker complains that we have an upper bound but no lower
bound. I suspect neither are really required but it doesn't hurt to add
a check for negatives.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to 'drivers/net/wireless/realtek')
-rw-r--r-- | drivers/net/wireless/realtek/rtlwifi/core.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/wireless/realtek/rtlwifi/core.c b/drivers/net/wireless/realtek/rtlwifi/core.c index c53cbf3d52bd..294a6b43d1bc 100644 --- a/drivers/net/wireless/realtek/rtlwifi/core.c +++ b/drivers/net/wireless/realtek/rtlwifi/core.c | |||
@@ -453,7 +453,8 @@ static void _rtl_add_wowlan_patterns(struct ieee80211_hw *hw, | |||
453 | for (i = 0; i < wow->n_patterns; i++) { | 453 | for (i = 0; i < wow->n_patterns; i++) { |
454 | memset(&rtl_pattern, 0, sizeof(struct rtl_wow_pattern)); | 454 | memset(&rtl_pattern, 0, sizeof(struct rtl_wow_pattern)); |
455 | memset(mask, 0, MAX_WOL_BIT_MASK_SIZE); | 455 | memset(mask, 0, MAX_WOL_BIT_MASK_SIZE); |
456 | if (patterns[i].pattern_len > MAX_WOL_PATTERN_SIZE) { | 456 | if (patterns[i].pattern_len < 0 || |
457 | patterns[i].pattern_len > MAX_WOL_PATTERN_SIZE) { | ||
457 | RT_TRACE(rtlpriv, COMP_POWER, DBG_WARNING, | 458 | RT_TRACE(rtlpriv, COMP_POWER, DBG_WARNING, |
458 | "Pattern[%d] is too long\n", i); | 459 | "Pattern[%d] is too long\n", i); |
459 | continue; | 460 | continue; |