aboutsummaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorSergey Ryazanov <ryazanov.s.a@gmail.com>2017-10-11 01:07:08 +0300
committerJohn Crispin <john@phrozen.org>2017-12-14 09:29:30 +0100
commit1a51241cb1be87a45cac12753a437f786329afc6 (patch)
treed288ae4797636105169c8dd4eeab0dd749c795d1 /tools
parent4d8a8c80d91e25b98fee5dc7cb298c33b7213b7f (diff)
downloadupstream-1a51241cb1be87a45cac12753a437f786329afc6.tar.gz
upstream-1a51241cb1be87a45cac12753a437f786329afc6.tar.bz2
upstream-1a51241cb1be87a45cac12753a437f786329afc6.zip
firmware-utils: mktplinkfw: fix JFFS2 EOF markers
mktplinkfw/mktplinkfw2 utilities put JFFS2 EOF market only at 64KB boundary, this could lead to current device configuration lost during the sysupgrade on a device, which is equpped with flash with the 4KB erase block size (e.g. TP-Link Archer C20). This happens when 64KB and 4KB alignments do not match, so the JFFS2 data is written not exactly at the partition beginnig and startup scripts can not find the JFFS2 during the first boot just after the sysupgrade. Fix this by placing additional JFFS2 EOF marker at a 4KB boundary. Also keep the marker at 64KB intact, so the utilities will produce images suitable for devices with both 4KB and 64KB erase blocks. Fixes: 29a2c2ea80441895a2ffe100d854d2b26d5fa606 (add ability to put jffs2 eof marker into the image) Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
Diffstat (limited to 'tools')
-rw-r--r--tools/firmware-utils/src/mktplinkfw-lib.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/firmware-utils/src/mktplinkfw-lib.c b/tools/firmware-utils/src/mktplinkfw-lib.c
index b2b6de26db..e3213274c1 100644
--- a/tools/firmware-utils/src/mktplinkfw-lib.c
+++ b/tools/firmware-utils/src/mktplinkfw-lib.c
@@ -121,7 +121,7 @@ static int pad_jffs2(char *buf, int currlen, int maxlen)
uint32_t pad_mask;
len = currlen;
- pad_mask = (64 * 1024);
+ pad_mask = (4 * 1024) | (64 * 1024); /* EOF at 4KB and at 64KB */
while ((len < maxlen) && (pad_mask != 0)) {
uint32_t mask;
int i;