Hi Simon,
On 04/07/15 00:06, Simon Glass wrote:
Hi Sudeep,
On 3 July 2015 at 11:29, Sudeep Holla sudeep.holla@arm.com 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.
Cc: Linus Walleij linus.walleij@linaro.org Cc: Tom Rini trini@konsulko.com Cc: Simon Glass sjg@chromium.org Cc: Hua Yanghao huayanghao@gmail.com Cc: Heiko Schocher hs@denx.de
common/cmd_fdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Hi,
You will see my signoff missing and it's intentional as I officially can't contribute to Uboot. I found this issue and it's been there for a while now, so thought of atleast reporting it(with a patch). So if anyone can review and submit this or any appropriate fix that would be great. I see this issue on vexpress TC2.
We can't take this without a sign-off - what stops you from adding it exactly?
Unfortunately it's not in the list of legally approved projects I can contribute. So I am bound not to do so legally.
Hi Linus,
Since you did a lot work on vexpress/Juno quite recently on ARM's request, can you cook up a similar patch to fix the above mentioned issue if that's OK for you ?
Regards, Sudeep