Changeset 4637c72 in mainline

Timestamp:

2010-08-03T10:57:19Z (14 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

482dde7, db4ec8d

Parents:

dba4a23

Message:

Instead of checking whether the first sector in a cluster is not beyond the end
of the device, check the last sector in a cluster for the same.

File:

• uspace/srv/fs/fat/fat_fat.c (modified) (1 diff)

uspace/srv/fs/fat/fat_fat.c

@@ -424 +424 @@
                 for (c = 0; c < BPS(bs) / sizeof(fat_cluster_t); c++, cl++) {
                         /*
-                         * Check if the cluster is physically there. This check
-                         * becomes necessary when the file system is created
-                         * with fewer total sectors than how many is inferred
-                         * from the size of the file allocation table.
+                         * Check if the entire cluster is physically there.
+                         * This check becomes necessary when the file system is
+                         * created with fewer total sectors than how many is
+                         * inferred from the size of the file allocation table
+                         * or when the last cluster ends beyond the end of the
+                         * device.
                          */
-                        if ((cl >= 2) &&
-                            ((cl - 2) * SPC(bs) + SSA(bs) >= TS(bs))) {
+                        if ((cl >= FAT_CLST_FIRST) &&
+                            CLBN2PBN(bs, cl, SPC(bs) - 1) >= TS(bs)) {
                                 rc = block_put(blk);
                                 if (rc != EOK)