Re: [U-Boot] [PATCH] env: fix potential stack overflow in environment functions

Dear Rob Herring,

In message 1363987581-28050-1-git-send-email-robherring2@gmail.com you wrote:

Most of the various environment functions create CONFIG_ENV_SIZE buffers on the stack. At least on ARM and PPC which have 4KB stacks, this can overflow the stack if we have large environment sizes. So move all the buffers off the stack to static buffers.

Could you please explain what exactly you mean with this "have 4KB stacks" statement?

Also, why do you think there is more space for data then for stack?

Has this patch been tested on any actual hardware?

I would expect problems, as this code is running before relocation, i. e. when the data segment is not writable?

Best regards,

Wolfgang Denk