]> git.ipfire.org Git - thirdparty/util-linux.git/commitdiff
libblkid: exfat: rename superblocks fields to match specification
authorThomas Weißschuh <thomas@t-8ch.de>
Sat, 19 Nov 2022 19:32:30 +0000 (20:32 +0100)
committerThomas Weißschuh <thomas@t-8ch.de>
Sat, 19 Nov 2022 19:46:42 +0000 (20:46 +0100)
Now that we have access to the official exFAT specification [0], let's
use its nomenclature.

[0] https://learn.microsoft.com/en-us/windows/win32/fileio/exfat-specification

libblkid/src/superblocks/exfat.c

index 3986194774c229f9da3db5d98ec7d28d395dda7d..003c08f6f4f147597002f129365707af41aeddd6 100644 (file)
@@ -7,27 +7,30 @@
 #include "superblocks.h"
 
 struct exfat_super_block {
-       uint8_t jump[3];
-       uint8_t oem_name[8];
-       uint8_t __unused1[53];
-       uint64_t block_start;
-       uint64_t block_count;
-       uint32_t fat_block_start;
-       uint32_t fat_block_count;
-       uint32_t cluster_block_start;
-       uint32_t cluster_count;
-       uint32_t rootdir_cluster;
-       uint8_t volume_serial[4];
+       uint8_t JumpBoot[3];
+       uint8_t FileSystemName[8];
+       uint8_t MustBeZero[53];
+       uint64_t PartitionOffset;
+       uint64_t VolumeLength;
+       uint32_t FatOffset;
+       uint32_t FatLength;
+       uint32_t ClusterHeapOffset;
+       uint32_t ClusterCount;
+       uint32_t FirstClusterOfRootDirectory;
+       uint8_t VolumeSerialNumber[4];
        struct {
                uint8_t vermin;
                uint8_t vermaj;
-       } version;
-       uint16_t volume_state;
-       uint8_t block_bits;
-       uint8_t bpc_bits;
-       uint8_t fat_count;
-       uint8_t drive_no;
-       uint8_t allocated_percent;
+       } FileSystemRevision;
+       uint16_t VolumeFlags;
+       uint8_t BytesPerSectorShift;
+       uint8_t SectorsPerClusterShift;
+       uint8_t NumberOfFats;
+       uint8_t DriveSelect;
+       uint8_t PercentInUse;
+       uint8_t Reserved[7];
+       uint8_t BootCode[390];
+       uint16_t BootSignature;
 } __attribute__((__packed__));
 
 struct exfat_entry_label {
@@ -37,8 +40,8 @@ struct exfat_entry_label {
        uint8_t reserved[8];
 } __attribute__((__packed__));
 
-#define BLOCK_SIZE(sb) ((sb)->block_bits < 32 ? (1u << (sb)->block_bits) : 0)
-#define CLUSTER_SIZE(sb) ((sb)->bpc_bits < 32 ? (BLOCK_SIZE(sb) << (sb)->bpc_bits) : 0)
+#define BLOCK_SIZE(sb) ((sb)->BytesPerSectorShift < 32 ? (1u << (sb)->BytesPerSectorShift) : 0)
+#define CLUSTER_SIZE(sb) ((sb)->SectorsPerClusterShift < 32 ? (BLOCK_SIZE(sb) << (sb)->SectorsPerClusterShift) : 0)
 #define EXFAT_FIRST_DATA_CLUSTER 2
 #define EXFAT_LAST_DATA_CLUSTER 0xffffff6
 #define EXFAT_ENTRY_SIZE 32
@@ -49,15 +52,15 @@ struct exfat_entry_label {
 static uint64_t block_to_offset(const struct exfat_super_block *sb,
                uint64_t block)
 {
-       return block << sb->block_bits;
+       return block << sb->BytesPerSectorShift;
 }
 
 static uint64_t cluster_to_block(const struct exfat_super_block *sb,
                uint32_t cluster)
 {
-       return le32_to_cpu(sb->cluster_block_start) +
+       return le32_to_cpu(sb->ClusterHeapOffset) +
                        ((uint64_t) (cluster - EXFAT_FIRST_DATA_CLUSTER)
-                                       << sb->bpc_bits);
+                                       << sb->SectorsPerClusterShift);
 }
 
 static uint64_t cluster_to_offset(const struct exfat_super_block *sb,
@@ -72,7 +75,7 @@ static uint32_t next_cluster(blkid_probe pr,
        uint32_t *next;
        uint64_t fat_offset;
 
-       fat_offset = block_to_offset(sb, le32_to_cpu(sb->fat_block_start))
+       fat_offset = block_to_offset(sb, le32_to_cpu(sb->FatOffset))
                + (uint64_t) cluster * sizeof(cluster);
        next = (uint32_t *) blkid_probe_get_buffer(pr, fat_offset,
                        sizeof(uint32_t));
@@ -84,7 +87,7 @@ static uint32_t next_cluster(blkid_probe pr,
 static struct exfat_entry_label *find_label(blkid_probe pr,
                const struct exfat_super_block *sb)
 {
-       uint32_t cluster = le32_to_cpu(sb->rootdir_cluster);
+       uint32_t cluster = le32_to_cpu(sb->FirstClusterOfRootDirectory);
        uint64_t offset = cluster_to_offset(sb, cluster);
        uint8_t *entry;
        const size_t max_iter = 10000;
@@ -175,13 +178,13 @@ static int probe_exfat(blkid_probe pr, const struct blkid_idmag *mag)
        else if (errno)
                return -errno;
 
-       blkid_probe_sprintf_uuid(pr, sb->volume_serial, 4,
+       blkid_probe_sprintf_uuid(pr, sb->VolumeSerialNumber, 4,
                        "%02hhX%02hhX-%02hhX%02hhX",
-                       sb->volume_serial[3], sb->volume_serial[2],
-                       sb->volume_serial[1], sb->volume_serial[0]);
+                       sb->VolumeSerialNumber[3], sb->VolumeSerialNumber[2],
+                       sb->VolumeSerialNumber[1], sb->VolumeSerialNumber[0]);
 
        blkid_probe_sprintf_version(pr, "%u.%u",
-                       sb->version.vermaj, sb->version.vermin);
+                       sb->FileSystemRevision.vermaj, sb->FileSystemRevision.vermin);
 
        blkid_probe_set_fsblocksize(pr, BLOCK_SIZE(sb));
        blkid_probe_set_block_size(pr, BLOCK_SIZE(sb));