On Wed, Jul 15, 2009 at 03:16:21PM +0800, Herbert Xu wrote:
> crypto: cryptd - Switch to template create API
>
> This patch changes cryptd to use the template->create function
> instead of alloc in anticipation for the switch to new style
> ahash algorithms.
>
> Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> ---
>
> crypto/cryptd.c | 53 ++++++++++++++++++++++++++----------------------
> crypto/internal.h | 3 --
> include/crypto/algapi.h | 3 ++
> 3 files changed, 32 insertions(+), 27 deletions(-)
>
> diff --git a/crypto/cryptd.c b/crypto/cryptd.c
> index 6e6722e..ad58f51 100644
> --- a/crypto/cryptd.c
> +++ b/crypto/cryptd.c
> @@ -287,8 +287,9 @@ out_free_inst:
> goto out;
> }
>
> -static struct crypto_instance *cryptd_alloc_blkcipher(
> - struct rtattr **tb, struct cryptd_queue *queue)
> +static int cryptd_create_blkcipher(struct crypto_template *tmpl,
> + struct rtattr **tb,
> + struct cryptd_queue *queue)
> {
> struct cryptd_instance_ctx *ctx;
> struct crypto_instance *inst;
> @@ -298,7 +299,7 @@ static struct crypto_instance *cryptd_alloc_blkcipher(
> alg = crypto_get_attr_alg(tb, CRYPTO_ALG_TYPE_BLKCIPHER,
> CRYPTO_ALG_TYPE_MASK);
> if (IS_ERR(alg))
> - return ERR_CAST(alg);
> + return PTR_ERR(alg);
>
> inst = cryptd_alloc_instance(alg, sizeof(*ctx));
> if (IS_ERR(inst))
> @@ -330,14 +331,16 @@ static struct crypto_instance *cryptd_alloc_blkcipher(
> inst->alg.cra_ablkcipher.encrypt = cryptd_blkcipher_encrypt_enqueue;
> inst->alg.cra_ablkcipher.decrypt = cryptd_blkcipher_decrypt_enqueue;
>
> + err = crypto_register_instance(tmpl, inst);
> + if (err) {
> + crypto_drop_spawn(&ctx->spawn);
> +out_free_inst:
> + kfree(inst);
> + }
> +
> out_put_alg:
> crypto_mod_put(alg);
> - return inst;
> -
> -out_free_inst:
> - kfree(inst);
> - inst = ERR_PTR(err);
> - goto out_put_alg;
> + return err;
This introduces an uninitialized return value warning. err is not initialized
if cryptd_alloc_instance() fails.
> }
>
> static int cryptd_hash_init_tfm(struct crypto_tfm *tfm)
> @@ -502,8 +505,8 @@ static int cryptd_hash_digest_enqueue(struct ahash_request *req)
> return cryptd_hash_enqueue(req, cryptd_hash_digest);
> }
>
> -static struct crypto_instance *cryptd_alloc_hash(
> - struct rtattr **tb, struct cryptd_queue *queue)
> +static int cryptd_create_hash(struct crypto_template *tmpl, struct rtattr **tb,
> + struct cryptd_queue *queue)
> {
> struct hashd_instance_ctx *ctx;
> struct crypto_instance *inst;
> @@ -513,7 +516,7 @@ static struct crypto_instance *cryptd_alloc_hash(
>
> salg = shash_attr_alg(tb[1], 0, 0);
> if (IS_ERR(salg))
> - return ERR_CAST(salg);
> + return PTR_ERR(salg);
>
> alg = &salg->base;
> inst = cryptd_alloc_instance(alg, sizeof(*ctx));
> @@ -542,34 +545,36 @@ static struct crypto_instance *cryptd_alloc_hash(
> inst->alg.cra_ahash.setkey = cryptd_hash_setkey;
> inst->alg.cra_ahash.digest = cryptd_hash_digest_enqueue;
>
> + err = crypto_register_instance(tmpl, inst);
> + if (err) {
> + crypto_drop_shash(&ctx->spawn);
> +out_free_inst:
> + kfree(inst);
> + }
> +
> out_put_alg:
> crypto_mod_put(alg);
> - return inst;
> -
> -out_free_inst:
> - kfree(inst);
> - inst = ERR_PTR(err);
> - goto out_put_alg;
> + return err;
Same here.
I'll send a patch to fix it.
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
