[U-Boot] [PATCH] tools: default image: use ih_size for checking data size

Wolfgang Denk wd at denx.de
Fri May 3 17:04:44 CEST 2013


Dear Jonas Gorski,

In message <1367581025-26740-1-git-send-email-jogo at openwrt.org> you wrote:
> Common image usage is uImage + appended rootfs, so the the uImage data

No, this is not at all "common usage".  Actually this something you
should never do.

> is only part of the total image. So read out and use the header's
> ih_size field instead of the total file size.
> 
> To prevent reading over the end of the buffer, check that the image file
> is big enough to contain the data before calculating its checksum.
> 
> Before:
> ~# mkimage -l dir665_fw_100NA.bin
> mkimage: ERROR: "dir665_fw_100NA/dir665_fw_100NA.bin" has corrupted data!

Sorry, I don't know how you create your image files, but you must be
doing something fundamentally wrong.  If mkimage reports a bug here,
it is probably right.  If the actual payload size is different from
the content of the  ih_size  field, then your image _is_ broken.

NAK.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The aim of science is not to open the door to everlasting wisdom  but
to set a limit on everlasting error.                 - Bertolt Brecht


More information about the U-Boot mailing list