Hi, On Mon, Jul 02 2012, Alan Cox wrote: > From: Alan Cox <alan@xxxxxxxxxxxxxxx> > > The effect of the existing code is that we continue blindly when we should > warn about an invalid allocation unit > > Reported-by: dcb314@xxxxxxxxxxx > Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=44061 > Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx> > --- > > drivers/mmc/core/sd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c > index c272c686..c52f12b 100644 > --- a/drivers/mmc/core/sd.c > +++ b/drivers/mmc/core/sd.c > @@ -244,7 +244,7 @@ static int mmc_read_ssr(struct mmc_card *card) > * bitfield positions accordingly. > */ > au = UNSTUFF_BITS(ssr, 428 - 384, 4); > - if (au > 0 || au <= 9) { > + if (au > 0 && au <= 9) { > card->ssr.au = 1 << (au + 4); > es = UNSTUFF_BITS(ssr, 408 - 384, 16); > et = UNSTUFF_BITS(ssr, 402 - 384, 6); Thanks, pushed to mmc-next for 3.6. - Chris. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html