summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJesper Juhl <jj@chaosbits.net>2012-01-29 23:39:22 +0100
committerHerbert Xu <herbert@gondor.apana.org.au>2012-02-05 15:12:25 +1100
commit0cfdec7a65c98de6637e547a04a33969dc9c61b1 (patch)
tree052e9b64882142a535e009f647235dadb0d9f1de
parent65a4a573b74f8f27e33a99c21a5a5d2fc4d82a1a (diff)
downloadop-kernel-dev-0cfdec7a65c98de6637e547a04a33969dc9c61b1.zip
op-kernel-dev-0cfdec7a65c98de6637e547a04a33969dc9c61b1.tar.gz
crypto: In crypto_add_alg(), 'exact' wants to be initialized to 0
We declare 'exact' without initializing it and then do: [...] if (strlen(p->cru_driver_name)) exact = 1; if (priority && !exact) return -EINVAL; [...] If the first 'if' is not true, then the second will test an uninitialized 'exact'. As far as I can tell, what we want is for 'exact' to be initialized to 0 (zero/false). Signed-off-by: Jesper Juhl <jj@chaosbits.net> Acked-by: Steffen Klassert <steffen.klassert@secunet.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r--crypto/crypto_user.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/crypto_user.c b/crypto/crypto_user.c
index 3ba6ef50..3e61cc1 100644
--- a/crypto/crypto_user.c
+++ b/crypto/crypto_user.c
@@ -304,7 +304,7 @@ static int crypto_del_alg(struct sk_buff *skb, struct nlmsghdr *nlh,
static int crypto_add_alg(struct sk_buff *skb, struct nlmsghdr *nlh,
struct nlattr **attrs)
{
- int exact;
+ int exact = 0;
const char *name;
struct crypto_alg *alg;
struct crypto_user_alg *p = nlmsg_data(nlh);
OpenPOWER on IntegriCloud