On Sun, Aug 13, 2017 at 03:16:17PM -0400, Reno Farnesi wrote: > The overflow calculation was incorrect. Adding the start block of the > partition is not needed because the sectors are already relative to the > beginning of the partition. If you attempted to write a file smaller > than cur_part_info.start blocks on a full partition the old calculation > fails to catch the overflow. This would cause an infinite loop in the > determine_fatent function. > > Old, incorrect calculation: > > ending sector of new file = start sector + file size (in sectors) > last sector = partition start + total sectors on the partition > > Adding the partition start block number is not needed because sectors > are already relative to the start of the partition. > > New calculation: > > ending sector of new file = start sector + file size (in sectors) > last sector = total sectors on the partition > > Signed-off-by: Reno Farnesi <nfarne...@gmail.com>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
