diff options
author | Alan Cox <alan@linux.intel.com> | 2012-07-02 17:55:13 (GMT) |
---|---|---|
committer | Chris Ball <cjb@laptop.org> | 2012-07-21 04:02:24 (GMT) |
commit | b63b5e819d5b21ae493c17c356018ffa98d3ee1c (patch) | |
tree | 79565f8d73049e1d66812cf308cd14671c528860 /drivers/mmc/core | |
parent | 68737043bb155b42d54943d25153b901fd16e535 (diff) | |
download | linux-b63b5e819d5b21ae493c17c356018ffa98d3ee1c.tar.xz |
mmc: core: correct invalid error checking
The effect of the existing code is that we continue blindly when we
should warn about an invalid allocation unit.
Reported-by: dcb314@hotmail.com
Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=44061
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Chris Ball <cjb@laptop.org>
Diffstat (limited to 'drivers/mmc/core')
-rw-r--r-- | drivers/mmc/core/sd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c index b0b9e37..33c6f26 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); |