Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=cf02f5da9437201d57d93f529839dd40aac8b5f9
Commit:     cf02f5da9437201d57d93f529839dd40aac8b5f9
Parent:     b5b7f08869340aa8cfa23303f7d195f161479592
Author:     Herbert Xu <[EMAIL PROTECTED]>
AuthorDate: Thu Mar 29 17:32:59 2007 +1000
Committer:  Herbert Xu <[EMAIL PROTECTED]>
CommitDate: Wed May 2 14:38:31 2007 +1000

    [CRYPTO] cryptomgr: Fix parsing of nested templates
    
    This patch allows the use of nested templates by allowing the use of
    brackets inside a template parameter.
    
    Signed-off-by: Herbert Xu <[EMAIL PROTECTED]>
---
 crypto/cryptomgr.c |   38 +++++++++++++++++++++++++-------------
 1 files changed, 25 insertions(+), 13 deletions(-)

diff --git a/crypto/cryptomgr.c b/crypto/cryptomgr.c
index 7a3df9b..6958ea8 100644
--- a/crypto/cryptomgr.c
+++ b/crypto/cryptomgr.c
@@ -14,17 +14,17 @@
 #include <linux/ctype.h>
 #include <linux/err.h>
 #include <linux/init.h>
+#include <linux/kthread.h>
 #include <linux/module.h>
 #include <linux/notifier.h>
 #include <linux/rtnetlink.h>
 #include <linux/sched.h>
 #include <linux/string.h>
-#include <linux/workqueue.h>
 
 #include "internal.h"
 
 struct cryptomgr_param {
-       struct work_struct work;
+       struct task_struct *thread;
 
        struct rtattr *tb[CRYPTOA_MAX];
 
@@ -45,10 +45,9 @@ struct cryptomgr_param {
        char template[CRYPTO_MAX_ALG_NAME];
 };
 
-static void cryptomgr_probe(struct work_struct *work)
+static int cryptomgr_probe(void *data)
 {
-       struct cryptomgr_param *param =
-               container_of(work, struct cryptomgr_param, work);
+       struct cryptomgr_param *param = data;
        struct crypto_template *tmpl;
        struct crypto_instance *inst;
        int err;
@@ -72,7 +71,7 @@ static void cryptomgr_probe(struct work_struct *work)
 
 out:
        kfree(param);
-       return;
+       module_put_and_exit(0);
 
 err:
        crypto_larval_error(param->larval.name, param->type.data.type,
@@ -87,9 +86,12 @@ static int cryptomgr_schedule_probe(struct crypto_larval 
*larval)
        const char *p;
        unsigned int len;
 
+       if (!try_module_get(THIS_MODULE))
+               goto err;
+
        param = kzalloc(sizeof(*param), GFP_KERNEL);
        if (!param)
-               goto err;
+               goto err_put_module;
 
        for (p = name; isalnum(*p) || *p == '-' || *p == '_'; p++)
                ;
@@ -101,11 +103,18 @@ static int cryptomgr_schedule_probe(struct crypto_larval 
*larval)
        memcpy(param->template, name, len);
 
        name = p + 1;
-       for (p = name; isalnum(*p) || *p == '-' || *p == '_'; p++)
-               ;
+       len = 0;
+       for (p = name; *p; p++) {
+               for (; isalnum(*p) || *p == '-' || *p == '_' || *p == '('; p++)
+                       ;
 
-       len = p - name;
-       if (!len || *p != ')' || p[1])
+               if (*p != ')')
+                       goto err_free_param;
+
+               len = p - name;
+       }
+
+       if (!len || name[len + 1])
                goto err_free_param;
 
        param->type.attr.rta_len = sizeof(param->type);
@@ -121,13 +130,16 @@ static int cryptomgr_schedule_probe(struct crypto_larval 
*larval)
 
        memcpy(param->larval.name, larval->alg.cra_name, CRYPTO_MAX_ALG_NAME);
 
-       INIT_WORK(&param->work, cryptomgr_probe);
-       schedule_work(&param->work);
+       param->thread = kthread_run(cryptomgr_probe, param, "cryptomgr");
+       if (IS_ERR(param->thread))
+               goto err_free_param;
 
        return NOTIFY_STOP;
 
 err_free_param:
        kfree(param);
+err_put_module:
+       module_put(THIS_MODULE);
 err:
        return NOTIFY_OK;
 }
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to