summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIngo Molnar <mingo@elte.hu>2008-11-25 23:19:24 +0800
committerHerbert Xu <herbert@gondor.apana.org.au>2008-12-25 11:02:01 +1100
commit664134d2916109be76648977705a2bea3ff76427 (patch)
tree4e98a60cfd06c783dc88f1b103db9c0ff1ea1736
parent5f459f0adc7581da01cdd9b7f84db14a4559a390 (diff)
downloadlinux-664134d2916109be76648977705a2bea3ff76427.tar.gz
linux-664134d2916109be76648977705a2bea3ff76427.tar.bz2
linux-664134d2916109be76648977705a2bea3ff76427.zip
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>
-rw-r--r--crypto/testmgr.c2
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,
}
for (i = 0; i < dtcount; i++) {
- int ilen, ret, dlen = COMP_BUF_SIZE;
+ int ilen, dlen = COMP_BUF_SIZE;
memset(result, 0, sizeof (result));