[PATCH 2.6.37.2 1/1] crypto: Uninitialized variable fix.

2011-02-25 Thread Atul Sowani

Array inter[4] was uninitialized in anubis_crypt() and was causing
compiler warning. Added the array initialization.

Signed-off-by: Atul Sowani sow...@gmail.com
---

diff a/crypto/anubis.c b/crypto/anubis.c
--- a/crypto/anubis.c   2011-02-25 04:39:00.0 +0530
+++ b/crypto/anubis.c   2011-02-25 23:07:12.0 +0530
@@ -578,7 +578,7 @@ static void anubis_crypt(u32 roundKey[AN
__be32 *dst = (__be32 *)ciphertext;
int i, r;
u32 state[4];
-   u32 inter[4];
+   u32 inter[4] = { 0, 0, 0, 0 };

/*
 * map plaintext block to cipher state (mu)
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2.6.37.2 1/1] crypto: Uninitialized variable fix.

2011-02-25 Thread Greg KH
On Fri, Feb 25, 2011 at 11:28:28PM +0530, Atul Sowani wrote:
 Array inter[4] was uninitialized in anubis_crypt() and was causing
 compiler warning. Added the array initialization.
 
 Signed-off-by: Atul Sowani sow...@gmail.com
 ---
 
 diff a/crypto/anubis.c b/crypto/anubis.c
 --- a/crypto/anubis.c   2011-02-25 04:39:00.0 +0530
 +++ b/crypto/anubis.c   2011-02-25 23:07:12.0 +0530
 @@ -578,7 +578,7 @@ static void anubis_crypt(u32 roundKey[AN
 __be32 *dst = (__be32 *)ciphertext;
 int i, r;
 u32 state[4];
 -   u32 inter[4];
 +   u32 inter[4] = { 0, 0, 0, 0 };

That's nice, it looks like a bug in the compiler, preinitializing this
variable doesn't fix any real bug.

sorry,

greg k-h
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html