On 16 July 2015 at 10:34, Simon Glass <[email protected]> wrote:
> On 16 July 2015 at 07:26, Linus Walleij <[email protected]> wrote:
>> On Fri, Jul 10, 2015 at 6:18 PM, Sudeep Holla <[email protected]> wrote:
>>
>>> Commit 90fbee3e4051 ("cmd_fdt: Actually fix fdt command in sandbox")
>>> changed the format(from hex address to unsigned long) in which "fdtaddr"
>>> is saved . However do_fdt continues reads the "fdtaddr" assuming it to
>>> be in hex format. This may lead to fdt being either loaded or attempted
>>> to load at erroneous address generating fault if the address is out of
>>> memory.
>>>
>>> This patch changes back the format to hex while saving the "fdtaddr"
>>> as it was done before.
>>>
>>> Signed-off-by: Sudeep Holla <[email protected]>
>>> Cc: Linus Walleij <[email protected]>
>>> Cc: Tom Rini <[email protected]>
>>> Cc: Simon Glass <[email protected]>
>>> Cc: Hua Yanghao <[email protected]>
>>> Cc: Heiko Schocher <[email protected]>
>>
>> Looks like the right fix:
>> Acked-by: Linus Walleij <[email protected]>
>>
>> Yours,
>> Linus Walleij
>
> Acked-by: Simon Glass <[email protected]>

Applied to u-boot-fdt, thanks!
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to