From: Sergii Boryshchenko <sergii.boryshchenko@xxxxxxxxxxxxxxx> The `ufs_nfs_get_inode` function contains a check to validate the inode number (`ino`) against the valid range of inode numbers. However, this check is redundant because the same validation is already performed in the `ufs_iget` function, which is called immediately afterward. By removing this redundant check, we simplify the code and avoid unnecessary double-checking of the inode number, while still ensuring that invalid inode numbers are properly handled by the `ufs_iget` function. This change has no impact on the functionality since `ufs_iget` provides the necessary validation for all callers. Signed-off-by: Sergii Boryshchenko <sergii.boryshchenko@xxxxxxxxxxxxxxx> --- fs/ufs/super.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/fs/ufs/super.c b/fs/ufs/super.c index bc625788589c..11e8b869e0ba 100644 --- a/fs/ufs/super.c +++ b/fs/ufs/super.c @@ -101,9 +101,6 @@ static struct inode *ufs_nfs_get_inode(struct super_block *sb, u64 ino, u32 gene struct ufs_sb_private_info *uspi = UFS_SB(sb)->s_uspi; struct inode *inode; - if (ino < UFS_ROOTINO || ino > (u64)uspi->s_ncg * uspi->s_ipg) - return ERR_PTR(-ESTALE); - inode = ufs_iget(sb, ino); if (IS_ERR(inode)) return ERR_CAST(inode); -- 2.25.1
