Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(15)

Issue 6726045: verity: fix valgrind warnings in OddLeafCount (Closed)

Created:
9 years, 9 months ago by Mandeep Singh Baines
Modified:
9 years, 7 months ago
Reviewers:
Will Drewry, Paul T
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

verity: fix valgrind warnings in OddLeafCount We don't clear the end bits (non-hash) of the last block. This results in uninitialized bits being factored into the root hash. This causes test flakiness. Fix by memset all leaf blocks after allocation. BUG=none TEST=Ran OddLeafCount with valgrind. Change-Id: I626f1ab386a67d0da8dcaff40d25245bfc747994 R=taysom@chromium.org,wad@chromium.org Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=08c5dc5

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M dm-bht.c View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Mandeep Singh Baines
9 years, 9 months ago (2011-03-25 22:11:48 UTC) #1
Paul T
9 years, 9 months ago (2011-03-25 22:15:00 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698