From e775da1437a9532765046baffdbc22d5252c0cc7 Mon Sep 17 00:00:00 2001 From: Justin Pettit Date: Tue, 14 Jul 2009 00:25:44 -0700 Subject: [PATCH] Fix unitialized variable in coverage_log() When providing the ability to force coverage printouts to occur, some code was moved around that allowed the "hash" variable to be used unitialized. This fixes that. Thanks to Ben for pointing out the problem. Bug #1577 --- lib/coverage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/coverage.c b/lib/coverage.c index 6f805e9e..ba5d68d8 100644 --- a/lib/coverage.c +++ b/lib/coverage.c @@ -134,8 +134,8 @@ coverage_log(enum vlog_level level, bool suppress_dups) return; } + hash = coverage_hash(); if (suppress_dups) { - hash = coverage_hash(); if (coverage_hit(hash)) { VLOG(level, "Skipping details of duplicate event coverage for " "hash=%08"PRIx32" in epoch %u", hash, epoch); -- 2.30.2