Commit f5aa0d21 authored by Ujjal Roy's avatar Ujjal Roy Committed by Johannes Berg

cfg80211: add sanity check for retry limit in wext-compat

Block setting the wrong values through iwconfig retry
command. Add sanity checking before sending the retry
limit to the driver.
Signed-off-by: default avatarUjjal Roy <royujjal@gmail.com>
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent bdfbec2d
...@@ -370,7 +370,7 @@ static int cfg80211_wext_siwretry(struct net_device *dev, ...@@ -370,7 +370,7 @@ static int cfg80211_wext_siwretry(struct net_device *dev,
u8 oshort = wdev->wiphy->retry_short; u8 oshort = wdev->wiphy->retry_short;
int err; int err;
if (retry->disabled || if (retry->disabled || retry->value < 1 || retry->value > 255 ||
(retry->flags & IW_RETRY_TYPE) != IW_RETRY_LIMIT) (retry->flags & IW_RETRY_TYPE) != IW_RETRY_LIMIT)
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment