ieee80211: cleanup crypto list handling, other minor cleanups.

This commit is contained in:
Christoph Hellwig 2005-11-09 01:01:04 -05:00 committed by Jeff Garzik
parent c2a8fad433
commit e3305626e0
2 changed files with 40 additions and 109 deletions

View File

@ -31,6 +31,7 @@ enum {
struct ieee80211_crypto_ops { struct ieee80211_crypto_ops {
const char *name; const char *name;
struct list_head list;
/* init new crypto context (e.g., allocate private data space, /* init new crypto context (e.g., allocate private data space,
* select IV, etc.); returns NULL on failure or pointer to allocated * select IV, etc.); returns NULL on failure or pointer to allocated

View File

@ -11,15 +11,14 @@
* *
*/ */
#include <linux/config.h> #include <linux/errno.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/slab.h> #include <linux/slab.h>
#include <asm/string.h> #include <linux/string.h>
#include <asm/errno.h>
#include <net/ieee80211.h> #include <net/ieee80211.h>
MODULE_AUTHOR("Jouni Malinen"); MODULE_AUTHOR("Jouni Malinen");
MODULE_DESCRIPTION("HostAP crypto"); MODULE_DESCRIPTION("HostAP crypto");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
@ -29,32 +28,20 @@ struct ieee80211_crypto_alg {
struct ieee80211_crypto_ops *ops; struct ieee80211_crypto_ops *ops;
}; };
struct ieee80211_crypto { static LIST_HEAD(ieee80211_crypto_algs);
struct list_head algs; static DEFINE_SPINLOCK(ieee80211_crypto_lock);
spinlock_t lock;
};
static struct ieee80211_crypto *hcrypt;
void ieee80211_crypt_deinit_entries(struct ieee80211_device *ieee, int force) void ieee80211_crypt_deinit_entries(struct ieee80211_device *ieee, int force)
{ {
struct list_head *ptr, *n; struct ieee80211_crypt_data *entry, *next;
struct ieee80211_crypt_data *entry;
unsigned long flags; unsigned long flags;
spin_lock_irqsave(&ieee->lock, flags); spin_lock_irqsave(&ieee->lock, flags);
list_for_each_entry_safe(entry, next, &ieee->crypt_deinit_list, list) {
if (list_empty(&ieee->crypt_deinit_list))
goto unlock;
for (ptr = ieee->crypt_deinit_list.next, n = ptr->next;
ptr != &ieee->crypt_deinit_list; ptr = n, n = ptr->next) {
entry = list_entry(ptr, struct ieee80211_crypt_data, list);
if (atomic_read(&entry->refcnt) != 0 && !force) if (atomic_read(&entry->refcnt) != 0 && !force)
continue; continue;
list_del(ptr); list_del(&entry->list);
if (entry->ops) { if (entry->ops) {
entry->ops->deinit(entry->priv); entry->ops->deinit(entry->priv);
@ -62,7 +49,6 @@ void ieee80211_crypt_deinit_entries(struct ieee80211_device *ieee, int force)
} }
kfree(entry); kfree(entry);
} }
unlock:
spin_unlock_irqrestore(&ieee->lock, flags); spin_unlock_irqrestore(&ieee->lock, flags);
} }
@ -125,9 +111,6 @@ int ieee80211_register_crypto_ops(struct ieee80211_crypto_ops *ops)
unsigned long flags; unsigned long flags;
struct ieee80211_crypto_alg *alg; struct ieee80211_crypto_alg *alg;
if (hcrypt == NULL)
return -1;
alg = kmalloc(sizeof(*alg), GFP_KERNEL); alg = kmalloc(sizeof(*alg), GFP_KERNEL);
if (alg == NULL) if (alg == NULL)
return -ENOMEM; return -ENOMEM;
@ -135,9 +118,9 @@ int ieee80211_register_crypto_ops(struct ieee80211_crypto_ops *ops)
memset(alg, 0, sizeof(*alg)); memset(alg, 0, sizeof(*alg));
alg->ops = ops; alg->ops = ops;
spin_lock_irqsave(&hcrypt->lock, flags); spin_lock_irqsave(&ieee80211_crypto_lock, flags);
list_add(&alg->list, &hcrypt->algs); list_add(&alg->list, &ieee80211_crypto_algs);
spin_unlock_irqrestore(&hcrypt->lock, flags); spin_unlock_irqrestore(&ieee80211_crypto_lock, flags);
printk(KERN_DEBUG "ieee80211_crypt: registered algorithm '%s'\n", printk(KERN_DEBUG "ieee80211_crypt: registered algorithm '%s'\n",
ops->name); ops->name);
@ -147,64 +130,49 @@ int ieee80211_register_crypto_ops(struct ieee80211_crypto_ops *ops)
int ieee80211_unregister_crypto_ops(struct ieee80211_crypto_ops *ops) int ieee80211_unregister_crypto_ops(struct ieee80211_crypto_ops *ops)
{ {
struct ieee80211_crypto_alg *alg;
unsigned long flags; unsigned long flags;
struct list_head *ptr;
struct ieee80211_crypto_alg *del_alg = NULL;
if (hcrypt == NULL) spin_lock_irqsave(&ieee80211_crypto_lock, flags);
return -1; list_for_each_entry(alg, &ieee80211_crypto_algs, list) {
if (alg->ops == ops)
spin_lock_irqsave(&hcrypt->lock, flags); goto found;
for (ptr = hcrypt->algs.next; ptr != &hcrypt->algs; ptr = ptr->next) {
struct ieee80211_crypto_alg *alg =
(struct ieee80211_crypto_alg *)ptr;
if (alg->ops == ops) {
list_del(&alg->list);
del_alg = alg;
break;
}
} }
spin_unlock_irqrestore(&hcrypt->lock, flags); spin_unlock_irqrestore(&ieee80211_crypto_lock, flags);
return -EINVAL;
if (del_alg) { found:
printk(KERN_DEBUG "ieee80211_crypt: unregistered algorithm " printk(KERN_DEBUG "ieee80211_crypt: unregistered algorithm "
"'%s'\n", ops->name); "'%s'\n", ops->name);
kfree(del_alg); list_del(&alg->list);
} spin_unlock_irqrestore(&ieee80211_crypto_lock, flags);
kfree(alg);
return del_alg ? 0 : -1; return 0;
} }
struct ieee80211_crypto_ops *ieee80211_get_crypto_ops(const char *name) struct ieee80211_crypto_ops *ieee80211_get_crypto_ops(const char *name)
{ {
struct ieee80211_crypto_alg *alg;
unsigned long flags; unsigned long flags;
struct list_head *ptr;
struct ieee80211_crypto_alg *found_alg = NULL;
if (hcrypt == NULL) spin_lock_irqsave(&ieee80211_crypto_lock, flags);
return NULL; list_for_each_entry(alg, &ieee80211_crypto_algs, list) {
if (strcmp(alg->ops->name, name) == 0)
spin_lock_irqsave(&hcrypt->lock, flags); goto found;
for (ptr = hcrypt->algs.next; ptr != &hcrypt->algs; ptr = ptr->next) {
struct ieee80211_crypto_alg *alg =
(struct ieee80211_crypto_alg *)ptr;
if (strcmp(alg->ops->name, name) == 0) {
found_alg = alg;
break;
}
} }
spin_unlock_irqrestore(&hcrypt->lock, flags); spin_unlock_irqrestore(&ieee80211_crypto_lock, flags);
return NULL;
if (found_alg) found:
return found_alg->ops; spin_unlock_irqrestore(&ieee80211_crypto_lock, flags);
else return alg->ops;
return NULL;
} }
static void *ieee80211_crypt_null_init(int keyidx) static void *ieee80211_crypt_null_init(int keyidx)
{ {
return (void *)1; return (void *)1;
} }
static void ieee80211_crypt_null_deinit(void *priv) static void ieee80211_crypt_null_deinit(void *priv)
{ {
} }
@ -213,56 +181,18 @@ static struct ieee80211_crypto_ops ieee80211_crypt_null = {
.name = "NULL", .name = "NULL",
.init = ieee80211_crypt_null_init, .init = ieee80211_crypt_null_init,
.deinit = ieee80211_crypt_null_deinit, .deinit = ieee80211_crypt_null_deinit,
.encrypt_mpdu = NULL,
.decrypt_mpdu = NULL,
.encrypt_msdu = NULL,
.decrypt_msdu = NULL,
.set_key = NULL,
.get_key = NULL,
.extra_mpdu_prefix_len = 0,
.extra_mpdu_postfix_len = 0,
.owner = THIS_MODULE, .owner = THIS_MODULE,
}; };
static int __init ieee80211_crypto_init(void) static int __init ieee80211_crypto_init(void)
{ {
int ret = -ENOMEM; return ieee80211_register_crypto_ops(&ieee80211_crypt_null);
hcrypt = kmalloc(sizeof(*hcrypt), GFP_KERNEL);
if (!hcrypt)
goto out;
memset(hcrypt, 0, sizeof(*hcrypt));
INIT_LIST_HEAD(&hcrypt->algs);
spin_lock_init(&hcrypt->lock);
ret = ieee80211_register_crypto_ops(&ieee80211_crypt_null);
if (ret < 0) {
kfree(hcrypt);
hcrypt = NULL;
}
out:
return ret;
} }
static void __exit ieee80211_crypto_deinit(void) static void __exit ieee80211_crypto_deinit(void)
{ {
struct list_head *ptr, *n; ieee80211_unregister_crypto_ops(&ieee80211_crypt_null);
BUG_ON(!list_empty(&ieee80211_crypto_algs));
if (hcrypt == NULL)
return;
for (ptr = hcrypt->algs.next, n = ptr->next; ptr != &hcrypt->algs;
ptr = n, n = ptr->next) {
struct ieee80211_crypto_alg *alg =
(struct ieee80211_crypto_alg *)ptr;
list_del(ptr);
printk(KERN_DEBUG "ieee80211_crypt: unregistered algorithm "
"'%s' (deinit)\n", alg->ops->name);
kfree(alg);
}
kfree(hcrypt);
} }
EXPORT_SYMBOL(ieee80211_crypt_deinit_entries); EXPORT_SYMBOL(ieee80211_crypt_deinit_entries);