Blame SOURCES/0101-fs-ntfs3-Refactor-ntfs_read_mft.patch

Kmods SIG d83023
From 22b05f1ac0332cb47701649206997d8d9a1a7f24 Mon Sep 17 00:00:00 2001
Kmods SIG d83023
From: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Kmods SIG d83023
Date: Tue, 5 Oct 2021 19:33:26 +0300
Kmods SIG d83023
Subject: [Backport 22b05f1ac033] src: Refactor ntfs_read_mft
Kmods SIG d83023
Kmods SIG d83023
Don't save size of attribute reparse point as size of symlink.
Kmods SIG d83023
Kmods SIG d83023
Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Kmods SIG d83023
---
Kmods SIG d83023
 src/inode.c | 22 ++++++++--------------
Kmods SIG d83023
 1 file changed, 8 insertions(+), 14 deletions(-)
Kmods SIG d83023
Kmods SIG d83023
diff --git a/src/inode.c b/src/inode.c
Kmods SIG d83023
index bdebbbd53e766f7435640e55c1551be0fadccd19..859951d785cb2f4380c85b1d799f66f008da7d45 100644
Kmods SIG d83023
--- a/src/inode.c
Kmods SIG d83023
+++ b/src/inode.c
Kmods SIG d83023
@@ -222,9 +222,6 @@ static struct inode *ntfs_read_mft(struct inode *inode,
Kmods SIG d83023
 		if (!attr->non_res) {
Kmods SIG d83023
 			ni->i_valid = inode->i_size = rsize;
Kmods SIG d83023
 			inode_set_bytes(inode, rsize);
Kmods SIG d83023
-			t32 = asize;
Kmods SIG d83023
-		} else {
Kmods SIG d83023
-			t32 = le16_to_cpu(attr->nres.run_off);
Kmods SIG d83023
 		}
Kmods SIG d83023
 
Kmods SIG d83023
 		mode = S_IFREG | (0777 & sbi->options->fs_fmask_inv);
Kmods SIG d83023
@@ -313,17 +310,14 @@ static struct inode *ntfs_read_mft(struct inode *inode,
Kmods SIG d83023
 		rp_fa = ni_parse_reparse(ni, attr, &rp);
Kmods SIG d83023
 		switch (rp_fa) {
Kmods SIG d83023
 		case REPARSE_LINK:
Kmods SIG d83023
-			if (!attr->non_res) {
Kmods SIG d83023
-				inode->i_size = rsize;
Kmods SIG d83023
-				inode_set_bytes(inode, rsize);
Kmods SIG d83023
-				t32 = asize;
Kmods SIG d83023
-			} else {
Kmods SIG d83023
-				inode->i_size =
Kmods SIG d83023
-					le64_to_cpu(attr->nres.data_size);
Kmods SIG d83023
-				t32 = le16_to_cpu(attr->nres.run_off);
Kmods SIG d83023
-			}
Kmods SIG d83023
+			/*
Kmods SIG d83023
+			 * Normal symlink.
Kmods SIG d83023
+			 * Assume one unicode symbol == one utf8.
Kmods SIG d83023
+			 */
Kmods SIG d83023
+			inode->i_size = le16_to_cpu(rp.SymbolicLinkReparseBuffer
Kmods SIG d83023
+							    .PrintNameLength) /
Kmods SIG d83023
+					sizeof(u16);
Kmods SIG d83023
 
Kmods SIG d83023
-			/* Looks like normal symlink. */
Kmods SIG d83023
 			ni->i_valid = inode->i_size;
Kmods SIG d83023
 
Kmods SIG d83023
 			/* Clear directory bit. */
Kmods SIG d83023
@@ -420,7 +414,7 @@ static struct inode *ntfs_read_mft(struct inode *inode,
Kmods SIG d83023
 		ni->std_fa &= ~FILE_ATTRIBUTE_DIRECTORY;
Kmods SIG d83023
 		inode->i_op = &ntfs_link_inode_operations;
Kmods SIG d83023
 		inode->i_fop = NULL;
Kmods SIG d83023
-		inode_nohighmem(inode); // ??
Kmods SIG d83023
+		inode_nohighmem(inode);
Kmods SIG d83023
 	} else if (S_ISREG(mode)) {
Kmods SIG d83023
 		ni->std_fa &= ~FILE_ATTRIBUTE_DIRECTORY;
Kmods SIG d83023
 		inode->i_op = &ntfs_file_inode_operations;
Kmods SIG d83023
-- 
Kmods SIG d83023
2.31.1
Kmods SIG d83023