summaryrefslogtreecommitdiffstats
path: root/rbutil/rbutilqt/base/bootloaderinstallams.cpp
diff options
context:
space:
mode:
authorRafaël Carré <rafael.carre@gmail.com>2010-02-19 14:10:26 +0000
committerRafaël Carré <rafael.carre@gmail.com>2010-02-19 14:10:26 +0000
commitb6c20c17e462d29192ea579ee061af8437457bac (patch)
tree8a4320a326623264a7424db75444b190850d1168 /rbutil/rbutilqt/base/bootloaderinstallams.cpp
parent5bae969a3e7d2bbc3119d913335fc325402f2700 (diff)
downloadrockbox-b6c20c17e462d29192ea579ee061af8437457bac.tar.gz
rockbox-b6c20c17e462d29192ea579ee061af8437457bac.zip
mkamsboot: prevents 2 potential problems
We checked if the new firmware block (bootloader+ucl function+packed bootloader & OF) fit in the OF file, but not if it would run properly. For example the Clipv2 OF is bigger than 0x50000 bytes uncompressed, but it fitted in this space when packed and concatenated to a packed bootloader + ucl function and dualboot code (but we use 1MB of RAM and not 0x50000 anyway). Now we check that both bootloader and OF are small enough to be unpacked at runtime: the unpacked data must be smaller than available memory and not overlap with ucl function and packed data (although the unpacked and packed data could probably overlap a bit, I don't know how to calculate this and this could be quite complex). total_size() is replaced by check_sizes() which will perform all the checks and set an error string if the firmware can't be patched. (both mkamsboot and rbutilqt modified accordingly) The second problem is that dualboot.S assumed r3 and r5 were left untouched in the device specific checks. This was undocumented and very error prone when modifying these checks. r3 is the last byte of packed copy (bootloader or OF) r5 is the entry point of uclunpack function derived from r3, so move r5 calculation after the device specific code. Even if r3 is currently unused in the device specific code, we store it in memory after copying the ucl function, when it points to the last byte of packed data (not yet copied at this point since we didn't chose if we boot the OF or the bootloader), and restore it just before using it so no restriction is placed on registers usage in device specific code. Add a new variable ucl_dest in dualboot.S set by mkamsboot.c, which represents the last bound of buffer where we copy the ucl function, and then the packed data (bootloader or OF). RAM_SIZE definition is moved from dualboot.S to mkamsboot.c new model_memory_size(), where it is a bit better documented. Tested on e200v2 and Clip+ git-svn-id: svn://svn.rockbox.org/rockbox/trunk@24772 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'rbutil/rbutilqt/base/bootloaderinstallams.cpp')
-rw-r--r--rbutil/rbutilqt/base/bootloaderinstallams.cpp8
1 files changed, 6 insertions, 2 deletions
diff --git a/rbutil/rbutilqt/base/bootloaderinstallams.cpp b/rbutil/rbutilqt/base/bootloaderinstallams.cpp
index 12698c221b..6d5749facd 100644
--- a/rbutil/rbutilqt/base/bootloaderinstallams.cpp
+++ b/rbutil/rbutilqt/base/bootloaderinstallams.cpp
@@ -72,6 +72,7 @@ void BootloaderInstallAms::installStage2(void)
int n;
int firmware_size;
int bootloader_size;
+ int patchable;
int totalsize;
char errstr[200];
@@ -109,10 +110,13 @@ void BootloaderInstallAms::installStage2(void)
}
/* check total size */
- totalsize = total_size(sum.model,rb_packedsize,of_packedsize);
- if (totalsize > firmware_size)
+ patchable = check_sizes(sum.model, rb_packedsize, bootloader_size,
+ of_packedsize, firmware_size, &totalsize, errstr, sizeof(errstr));
+
+ if (!patchable)
{
qDebug() << "[BootloaderInstallAms] No room to insert bootloader";
+ emit logItem(errstr, LOGERROR);
emit logItem(tr("No room to insert bootloader, try another firmware version"),
LOGERROR);
free(buf);