diff options
| author | Emmanuel Grumbach <[email protected]> | 2008-06-26 09:13:46 +0000 |
|---|---|---|
| committer | John W. Linville <[email protected]> | 2008-06-27 18:49:52 +0000 |
| commit | 00eb7fe77eb455f807c396f9917f0f623d4c84bb (patch) | |
| tree | ce67292589f9c387ff31203db36cb220202a7ee7 | |
| parent | prism: islpci_eth.c endianness fix (diff) | |
| download | kernel-00eb7fe77eb455f807c396f9917f0f623d4c84bb.tar.gz kernel-00eb7fe77eb455f807c396f9917f0f623d4c84bb.zip | |
mac80211: fix an oops in several failure paths in key allocation
This patch fixes an oops in several failure paths in key allocation. This
Oops occurs when freeing a key that has not been linked yet, so the
key->sdata is not set.
Signed-off-by: Emmanuel Grumbach <[email protected]>
Signed-off-by: Tomas Winkler <[email protected]>
Acked-by: Johannes Berg <[email protected]>
Signed-off-by: John W. Linville <[email protected]>
| -rw-r--r-- | net/mac80211/key.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/net/mac80211/key.c b/net/mac80211/key.c index 150d66dbda9d..220e83be3ef4 100644 --- a/net/mac80211/key.c +++ b/net/mac80211/key.c @@ -380,6 +380,15 @@ void ieee80211_key_free(struct ieee80211_key *key) if (!key) return; + if (!key->sdata) { + /* The key has not been linked yet, simply free it + * and don't Oops */ + if (key->conf.alg == ALG_CCMP) + ieee80211_aes_key_free(key->u.ccmp.tfm); + kfree(key); + return; + } + spin_lock_irqsave(&key->sdata->local->key_lock, flags); __ieee80211_key_free(key); spin_unlock_irqrestore(&key->sdata->local->key_lock, flags); |