diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2014-09-10 22:28:06 +0300 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2014-09-11 12:07:34 +0200 |
| commit | 78f686cae0c67a2edd167cbbe2f36017f0fa4b30 (patch) | |
| tree | 7cd313b911ecaeb47289d2825563530a386f3245 /include/net/cfg80211.h | |
| 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/cfg80211.h')
| -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 0d17ec9df69..c2c710c14a5 100644 --- a/include/net/cfg80211.h +++ b/include/net/cfg80211.h @@ -1980,14 +1980,12 @@ struct cfg80211_wowlan_wakeup { /** * struct cfg80211_gtk_rekey_data - rekey data - * @kek: key encryption key - * @kck: key confirmation key - * @replay_ctr: replay counter + * @kek: key encryption key (NL80211_KEK_LEN bytes) + * @kck: key confirmation key (NL80211_KCK_LEN bytes) + * @replay_ctr: replay counter (NL80211_REPLAY_CTR_LEN bytes) */ struct cfg80211_gtk_rekey_data { - u8 kek[NL80211_KEK_LEN]; - u8 kck[NL80211_KCK_LEN]; - u8 replay_ctr[NL80211_REPLAY_CTR_LEN]; + const u8 *kek, *kck, *replay_ctr; }; /** |