diff options
-rw-r--r-- | firmware/lib/vboot_kernel.c | 15 | ||||
-rw-r--r-- | tests/vboot_kernel_tests.c | 10 |
2 files changed, 1 insertions, 24 deletions
diff --git a/firmware/lib/vboot_kernel.c b/firmware/lib/vboot_kernel.c index 35a3df50..c0afd37b 100644 --- a/firmware/lib/vboot_kernel.c +++ b/firmware/lib/vboot_kernel.c @@ -435,13 +435,6 @@ VbError_t LoadKernel(LoadKernelParams *params, VbCommonParams *cparams) VbError_t retval = VBERROR_UNKNOWN; int recovery = VBNV_RECOVERY_LK_UNSPECIFIED; - /* Sanity Checks */ - if (!params->bytes_per_lba || !params->streaming_lba_count) { - VB2_DEBUG("LoadKernel() called with invalid params\n"); - retval = VBERROR_INVALID_PARAMETER; - goto load_kernel_exit; - } - /* Clear output params in case we fail */ params->partition_number = 0; params->bootloader_address = 0; @@ -673,17 +666,11 @@ load_kernel_exit: VBERROR_SUCCESS != retval ? recovery : VBNV_RECOVERY_NOT_REQUESTED); - /* - * If LoadKernel() was called with bad parameters, shcall may not be - * initialized. - */ - if (shcall) - shcall->return_code = (uint8_t)retval; - /* Store how much shared data we used, if any */ cparams->shared_data_size = shared->data_used; free(recovery_key); + shcall->return_code = (uint8_t)retval; return retval; } 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 @@ -570,16 +570,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, "Bad GPT"); |