pstore: fix leaking ->i_private
Move kfree() of i_private out of ->unlink() and into ->evict_inode() Signed-off-by: Tony Luck <tony.luck@intel.com> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
parent
b7ed78f565
commit
a872d51010
|
@ -73,11 +73,16 @@ static int pstore_unlink(struct inode *dir, struct dentry *dentry)
|
||||||
struct pstore_private *p = dentry->d_inode->i_private;
|
struct pstore_private *p = dentry->d_inode->i_private;
|
||||||
|
|
||||||
p->erase(p->id);
|
p->erase(p->id);
|
||||||
kfree(p);
|
|
||||||
|
|
||||||
return simple_unlink(dir, dentry);
|
return simple_unlink(dir, dentry);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static void pstore_evict_inode(struct inode *inode)
|
||||||
|
{
|
||||||
|
end_writeback(inode);
|
||||||
|
kfree(inode->i_private);
|
||||||
|
}
|
||||||
|
|
||||||
static const struct inode_operations pstore_dir_inode_operations = {
|
static const struct inode_operations pstore_dir_inode_operations = {
|
||||||
.lookup = simple_lookup,
|
.lookup = simple_lookup,
|
||||||
.unlink = pstore_unlink,
|
.unlink = pstore_unlink,
|
||||||
|
@ -110,6 +115,7 @@ static struct inode *pstore_get_inode(struct super_block *sb,
|
||||||
static const struct super_operations pstore_ops = {
|
static const struct super_operations pstore_ops = {
|
||||||
.statfs = simple_statfs,
|
.statfs = simple_statfs,
|
||||||
.drop_inode = generic_delete_inode,
|
.drop_inode = generic_delete_inode,
|
||||||
|
.evict_inode = pstore_evict_inode,
|
||||||
.show_options = generic_show_options,
|
.show_options = generic_show_options,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue