Re: [U-Boot] [PATCH] Make sure that argv[] argument pointers are not modified.

2010-07-04 Thread Wolfgang Denk
In message <1277804965-1086-1-git-send-email...@denx.de> you wrote: > The hush shell dynamically allocates (and re-allocates) memory for the > argument strings in the "char *argv[]" argument vector passed to > commands. Any code that modifies these pointers will cause serious > corruption of the m

Re: [U-Boot] [PATCH] Make sure that argv[] argument pointers are not modified.

2010-06-29 Thread Mike Frysinger
On Tuesday, June 29, 2010 17:08:51 Wolfgang Denk wrote: > Mike Frysinger wrote: > > > arch/blackfin/cpu/bootrom-asm-offsets.c.in|2 +- > > > > this file isnt linked into u-boot, so i dont think it needs to be > > changed. it is actually compiled, just preprocessed and assembly > > output.

Re: [U-Boot] [PATCH] Make sure that argv[] argument pointers are not modified.

2010-06-29 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201006291559.55916.vap...@gentoo.org> you wrote: > > > arch/blackfin/cpu/bootrom-asm-offsets.c.in|2 +- > > this file isnt linked into u-boot, so i dont think it needs to be changed. > it > is actually compiled, just preprocessed and assembly output. I

Re: [U-Boot] [PATCH] Make sure that argv[] argument pointers are not modified.

2010-06-29 Thread Mike Frysinger
On Tuesday, June 29, 2010 05:49:25 Wolfgang Denk wrote: > arch/blackfin/cpu/bootrom-asm-offsets.c.in|2 +- this file isnt linked into u-boot, so i dont think it needs to be changed. it is actually compiled, just preprocessed and assembly output. > arch/blackfin/cpu/reset.c

[U-Boot] [PATCH] Make sure that argv[] argument pointers are not modified - Part 2

2010-06-29 Thread Wolfgang Denk
Fix compile problems cause by this commit. Signed-off-by: Wolfgang Denk --- Sorry, by accident these changes were not checked in yet when I submitted the patch. Given the size of the patch and the local impact of this fix I post this as an incremental patch. If nobody complains I will squash th