summaryrefslogtreecommitdiffstats
path: root/sbin/fsck_msdosfs/main.c
diff options
context:
space:
mode:
authorbde <bde@FreeBSD.org>2004-02-05 15:18:18 +0000
committerbde <bde@FreeBSD.org>2004-02-05 15:18:18 +0000
commit9cf605d618e7c575664f866650356b95ee967870 (patch)
tree2d278a7ffa616df58d57826bb346ef50deb60233 /sbin/fsck_msdosfs/main.c
parentcbc1462d2660010af7ba3bd17f5bdeec93126fee (diff)
downloadFreeBSD-src-9cf605d618e7c575664f866650356b95ee967870.zip
FreeBSD-src-9cf605d618e7c575664f866650356b95ee967870.tar.gz
Fixed some bugs in checkdirty(). The check for the clean bit was
combined with the the signature check in a wrong way (basically (dirty:= signature_recognised() && !clean) instead of (mightbedirty:= !signature_recognized || !clean), so file systems with unrecognized signatures were considered clean. Many of the don't-care and reserved bits were not ignored, so some file systems with valid signatures were unrecognized. One of my FAT32 file systems has a signature of f8,ff,ff,ff,ff,ff,ff,f7 when dirty, but only f8,ff,ff,0f,ff,ff,ff,07 was recognised as dirty for FAT32, so the fail-unsafeness made my file system always considered clean. Check the i/o non-error bit in checkdirty(). Its absence would give an unrecognized signature in code that is unaware of it, but we now mask it out of the signature so we have to check it explicitly. This combines naturally with the check of the clean bit. Reviewed by: rnordier (except for final details)
Diffstat (limited to 'sbin/fsck_msdosfs/main.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud