[U-Boot] [PATCH 1/2] mtdparts: fix write through NULL pointer

Wolfgang Denk wd at denx.de
Sat May 1 00:39:49 CEST 2010


Dear Stefano Babic,

In message <4BD81816.2050601 at denx.de> you wrote:
>
> I have already sent a patch fixing this issue,

I've seen it (unfortunately only after running into this myself).

> http://lists.denx.de/pipermail/u-boot/2010-April/070347.html
> 
> However, I preferred (easier way) to fix the calling function as the
> device_parse(). I do not know if Stefan has already merged it.

I tend to prefer my version, as it seems to be more defensive, and it
also adds an assert() for the 3rd parameter.

If you agree, I'll apply my version?

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
Don't you know anything? I should have thought anyone knows that  who
knows anything about anything...      - Terry Pratchett, _Soul Music_


More information about the U-Boot mailing list