diff options
| author | Petr Štetiar | 2019-07-19 13:52:46 +0000 |
|---|---|---|
| committer | Petr Štetiar | 2019-07-19 14:16:15 +0000 |
| commit | 93728908c4854f1138e81878c519221c309020bd (patch) | |
| tree | ccd1cd4fdfefe1c0a01810efe5f7602ec3a1a78b | |
| parent | 2d8ad31974f25e2420f611f64768ab808d375daa (diff) | |
| download | firmware-utils-93728908c4854f1138e81878c519221c309020bd.tar.gz | |
firmware-utils: mkfwimage: provide human readable error
While looking at the ath25 build breakage of 19.07 images today I've
encountered following error:
mkfwimage -B XS5 -v [...] ath25-ubnt5-squashfs-sysupgrade.bin.new
ERROR: Failed creating firmware layout description - error code: -2
Which is barely human readable and needs poking into the source code, so
this patch makes the error more verbose and usable by mere mortals.
Signed-off-by: Petr Štetiar <ynezz@true.cz>
| -rw-r--r-- | src/mkfwimage.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/mkfwimage.c b/src/mkfwimage.c index c919a2a..6f993d1 100644 --- a/src/mkfwimage.c +++ b/src/mkfwimage.c @@ -297,6 +297,7 @@ static u_int32_t filelength(const char* file) static int create_image_layout(const char* kernelfile, const char* rootfsfile, char* board_name, image_info_t* im) { + uint32_t rootfs_len = 0; part_data_t* kernel = &im->parts[0]; part_data_t* rootfs = &im->parts[1]; @@ -311,8 +312,13 @@ static int create_image_layout(const char* kernelfile, const char* rootfsfile, c kernel->partition_entryaddr = p->kern_entry; strncpy(kernel->filename, kernelfile, sizeof(kernel->filename)); - if (filelength(rootfsfile) + kernel->partition_length > p->firmware_max_length) + rootfs_len = filelength(rootfsfile); + if (rootfs_len + kernel->partition_length > p->firmware_max_length) { + ERROR("File '%s' too big (0x%08X) - max size: 0x%08X (exceeds %u bytes)\n", + rootfsfile, rootfs_len, p->firmware_max_length, + (rootfs_len + kernel->partition_length) - p->firmware_max_length); return (-2); + } strcpy(rootfs->partition_name, "rootfs"); rootfs->partition_index = 2; |