Dear Wolfgang Denk,
On Tue, Jan 10, 2012 at 2:48 PM, Wolfgang Denk wd@denx.de wrote:
You should do so in the commit mesage (but judging from my current state of mind you can save the efforts for a v4).
Got it. If we end up with a v4, I'll add it.
I just replied to one of your earlier patches that did not fix the overflow, but just shifted it to happen later, in Linux.
I just responded to that one. ...but to reiterate here: I'm happy to add a CONFIG option to u-boot to allow setting the command line length (so I can use the 1024 bytes I have available to me in ARM) and also to avoid the buffer overrun.
If this is the approach you'd like me to go, I'm happy to do it.
But this does the very same, just in a different way (requiring more code). And it does not prevent the overflow at all.
Sorry, I must have misunderstood your earlier suggestions back in October. My apologies there. :(