aboutsummaryrefslogtreecommitdiffstats
path: root/fs/jfs
diff options
context:
space:
mode:
authorLi Zefan <lizf@cn.fujitsu.com>2008-05-28 08:58:56 -0500
committerDave Kleikamp <shaggy@linux.vnet.ibm.com>2008-05-28 08:58:56 -0500
commit3c65e8743bf8b5cf0f90e8d767bf1d8b50c14c76 (patch)
treeb475d031cd1abaf1a2ffef99968c00950ae2c431 /fs/jfs
parentJFS: skip bad iput() call in error path (diff)
downloadlinux-dev-3c65e8743bf8b5cf0f90e8d767bf1d8b50c14c76.tar.xz
linux-dev-3c65e8743bf8b5cf0f90e8d767bf1d8b50c14c76.zip
JFS: diAlloc() should return -EIO rather than EIO
The comment above the function says one of its return value is -EIO, and also the caller of diAlloc() checks for -EIO: struct inode *ialloc(struct inode *parent, umode_t mode) { ... rc = diAlloc(parent, S_ISDIR(mode), inode); if (rc) { jfs_warn("ialloc: diAlloc returned %d!", rc); if (rc == -EIO) make_bad_inode(inode); ... Signed-off-by: Li Zefan <lizf@cn.fujitsu.com> Signed-off-by: Dave Kleikamp <shaggy@linux.vnet.ibm.com>
Diffstat (limited to 'fs/jfs')
-rw-r--r--fs/jfs/jfs_imap.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/jfs/jfs_imap.c b/fs/jfs/jfs_imap.c
index 734ec916beaf..d6363d8309d0 100644
--- a/fs/jfs/jfs_imap.c
+++ b/fs/jfs/jfs_imap.c
@@ -1520,7 +1520,7 @@ int diAlloc(struct inode *pip, bool dir, struct inode *ip)
jfs_error(ip->i_sb,
"diAlloc: can't find free bit "
"in wmap");
- return EIO;
+ return -EIO;
}
/* determine the inode number within the