commit 575e21e8cdeeba679463602b6d11a7a282931baa
author J. Vijayanand <vijayanand.jayaraman@in.bosch.com> Wed Oct 19 07:43:08 2011 +0000
committer Wolfgang Denk <wd@denx.de> Thu Oct 27 23:53:57 2011 +0200
tree c1d8cabd43e514b1ca67a9febb37261ab97e2c71
parent 27e5620542b6b827ccbd55e04bd38e4da3ff254b

fat: correct ATTR_VFAT check

ATTR_VFAT condition requires multiple bits to be set but the present
condition checking in do_fat_read() & get_dentfromdir() ends up
passing on even a single bit being set.

Signed-off-by: J. Vijayanand <vijayanand.jayaraman@in.bosch.com>
Signed-off-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>

fs/fat/fat.c

diff --git a/fs/fat/fat.c b/fs/fat/fat.c
index 756ac64..28baa54 100644
--- a/fs/fat/fat.c
+++ b/fs/fat/fat.c
@@ -572,8 +572,8 @@
 			}
 			if ((dentptr->attr & ATTR_VOLUME)) {
 #ifdef CONFIG_SUPPORT_VFAT
-				if ((dentptr->attr & ATTR_VFAT) &&
-				    (dentptr-> name[0] & LAST_LONG_ENTRY_MASK)) {
+				if ((dentptr->attr & ATTR_VFAT) == ATTR_VFAT &&
+				    (dentptr->name[0] & LAST_LONG_ENTRY_MASK)) {
 					prevcksum = ((dir_slot *)dentptr)->alias_checksum;
 					get_vfatname(mydata, curclust,
 						     get_dentfromdir_block,
@@ -897,7 +897,7 @@
 			}
 			if ((dentptr->attr & ATTR_VOLUME)) {
 #ifdef CONFIG_SUPPORT_VFAT
-				if ((dentptr->attr & ATTR_VFAT) &&
+				if ((dentptr->attr & ATTR_VFAT) == ATTR_VFAT &&
 				    (dentptr->name[0] & LAST_LONG_ENTRY_MASK)) {
 					prevcksum =
 						((dir_slot *)dentptr)->alias_checksum;