Hi Stefan,
Hi Detlev,
On Monday 28 March 2011 17:55:03 Detlev Zundel wrote:
The function find_sector() does not take into account if the flash bank has changed since the last call. This could lead to illegal accesses inside and beyond the flash_info_t info strcture. For example if the current flash bank has less sectors than the last used flash bank.
This patch adds two cheks. One that insures, that the current sector does not exceed the allowed maximum (which is always a good idea). And one that checks if the current access is to the same flash bank as the last access. If not, the search loop will start with sector 0.
Signed-off-by: Martin Krause martin.krause@tqs.de
Can you please comment on Martins fix? Thanks!
I already did and asked for a non line-wrapped patch version:
http://lists.denx.de/pipermail/u-boot/2011-March/088950.html
Still no answer though. Martin, how is your schedule here? Will you find the time to send an updated patch shortly?
Well actually I followed up to a patch version that Martin sent out pretty much immediately and which I can apply without problems. Can you please try again if the patch works for you also?
Thanks! Detlev