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

Unified Diff: src/platform/vboot_reference/utils/kernel_image.c

Issue 1320010: VBoot Reference: Add kernel rollback prevention and choosing logic. (Closed)
Patch Set: fix priority in test case Created 10 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/platform/vboot_reference/utils/firmware_image.c ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/platform/vboot_reference/utils/kernel_image.c
diff --git a/src/platform/vboot_reference/utils/kernel_image.c b/src/platform/vboot_reference/utils/kernel_image.c
index 991374f0415914e1242bb40c783202cd57a342eb..e63529162ecc630ac3a9a13b9903900f61014eb4 100644
--- a/src/platform/vboot_reference/utils/kernel_image.c
+++ b/src/platform/vboot_reference/utils/kernel_image.c
@@ -15,6 +15,7 @@
#include "file_keys.h"
#include "padding.h"
+#include "rollback_index.h"
#include "rsa_utility.h"
#include "sha_utility.h"
#include "signature_digest.h"
@@ -710,3 +711,147 @@ int AddKernelSignature(KernelImage* image,
Free(kernel_signature);
return 1;
}
+
+uint32_t GetLogicalKernelVersion(uint8_t* kernel_blob) {
+ uint8_t* kernel_ptr;
+ uint16_t kernel_key_version;
+ uint16_t kernel_version;
+ uint16_t firmware_sign_algorithm;
+ uint16_t kernel_sign_algorithm;
+ int kernel_key_signature_len;
+ int kernel_sign_key_len;
+ kernel_ptr = kernel_blob + (FIELD_LEN(magic) +
+ FIELD_LEN(header_version) +
+ FIELD_LEN(header_len));
+ Memcpy(&firmware_sign_algorithm, kernel_ptr, sizeof(firmware_sign_algorithm));
+ kernel_ptr += FIELD_LEN(firmware_sign_algorithm);
+ Memcpy(&kernel_sign_algorithm, kernel_ptr, sizeof(kernel_sign_algorithm));
+ kernel_ptr += FIELD_LEN(kernel_sign_algorithm);
+ Memcpy(&kernel_key_version, kernel_ptr, sizeof(kernel_key_version));
+
+ if (firmware_sign_algorithm >= kNumAlgorithms)
+ return 0;
+ if (kernel_sign_algorithm >= kNumAlgorithms)
+ return 0;
+ kernel_key_signature_len = siglen_map[firmware_sign_algorithm];
+ kernel_sign_key_len = RSAProcessedKeySize(kernel_sign_algorithm);
+ kernel_ptr += (FIELD_LEN(kernel_key_version) +
+ kernel_sign_key_len +
+ FIELD_LEN(header_checksum) +
+ kernel_key_signature_len);
+ Memcpy(&kernel_version, kernel_ptr, sizeof(kernel_version));
+ return CombineUint16Pair(kernel_key_version, kernel_version);
+}
+
+void PrintKernelEntry(kernel_entry* entry) {
+ fprintf(stderr, "Boot Priority = %d\n", entry->boot_priority);
+ fprintf(stderr, "Boot Tries Remaining = %d\n", entry->boot_tries_remaining);
+ fprintf(stderr, "Boot Success Flag = %d\n", entry->boot_success_flag);
+}
+
+int VerifyKernelDriver_f(uint8_t* firmware_key_blob,
+ kernel_entry* kernelA,
+ kernel_entry* kernelB,
+ int dev_mode) {
+ int i;
+ /* Contains the logical kernel version (32-bit) which is calculated as
+ * (kernel_key_version << 16 | kernel_version) where
+ * [kernel_key_version], [firmware_version] are both 16-bit.
+ */
+ uint32_t kernelA_lversion, kernelB_lversion;
+ uint32_t min_lversion; /* Minimum of kernel A and kernel B lversion. */
+ uint32_t stored_lversion; /* Stored logical version in the TPM. */
+ kernel_entry* try_kernel[2]; /* Kernel in try order. */
+ int try_kernel_which[2]; /* Which corresponding kernel in the try order */
+ uint32_t try_kernel_lversion[2]; /* Their logical versions. */
+
+ /* [kernel_to_boot] will eventually contain the boot path to follow
+ * and is returned to the caller. Initially, we set it to recovery. If
+ * a valid bootable kernel is found, it will be set to that. */
+ int kernel_to_boot = BOOT_KERNEL_RECOVERY_CONTINUE;
+
+
+ /* The TPM must already have be initialized, so no need to call SetupTPM(). */
+
+ /* We get the key versions by reading directly from the image blobs without
+ * any additional (expensive) sanity checking on the blob since it's faster to
+ * outright reject a kernel with an older kernel key version. A malformed
+ * or corrupted kernel blob will still fail when VerifyKernel() is called
+ * on it.
+ */
+ kernelA_lversion = GetLogicalKernelVersion(kernelA->kernel_blob);
+ kernelB_lversion = GetLogicalKernelVersion(kernelB->kernel_blob);
+ min_lversion = Min(kernelA_lversion, kernelB_lversion);
+ stored_lversion = CombineUint16Pair(GetStoredVersion(KERNEL_KEY_VERSION),
+ GetStoredVersion(KERNEL_VERSION));
+
+ /* TODO(gauravsh): The kernel entries kernelA and kernelB come from the
+ * partition table - verify its signature/checksum before proceeding
+ * further. */
+
+ /* The logic for deciding which kernel to boot from is taken from the
+ * the Chromium OS Drive Map design document.
+ *
+ * We went to consider the kernels in their according to their boot
+ * priority attribute value.
+ */
+
+ if (kernelA->boot_priority >= kernelB->boot_priority) {
+ try_kernel[0] = kernelA;
+ try_kernel_which[0] = BOOT_KERNEL_A_CONTINUE;
+ try_kernel_lversion[0] = kernelA_lversion;
+ try_kernel[1] = kernelB;
+ try_kernel_which[1] = BOOT_KERNEL_B_CONTINUE;
+ try_kernel_lversion[1] = kernelB_lversion;
+ } else {
+ try_kernel[0] = kernelB;
+ try_kernel_which[0] = BOOT_KERNEL_B_CONTINUE;
+ try_kernel_lversion[0] = kernelB_lversion;
+ try_kernel[1] = kernelA;
+ try_kernel_which[1] = BOOT_KERNEL_A_CONTINUE;
+ try_kernel_lversion[1] = kernelA_lversion;
+ }
+
+ /* TODO(gauravsh): Changes to boot_tries_remaining and boot_priority
+ * below should be propagated to partition table. This will be added
+ * once the firmware parition table parsing code is in. */
+ for (i = 0; i < 2; i++) {
+ if ((try_kernel[i]->boot_success_flag ||
+ try_kernel[i]->boot_tries_remaining) &&
+ (VERIFY_KERNEL_SUCCESS == VerifyKernel(firmware_key_blob,
+ try_kernel[i]->kernel_blob,
+ dev_mode))) {
+ if (try_kernel[i]->boot_tries_remaining > 0)
+ try_kernel[i]->boot_tries_remaining--;
+ if (stored_lversion > try_kernel_lversion[i])
+ continue; /* Rollback: I am afraid I can't let you do that Dave. */
+ if (i == 0 && (stored_lversion < try_kernel_lversion[1])) {
+ /* The higher priority kernel is valid and bootable, See if we
+ * need to update the stored version for rollback prevention. */
+ if (VERIFY_KERNEL_SUCCESS == VerifyKernel(firmware_key_blob,
+ try_kernel[1]->kernel_blob,
+ dev_mode)) {
+ WriteStoredVersion(KERNEL_KEY_VERSION,
+ (uint16_t) (min_lversion >> 16));
+ WriteStoredVersion(KERNEL_VERSION,
+ (uint16_t) (min_lversion & 0xFFFF));
+ stored_lversion = min_lversion; /* Update stored version as it's
+ * used later. */
+ }
+ }
+ kernel_to_boot = try_kernel_which[i];
+ break; /* We found a valid kernel. */
+ }
+ try_kernel[i]->boot_priority = 0;
+ } /* for loop. */
+
+ /* Lock Kernel TPM rollback indices from further writes.
+ * TODO(gauravsh): Figure out if these can be combined into one
+ * 32-bit location since we seem to always use them together. This can help
+ * us minimize the number of NVRAM writes/locks (which are limited over flash
+ * memory lifetimes.
+ */
+ LockStoredVersion(KERNEL_KEY_VERSION);
+ LockStoredVersion(KERNEL_VERSION);
+ return kernel_to_boot;
+}
« no previous file with comments | « src/platform/vboot_reference/utils/firmware_image.c ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698