Skip to content

Commit

Permalink
fs/ntfs3: Improve ntfs_dir_count
Browse files Browse the repository at this point in the history
[ Upstream commit 6a799c9 ]

Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
aalexandrovich authored and gregkh committed Mar 1, 2024
1 parent 06144cd commit d316783
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions fs/ntfs3/dir.c
Original file line number Diff line number Diff line change
Expand Up @@ -515,11 +515,9 @@ static int ntfs_dir_count(struct inode *dir, bool *is_empty, size_t *dirs,
struct INDEX_HDR *hdr;
const struct ATTR_FILE_NAME *fname;
u32 e_size, off, end;
u64 vbo = 0;
size_t drs = 0, fles = 0, bit = 0;
loff_t i_size = ni->vfs_inode.i_size;
struct indx_node *node = NULL;
u8 index_bits = ni->dir.index_bits;
size_t max_indx = ni->vfs_inode.i_size >> ni->dir.index_bits;

if (is_empty)
*is_empty = true;
Expand Down Expand Up @@ -563,7 +561,7 @@ static int ntfs_dir_count(struct inode *dir, bool *is_empty, size_t *dirs,
fles += 1;
}

if (vbo >= i_size)
if (bit >= max_indx)
goto out;

err = indx_used_bit(&ni->dir, ni, &bit);
Expand All @@ -573,8 +571,7 @@ static int ntfs_dir_count(struct inode *dir, bool *is_empty, size_t *dirs,
if (bit == MINUS_ONE_T)
goto out;

vbo = (u64)bit << index_bits;
if (vbo >= i_size)
if (bit >= max_indx)
goto out;

err = indx_read(&ni->dir, ni, bit << ni->dir.idx2vbn_bits,
Expand All @@ -584,7 +581,6 @@ static int ntfs_dir_count(struct inode *dir, bool *is_empty, size_t *dirs,

hdr = &node->index->ihdr;
bit += 1;
vbo = (u64)bit << ni->dir.idx2vbn_bits;
}

out:
Expand Down

0 comments on commit d316783

Please sign in to comment.