diff options
author | Ingo Molnar <mingo@elte.hu> | 2008-11-25 10:19:24 -0500 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2008-12-24 19:02:01 -0500 |
commit | 664134d2916109be76648977705a2bea3ff76427 (patch) | |
tree | 4e98a60cfd06c783dc88f1b103db9c0ff1ea1736 /crypto/testmgr.c | |
parent | 5f459f0adc7581da01cdd9b7f84db14a4559a390 (diff) |
crypto: testmgr - Fix error flow of test_comp
This warning:
crypto/testmgr.c: In function ‘test_comp’:
crypto/testmgr.c:829: warning: ‘ret’ may be used uninitialized in this function
triggers because GCC correctly notices that in the ctcount == 0 &&
dtcount != 0 input condition case this function can return an undefined
value, if the second loop fails.
Remove the shadowed 'ret' variable from the second loop that was probably
unintended.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/testmgr.c')
-rw-r--r-- | crypto/testmgr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/testmgr.c b/crypto/testmgr.c index 67dce77d49d5..67ff4aaa3c9c 100644 --- a/crypto/testmgr.c +++ b/crypto/testmgr.c | |||
@@ -853,7 +853,7 @@ static int test_comp(struct crypto_comp *tfm, struct comp_testvec *ctemplate, | |||
853 | } | 853 | } |
854 | 854 | ||
855 | for (i = 0; i < dtcount; i++) { | 855 | for (i = 0; i < dtcount; i++) { |
856 | int ilen, ret, dlen = COMP_BUF_SIZE; | 856 | int ilen, dlen = COMP_BUF_SIZE; |
857 | 857 | ||
858 | memset(result, 0, sizeof (result)); | 858 | memset(result, 0, sizeof (result)); |
859 | 859 | ||