nl/mac80211: allow zero plink timeout to disable STA expiration
authorMasashi Honma <masashi.honma@gmail.com>
Tue, 24 Feb 2015 13:42:16 +0000 (22:42 +0900)
committerJohannes Berg <johannes.berg@intel.com>
Sat, 28 Feb 2015 20:31:10 +0000 (21:31 +0100)
commit31f909a2c0abfc1a1a76b2981d28ac85d33210e7
tree204d09f9f14303ca93fb38c5526185b4cbd2ee3b
parentcd37a90b2a417e5882414e19954eeed174aa4d29
nl/mac80211: allow zero plink timeout to disable STA expiration

Both wpa_supplicant and mac80211 have and inactivity timer. By default
wpa_supplicant will be timed out in 5 minutes and mac80211's it is 30
minutes. If wpa_supplicant uses a longer timer than mac80211, it will
get unexpected disconnection by mac80211.

Using 0xffffffff instead as the configured value could solve this w/o
changing the code, but due to integer overflow in the expression used
this doesn't work. The expression is:

(current jiffies) > (frame Rx jiffies + NL80211_MESHCONF_PLINK_TIMEOUT * 250)

On 32bit system, the right side would overflow and be a very small
value if NL80211_MESHCONF_PLINK_TIMEOUT is sufficiently large,
causing unexpectedly early disconnections.

Instead allow disabling the inactivity timer to avoid this situation,
by passing the (previously invalid and useless) value 0.

Signed-off-by: Masashi Honma <masashi.honma@gmail.com>
[reword/rewrap commit log]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
include/uapi/linux/nl80211.h
net/mac80211/mesh.c
net/wireless/nl80211.c