diff options
author | Paulo Alcantara <pcacjr@zytor.com> | 2012-07-14 20:18:12 -0300 |
---|---|---|
committer | Paulo Alcantara <pcacjr@zytor.com> | 2012-07-21 01:21:46 -0300 |
commit | 4f9c17d3be09514be90f50a3b705fef92774acd4 (patch) | |
tree | 2b8dbd5b329830f30b74bb2b7dfe691a9df19953 | |
parent | ca104cd54f6c6546cc3e542d98dbfa6d0be0f675 (diff) | |
download | syslinux-4f9c17d3be09514be90f50a3b705fef92774acd4.tar.gz syslinux-4f9c17d3be09514be90f50a3b705fef92774acd4.tar.xz syslinux-4f9c17d3be09514be90f50a3b705fef92774acd4.zip |
xfs: Remove unnecessary check in xfs_iget() function
xfs_iget() function is _never_ called with parent inodes with mode set
to DT_REG (regular file) and this check was used for other puporses while
doing some test cases.
Signed-off-by: Paulo Alcantara <pcacjr@zytor.com>
-rw-r--r-- | core/fs/xfs/xfs.c | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/core/fs/xfs/xfs.c b/core/fs/xfs/xfs.c index 064a6fb0..85414ae6 100644 --- a/core/fs/xfs/xfs.c +++ b/core/fs/xfs/xfs.c @@ -357,17 +357,13 @@ static struct inode *xfs_iget(const char *dname, struct inode *parent) core = xfs_get_ino_core(fs, parent->ino); } - if (parent->mode == DT_DIR) { /* Is this inode a directory ? */ - xfs_debug("Parent inode is a directory"); - - /* TODO: Handle both shortform directories and directory blocks */ - if (core->di_format == XFS_DINODE_FMT_LOCAL) { - inode = xfs_fmt_local_find_entry(dname, parent, core); - } else { - xfs_debug("format %hhu", core->di_format); - xfs_debug("TODO: format \"local\" is the only supported ATM"); - goto out; - } + /* TODO: Handle both shortform directories and directory blocks */ + if (core->di_format == XFS_DINODE_FMT_LOCAL) { + inode = xfs_fmt_local_find_entry(dname, parent, core); + } else { + xfs_debug("format %hhu", core->di_format); + xfs_debug("TODO: format \"local\" is the only supported ATM"); + goto out; } xfs_ino_core_free(inode, core); |