Blame SOURCES/0087-fs-ntfs3-Reject-mount-if-boot-s-cluster-size-media-s.patch

Kmods SIG d83023
From 09f7c338da7818fd33af401d855b895550e7c170 Mon Sep 17 00:00:00 2001
Kmods SIG d83023
From: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Kmods SIG d83023
Date: Tue, 28 Sep 2021 19:19:17 +0300
Kmods SIG d83023
Subject: [Backport 09f7c338da78] src: Reject mount if boot's cluster size
Kmods SIG d83023
 < media sector size
Kmods SIG d83023
Kmods SIG d83023
If we continue to work in this case, then we can corrupt fs.
Kmods SIG d83023
Fixes: 82cae269cfa9 ("src: Add initialization of super block").
Kmods SIG d83023
Kmods SIG d83023
Reviewed-by: Kari Argillander <kari.argillander@gmail.com>
Kmods SIG d83023
Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Kmods SIG d83023
---
Kmods SIG d83023
 src/super.c | 13 ++++++++++++-
Kmods SIG d83023
 1 file changed, 12 insertions(+), 1 deletion(-)
Kmods SIG d83023
Kmods SIG d83023
diff --git a/src/super.c b/src/super.c
Kmods SIG d83023
index 800897777eb03855f702e8acf15b39d02c1c2fbf..e9b6a37345769700f682809832d84d8dfc7b9414 100644
Kmods SIG d83023
--- a/src/super.c
Kmods SIG d83023
+++ b/src/super.c
Kmods SIG d83023
@@ -763,9 +763,20 @@ static int ntfs_init_from_boot(struct super_block *sb, u32 sector_size,
Kmods SIG d83023
 	sbi->mft.lbo = mlcn << sbi->cluster_bits;
Kmods SIG d83023
 	sbi->mft.lbo2 = mlcn2 << sbi->cluster_bits;
Kmods SIG d83023
 
Kmods SIG d83023
-	if (sbi->cluster_size < sbi->sector_size)
Kmods SIG d83023
+	/* Compare boot's cluster and sector. */
Kmods SIG d83023
+	if (sbi->cluster_size < boot_sector_size)
Kmods SIG d83023
 		goto out;
Kmods SIG d83023
 
Kmods SIG d83023
+	/* Compare boot's cluster and media sector. */
Kmods SIG d83023
+	if (sbi->cluster_size < sector_size) {
Kmods SIG d83023
+		/* No way to use ntfs_get_block in this case. */
Kmods SIG d83023
+		ntfs_err(
Kmods SIG d83023
+			sb,
Kmods SIG d83023
+			"Failed to mount 'cause NTFS's cluster size (%u) is less than media sector size (%u)",
Kmods SIG d83023
+			sbi->cluster_size, sector_size);
Kmods SIG d83023
+		goto out;
Kmods SIG d83023
+	}
Kmods SIG d83023
+
Kmods SIG d83023
 	sbi->cluster_mask = sbi->cluster_size - 1;
Kmods SIG d83023
 	sbi->cluster_mask_inv = ~(u64)sbi->cluster_mask;
Kmods SIG d83023
 	sbi->record_size = record_size = boot->record_size < 0
Kmods SIG d83023
-- 
Kmods SIG d83023
2.31.1
Kmods SIG d83023