On Thu, Aug 15, 2019 at 03:55:26PM -0500, Andreas Dannenberg wrote: > During FIT reading through ymodem_read_fit() the function > xyzModem_stream_read() is being used which returns zero once the end > of a stream has been reached. This could lead to an premature exit from > ymodem_read_fit() with certain-sized FIT images reporting that zero > bytes overall were read. Such a premature exit would then result in an > -EIO failure being triggered within the spl_load_simple_fit() caller > function and ultimately lead to a boot failure. > > Fix this logic by simply aborting the stream read loops and continuing > with the regular code flow which ultimately would lead to returning > the number of bytes to be read ('size') as expected by the callers of > ymodem_read_fit(). > > Fixes: fa715193c083 ("spl: Add an option to load a FIT containing U-Boot from > UART") > Signed-off-by: Andreas Dannenberg <dannenb...@ti.com>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot