From 9b10f9414f55dfa34a5b392490ee38c7dd6e459a Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Mon, 19 Apr 2004 20:07:25 +0000 Subject: [PATCH] (isaac_seed_start) [AVOID_USED_UNINITIALIZED_WARNINGS]: Initialize a buffer to avoid warnings from tools like valgrind. --- src/shred.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/shred.c b/src/shred.c index 42276771a7..80415bf3d5 100644 --- a/src/shred.c +++ b/src/shred.c @@ -466,7 +466,14 @@ isaac_seed_start (struct isaac_state *s) #endif for (i = 0; i < 8; i++) s->iv[i] = iv[i]; - /* We could initialize s->mm to zero, but why bother? */ + + /* Enable the following memset if you're worried about used-uninitialized + warnings involving code in isaac_refill from tools like valgrind. + Since this buffer is used to accumulate pseudo-random data, there's + no harm, and maybe even some benefit, in using it uninitialized. */ +#if AVOID_USED_UNINITIALIZED_WARNINGS + memset (s->mm, 0, sizeof s->mm); +#endif /* s->c gets used for a data pointer during the seeding phase */ s->a = s->b = s->c = 0; -- 2.47.2