From eedd4293582b86a43eab6400d0a967bccf14f1fe Mon Sep 17 00:00:00 2001 From: Randall Spangler Date: Tue, 1 Nov 2016 15:29:05 -0700 Subject: vboot: Remove LoadKernel() LBA size/count checks Now that LoadKernel() uses a stream API for its partition data, it doesn't care about those fields. They're blindly passed to cgptlib_internal, which does similar checks in CheckParameters() and CheckHeader(). So, don't duplicate the checks. BUG=chromium:611535 BRANCH=none TEST=make runtests; emerge-kevin coreboot depthcharge Change-Id: I72375496e5df7b7c17df25d358f2555fe41fe520 Signed-off-by: Randall Spangler Reviewed-on: https://chromium-review.googlesource.com/407053 Reviewed-by: Daisuke Nojiri --- tests/vboot_kernel_tests.c | 10 ---------- 1 file changed, 10 deletions(-) (limited to 'tests/vboot_kernel_tests.c') diff --git a/tests/vboot_kernel_tests.c b/tests/vboot_kernel_tests.c index 3c26ee2e..d3a25d58 100644 --- a/tests/vboot_kernel_tests.c +++ b/tests/vboot_kernel_tests.c @@ -569,16 +569,6 @@ static void ReadWriteGptTest(void) */ static void InvalidParamsTest(void) { - ResetMocks(); - lkp.bytes_per_lba = 0; - TEST_EQ(LoadKernel(&lkp, &cparams), VBERROR_INVALID_PARAMETER, - "Bad lba size"); - - ResetMocks(); - lkp.streaming_lba_count = 0; - TEST_EQ(LoadKernel(&lkp, &cparams), VBERROR_INVALID_PARAMETER, - "Bad lba count"); - ResetMocks(); gpt_init_fail = 1; TEST_EQ(LoadKernel(&lkp, &cparams), VBERROR_NO_KERNEL_FOUND, -- cgit v1.2.1