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

Issue 6632013: CHROMIUM: verity: add BUG in check_block for consistency with verify_path (Closed)

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

Description

CHROMIUM: verity: add BUG in check_block for consistency with verify_path We assume that the caller guarantees that all nodes along the path are already populated. We already have a BUG_ON checking this in verify_path. This change adds a similar BUG_ON in check_path and removes the code which checks the entry state. TBR=wad(code already LGTMed and submitted to kernel.git) BUG=9752 TEST=Ran unit tests in dm-verity.git. Ran platform_DMVerityCorruption on H/W. Change-Id: Id456d9e495978c93d0be28bd895e5ced28d9973b Signed-off-by: Mandeep Singh Baines <msb@chromium.org>; kernel.git Review URL: http://codereview.chromium.org/6246094 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=653cdca

Patch Set 1 #

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

Messages

Total messages: 1 (0 generated)
Mandeep Singh Baines
9 years, 9 months ago (2011-03-04 21:30:12 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698