diff options
author | Aidan MacDonald <amachronic@protonmail.com> | 2022-12-22 19:23:29 +0000 |
---|---|---|
committer | Aidan MacDonald <amachronic@protonmail.com> | 2024-03-31 16:57:19 +0100 |
commit | 6ffd42548bf10cda13a01555ff4fa56d4213cdf2 (patch) | |
tree | 975d6a4f88a0a3469c1450476e841ef0bf8fbb85 /firmware/common/rb-loader.c | |
parent | af644e02a151bb6d4c229cc1d4846c7ffe952135 (diff) | |
download | rockbox-6ffd42548bf10cda13a01555ff4fa56d4213cdf2.tar.gz rockbox-6ffd42548bf10cda13a01555ff4fa56d4213cdf2.zip |
multiboot: Refactor boot data validation, add version numbers
Instead of verifying the CRC before every access of the boot data,
verify the CRC once at startup and set a flag to indicate the boot
data is valid.
Also add a framework to support multiple boot protocol versions.
Firmware declares the maximum supported protocol version using a
version byte in the boot data header. The bootloader chooses the
highest version supported by it and the firmware when deciding
what boot protocol to use.
Change-Id: I810194625dc0833f026d2a23b8d64ed467fa6aca
Diffstat (limited to 'firmware/common/rb-loader.c')
-rw-r--r-- | firmware/common/rb-loader.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/firmware/common/rb-loader.c b/firmware/common/rb-loader.c index 0256f21884..61d8b1ddd2 100644 --- a/firmware/common/rb-loader.c +++ b/firmware/common/rb-loader.c | |||
@@ -112,7 +112,7 @@ int load_firmware(unsigned char* buf, const char* firmware, int buffer_size) | |||
112 | { | 112 | { |
113 | ret = load_firmware_filename(buf, filename, buffer_size); | 113 | ret = load_firmware_filename(buf, filename, buffer_size); |
114 | /* if firmware has no boot_data don't load from external drive */ | 114 | /* if firmware has no boot_data don't load from external drive */ |
115 | if (write_bootdata(buf, ret, i) <= 0) | 115 | if (!write_bootdata(buf, ret, i)) |
116 | ret = EKEY_NOT_FOUND; | 116 | ret = EKEY_NOT_FOUND; |
117 | } | 117 | } |
118 | /* if ret is valid breaks from loop to continue loading */ | 118 | /* if ret is valid breaks from loop to continue loading */ |