diff options
author | Johannes Berg <johannes.berg@intel.com> | 2014-09-10 15:28:06 -0400 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2014-09-11 06:07:34 -0400 |
commit | 78f686cae0c67a2edd167cbbe2f36017f0fa4b30 (patch) | |
tree | 7cd313b911ecaeb47289d2825563530a386f3245 /include/net | |
parent | 538c9eb8b3fd33d3a0722b2c04ec4f574eaa6e9f (diff) |
cfg80211: don't put kek/kck/replay counter on the stack
There's no need to put the values on the stack, just pass a
pointer to the data in the nl80211 message. This reduces stack
usage and avoids potential issues with putting sensitive data
on the stack.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'include/net')
-rw-r--r-- | include/net/cfg80211.h | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h index 0d17ec9df692..c2c710c14a50 100644 --- a/include/net/cfg80211.h +++ b/include/net/cfg80211.h | |||
@@ -1980,14 +1980,12 @@ struct cfg80211_wowlan_wakeup { | |||
1980 | 1980 | ||
1981 | /** | 1981 | /** |
1982 | * struct cfg80211_gtk_rekey_data - rekey data | 1982 | * struct cfg80211_gtk_rekey_data - rekey data |
1983 | * @kek: key encryption key | 1983 | * @kek: key encryption key (NL80211_KEK_LEN bytes) |
1984 | * @kck: key confirmation key | 1984 | * @kck: key confirmation key (NL80211_KCK_LEN bytes) |
1985 | * @replay_ctr: replay counter | 1985 | * @replay_ctr: replay counter (NL80211_REPLAY_CTR_LEN bytes) |
1986 | */ | 1986 | */ |
1987 | struct cfg80211_gtk_rekey_data { | 1987 | struct cfg80211_gtk_rekey_data { |
1988 | u8 kek[NL80211_KEK_LEN]; | 1988 | const u8 *kek, *kck, *replay_ctr; |
1989 | u8 kck[NL80211_KCK_LEN]; | ||
1990 | u8 replay_ctr[NL80211_REPLAY_CTR_LEN]; | ||
1991 | }; | 1989 | }; |
1992 | 1990 | ||
1993 | /** | 1991 | /** |