Commit c0b2bbd8 authored by Johannes Berg's avatar Johannes Berg Committed by John W. Linville

nl80211: add missing parameter clearing

Jouni and Maxim reported an oops when using wpa_supplicant -Dnl80211,
which seems to be due to random data being contained in the crypto
settings for the assoc() command. This seems to be due to the missing
memset here, so add it -- it's certainly missing but I'm not 100%
certain that it will fix the problem.
Signed-off-by: default avatarJohannes Berg <johannes@sipsolutions.net>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent cbdec975
...@@ -3385,6 +3385,8 @@ static int nl80211_crypto_settings(struct genl_info *info, ...@@ -3385,6 +3385,8 @@ static int nl80211_crypto_settings(struct genl_info *info,
struct cfg80211_crypto_settings *settings, struct cfg80211_crypto_settings *settings,
int cipher_limit) int cipher_limit)
{ {
memset(settings, 0, sizeof(*settings));
settings->control_port = info->attrs[NL80211_ATTR_CONTROL_PORT]; settings->control_port = info->attrs[NL80211_ATTR_CONTROL_PORT];
if (info->attrs[NL80211_ATTR_CIPHER_SUITES_PAIRWISE]) { if (info->attrs[NL80211_ATTR_CIPHER_SUITES_PAIRWISE]) {
......
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