[U-Boot] [PATCH v2 2/9] make the hwconfig buffer deeper
Wolfgang Denk
wd at denx.de
Fri Jan 7 18:50:55 CET 2011
Dear Kumar Gala,
In message <740DF509-6661-4611-BE1B-BBB629012697 at kernel.crashing.org> you wrote:
>
> >> Making a buffer in a function's stack and make use of it out of the
> >> scope, that's the buggy code I am referring.
> >
> > That is bad indeed, and needs to be fixed.
>
> Where is that happening?
__hwconfig() calls hwconfig_parse() an passes a local stack buffer
(buf) as argument; hwconfig_parse() returns a pointer into this
buffer; then __hwconfig() returns the return value from
hwconfig_parse() - but here the local stack buffer gots out of scope.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Make it right before you make it faster.
More information about the U-Boot
mailing list