commit 723d568aa585028a145c79a744dba2e018815873
author Johannes Berg <johannes.berg@intel.com> Tue Feb 26 13:56:40 2013 +0100
committer Johannes Berg <johannes.berg@intel.com> Wed Mar 06 16:35:50 2013 +0100
tree f6fc5ffdab309a7c6c3b8eccc08abd5e66f65f61
parent d339d5ca8eee34f3c70386cf2545edc53e546a13

cfg80211: prohibit zero keepalive interval

It's not useful to specify a 0 keepalive interval, this
would send too much data. Prohibit this to also avoid
device issues.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>

net/wireless/nl80211.c

diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
index 3acde3f..a8bd453 100644
--- a/net/wireless/nl80211.c
+++ b/net/wireless/nl80211.c
@@ -7636,7 +7636,8 @@
 		return -EINVAL;
 
 	if (nla_get_u32(tb[NL80211_WOWLAN_TCP_DATA_INTERVAL]) >
-			rdev->wiphy.wowlan.tcp->data_interval_max)
+			rdev->wiphy.wowlan.tcp->data_interval_max ||
+	    nla_get_u32(tb[NL80211_WOWLAN_TCP_DATA_INTERVAL]) == 0)
 		return -EINVAL;
 
 	wake_size = nla_len(tb[NL80211_WOWLAN_TCP_WAKE_PAYLOAD]);