diff options
author | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2022-01-15 20:12:56 +0100 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2022-01-24 10:35:10 -0500 |
commit | c28f2499952a78609e80d3b1f7aca9b0aab8d6d6 (patch) | |
tree | a45f95eddd0481052a20027013e5ad96e5fd8c1a | |
parent | 800f0d05e1a1bef75ca1aa8a580f1b0139bceb20 (diff) | |
download | u-boot-c28f2499952a78609e80d3b1f7aca9b0aab8d6d6.zip u-boot-c28f2499952a78609e80d3b1f7aca9b0aab8d6d6.tar.gz u-boot-c28f2499952a78609e80d3b1f7aca9b0aab8d6d6.tar.bz2 |
mkimage: struct stat.st_size may not be long
The component st_size of struct stat is of type off_t. Depending on the
system printing it using %ld leads to a warning:
tools/mkimage.c:438:54: warning: format '%ld' expects argument of type
'long int', but argument 5 has type
'off_t' {aka 'long long int'} [-Wformat=]
438 | "%s: Bad size: \"%s\" is not valid image: size %ld < %u\n",
| ~~^
| |
| long int
| %lld
When comparing an off_t value to a 32bit integer we should not convert to
uint32_t but to off_t which may be wider.
Reported-by: Milan P. Stanić <mps@arvanta.net>
Fixes: 331f0800f1a3 ("mkimage: allow -l to work on block devices on Linux")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Reviewed-by: Mark Kettenis <kettenis@openbsd.org>
-rw-r--r-- | tools/mkimage.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tools/mkimage.c b/tools/mkimage.c index ddb7933..0ec28da 100644 --- a/tools/mkimage.c +++ b/tools/mkimage.c @@ -435,11 +435,12 @@ int main(int argc, char **argv) params.cmdname, params.imagefile); exit (EXIT_FAILURE); #endif - } else if ((unsigned)sbuf.st_size < tparams->header_size) { + } else if (sbuf.st_size < (off_t)tparams->header_size) { fprintf (stderr, - "%s: Bad size: \"%s\" is not valid image: size %ld < %u\n", + "%s: Bad size: \"%s\" is not valid image: size %llu < %u\n", params.cmdname, params.imagefile, - sbuf.st_size, tparams->header_size); + (unsigned long long) sbuf.st_size, + tparams->header_size); exit (EXIT_FAILURE); } else { size = sbuf.st_size; |