[U-Boot] [PATCH 2/3] mmc: add boundary check for mmc operation
Wolfgang Denk
wd at denx.de
Thu Sep 9 16:51:10 CEST 2010
Dear Lei Wen,
In message <AANLkTi=m3P2w33-a7-ChpVPwU=XA0WTTL2gHV1dTJOWY at mail.gmail.com> you wrote:
> >>
> >> + if (start > mmc->block_dev.lba || (start + blkcnt) mmc->block_dev.lba) {
> >> + printf("\noperation excceed mmc boudary..\n");
> >
> > No initial newline, please.
>
> You mean add additional line or just reduce a line here?...
I mean change
printf("\noperation ...");
into
printf("operation ...");
> > Hm... Maybe you could change the message so it is more informatiove,
> > i. e. tell the user which boundary was exceeded, and what caused this
> > condition - is this a user error or a problem in the code?
>
> Give error message for each judgment of the two?
Not really needed, but you might print the legal range and the current
value in the error message.
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
Time is a drug. Too much of it kills you.
- Terry Pratchett, _Small Gods_
More information about the U-Boot
mailing list