iget: stop AFFS from using iget() and read_inode()
Stop the AFFS filesystem from using iget() and read_inode(). Replace affs_read_inode() with affs_iget(), and call that instead of iget(). affs_iget() then uses iget_locked() directly and returns a proper error code instead of an inode in the event of an error. affs_fill_super() returns any error incurred when getting the root inode instead of EINVAL. [akpm@linux-foundation.org: coding-style fixes] Signed-off-by: David Howells <dhowells@redhat.com> Cc: Roman Zippel <zippel@linux-m68k.org> Acked-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
69840b0d06
commit
210f855963
|
@ -174,7 +174,8 @@ extern void affs_put_inode(struct inode *inode);
|
||||||
extern void affs_drop_inode(struct inode *inode);
|
extern void affs_drop_inode(struct inode *inode);
|
||||||
extern void affs_delete_inode(struct inode *inode);
|
extern void affs_delete_inode(struct inode *inode);
|
||||||
extern void affs_clear_inode(struct inode *inode);
|
extern void affs_clear_inode(struct inode *inode);
|
||||||
extern void affs_read_inode(struct inode *inode);
|
extern struct inode *affs_iget(struct super_block *sb,
|
||||||
|
unsigned long ino);
|
||||||
extern int affs_write_inode(struct inode *inode, int);
|
extern int affs_write_inode(struct inode *inode, int);
|
||||||
extern int affs_add_entry(struct inode *dir, struct inode *inode, struct dentry *dentry, s32 type);
|
extern int affs_add_entry(struct inode *dir, struct inode *inode, struct dentry *dentry, s32 type);
|
||||||
|
|
||||||
|
|
|
@ -170,9 +170,11 @@ affs_remove_link(struct dentry *dentry)
|
||||||
if (!link_bh)
|
if (!link_bh)
|
||||||
goto done;
|
goto done;
|
||||||
|
|
||||||
dir = iget(sb, be32_to_cpu(AFFS_TAIL(sb, link_bh)->parent));
|
dir = affs_iget(sb, be32_to_cpu(AFFS_TAIL(sb, link_bh)->parent));
|
||||||
if (!dir)
|
if (IS_ERR(dir)) {
|
||||||
|
retval = PTR_ERR(dir);
|
||||||
goto done;
|
goto done;
|
||||||
|
}
|
||||||
|
|
||||||
affs_lock_dir(dir);
|
affs_lock_dir(dir);
|
||||||
affs_fix_dcache(dentry, link_ino);
|
affs_fix_dcache(dentry, link_ino);
|
||||||
|
|
|
@ -15,20 +15,25 @@
|
||||||
extern const struct inode_operations affs_symlink_inode_operations;
|
extern const struct inode_operations affs_symlink_inode_operations;
|
||||||
extern struct timezone sys_tz;
|
extern struct timezone sys_tz;
|
||||||
|
|
||||||
void
|
struct inode *affs_iget(struct super_block *sb, unsigned long ino)
|
||||||
affs_read_inode(struct inode *inode)
|
|
||||||
{
|
{
|
||||||
struct super_block *sb = inode->i_sb;
|
|
||||||
struct affs_sb_info *sbi = AFFS_SB(sb);
|
struct affs_sb_info *sbi = AFFS_SB(sb);
|
||||||
struct buffer_head *bh;
|
struct buffer_head *bh;
|
||||||
struct affs_head *head;
|
struct affs_head *head;
|
||||||
struct affs_tail *tail;
|
struct affs_tail *tail;
|
||||||
|
struct inode *inode;
|
||||||
u32 block;
|
u32 block;
|
||||||
u32 size;
|
u32 size;
|
||||||
u32 prot;
|
u32 prot;
|
||||||
u16 id;
|
u16 id;
|
||||||
|
|
||||||
pr_debug("AFFS: read_inode(%lu)\n",inode->i_ino);
|
inode = iget_locked(sb, ino);
|
||||||
|
if (!inode)
|
||||||
|
return ERR_PTR(-ENOMEM);
|
||||||
|
if (!(inode->i_state & I_NEW))
|
||||||
|
return inode;
|
||||||
|
|
||||||
|
pr_debug("AFFS: affs_iget(%lu)\n", inode->i_ino);
|
||||||
|
|
||||||
block = inode->i_ino;
|
block = inode->i_ino;
|
||||||
bh = affs_bread(sb, block);
|
bh = affs_bread(sb, block);
|
||||||
|
@ -154,12 +159,13 @@ affs_read_inode(struct inode *inode)
|
||||||
sys_tz.tz_minuteswest * 60;
|
sys_tz.tz_minuteswest * 60;
|
||||||
inode->i_mtime.tv_nsec = inode->i_ctime.tv_nsec = inode->i_atime.tv_nsec = 0;
|
inode->i_mtime.tv_nsec = inode->i_ctime.tv_nsec = inode->i_atime.tv_nsec = 0;
|
||||||
affs_brelse(bh);
|
affs_brelse(bh);
|
||||||
return;
|
unlock_new_inode(inode);
|
||||||
|
return inode;
|
||||||
|
|
||||||
bad_inode:
|
bad_inode:
|
||||||
make_bad_inode(inode);
|
|
||||||
affs_brelse(bh);
|
affs_brelse(bh);
|
||||||
return;
|
iget_failed(inode);
|
||||||
|
return ERR_PTR(-EIO);
|
||||||
}
|
}
|
||||||
|
|
||||||
int
|
int
|
||||||
|
|
|
@ -208,9 +208,8 @@ affs_lookup(struct inode *dir, struct dentry *dentry, struct nameidata *nd)
|
||||||
affs_lock_dir(dir);
|
affs_lock_dir(dir);
|
||||||
bh = affs_find_entry(dir, dentry);
|
bh = affs_find_entry(dir, dentry);
|
||||||
affs_unlock_dir(dir);
|
affs_unlock_dir(dir);
|
||||||
if (IS_ERR(bh)) {
|
if (IS_ERR(bh))
|
||||||
return ERR_CAST(bh);
|
return ERR_CAST(bh);
|
||||||
}
|
|
||||||
if (bh) {
|
if (bh) {
|
||||||
u32 ino = bh->b_blocknr;
|
u32 ino = bh->b_blocknr;
|
||||||
|
|
||||||
|
@ -223,10 +222,9 @@ affs_lookup(struct inode *dir, struct dentry *dentry, struct nameidata *nd)
|
||||||
ino = be32_to_cpu(AFFS_TAIL(sb, bh)->original);
|
ino = be32_to_cpu(AFFS_TAIL(sb, bh)->original);
|
||||||
}
|
}
|
||||||
affs_brelse(bh);
|
affs_brelse(bh);
|
||||||
inode = iget(sb, ino);
|
inode = affs_iget(sb, ino);
|
||||||
if (!inode) {
|
if (IS_ERR(inode))
|
||||||
return ERR_PTR(-EACCES);
|
return ERR_PTR(PTR_ERR(inode));
|
||||||
}
|
|
||||||
}
|
}
|
||||||
dentry->d_op = AFFS_SB(sb)->s_flags & SF_INTL ? &affs_intl_dentry_operations : &affs_dentry_operations;
|
dentry->d_op = AFFS_SB(sb)->s_flags & SF_INTL ? &affs_intl_dentry_operations : &affs_dentry_operations;
|
||||||
d_add(dentry, inode);
|
d_add(dentry, inode);
|
||||||
|
|
|
@ -113,7 +113,6 @@ static void destroy_inodecache(void)
|
||||||
static const struct super_operations affs_sops = {
|
static const struct super_operations affs_sops = {
|
||||||
.alloc_inode = affs_alloc_inode,
|
.alloc_inode = affs_alloc_inode,
|
||||||
.destroy_inode = affs_destroy_inode,
|
.destroy_inode = affs_destroy_inode,
|
||||||
.read_inode = affs_read_inode,
|
|
||||||
.write_inode = affs_write_inode,
|
.write_inode = affs_write_inode,
|
||||||
.put_inode = affs_put_inode,
|
.put_inode = affs_put_inode,
|
||||||
.drop_inode = affs_drop_inode,
|
.drop_inode = affs_drop_inode,
|
||||||
|
@ -271,6 +270,7 @@ static int affs_fill_super(struct super_block *sb, void *data, int silent)
|
||||||
unsigned long mount_flags;
|
unsigned long mount_flags;
|
||||||
int tmp_flags; /* fix remount prototype... */
|
int tmp_flags; /* fix remount prototype... */
|
||||||
u8 sig[4];
|
u8 sig[4];
|
||||||
|
int ret = -EINVAL;
|
||||||
|
|
||||||
pr_debug("AFFS: read_super(%s)\n",data ? (const char *)data : "no options");
|
pr_debug("AFFS: read_super(%s)\n",data ? (const char *)data : "no options");
|
||||||
|
|
||||||
|
@ -444,7 +444,12 @@ got_root:
|
||||||
|
|
||||||
/* set up enough so that it can read an inode */
|
/* set up enough so that it can read an inode */
|
||||||
|
|
||||||
root_inode = iget(sb, root_block);
|
root_inode = affs_iget(sb, root_block);
|
||||||
|
if (IS_ERR(root_inode)) {
|
||||||
|
ret = PTR_ERR(root_inode);
|
||||||
|
goto out_error_noinode;
|
||||||
|
}
|
||||||
|
|
||||||
sb->s_root = d_alloc_root(root_inode);
|
sb->s_root = d_alloc_root(root_inode);
|
||||||
if (!sb->s_root) {
|
if (!sb->s_root) {
|
||||||
printk(KERN_ERR "AFFS: Get root inode failed\n");
|
printk(KERN_ERR "AFFS: Get root inode failed\n");
|
||||||
|
@ -461,12 +466,13 @@ got_root:
|
||||||
out_error:
|
out_error:
|
||||||
if (root_inode)
|
if (root_inode)
|
||||||
iput(root_inode);
|
iput(root_inode);
|
||||||
|
out_error_noinode:
|
||||||
kfree(sbi->s_bitmap);
|
kfree(sbi->s_bitmap);
|
||||||
affs_brelse(root_bh);
|
affs_brelse(root_bh);
|
||||||
kfree(sbi->s_prefix);
|
kfree(sbi->s_prefix);
|
||||||
kfree(sbi);
|
kfree(sbi);
|
||||||
sb->s_fs_info = NULL;
|
sb->s_fs_info = NULL;
|
||||||
return -EINVAL;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int
|
static int
|
||||||
|
|
Loading…
Reference in a new issue