diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2013-10-01 19:11:21 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2013-10-01 19:11:21 +0200 |
| commit | c2d22ecd3cf3c49aeecdc10ffd7af10ec1f7eab6 (patch) | |
| tree | cb180b5926c4b81a4385f785c7f0b856f8620fcd /fs/super.c | |
| parent | afs: dget_parent() can't return a negative dentry (diff) | |
| download | linux-c2d22ecd3cf3c49aeecdc10ffd7af10ec1f7eab6.tar.xz linux-c2d22ecd3cf3c49aeecdc10ffd7af10ec1f7eab6.zip | |
fs/super.c: fix lru_list leak for real
Freeing ->s_{inode,dentry}_lru in deactivate_locked_super() is wrong;
the right place is destroy_super(). As it is, we leak them if sget()
decides that new superblock it has allocated (and never shown to
anybody) isn't needed and should be freed.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to '')
| -rw-r--r-- | fs/super.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/super.c b/fs/super.c index 3a96c9783a8b..0225c20f8770 100644 --- a/fs/super.c +++ b/fs/super.c @@ -264,6 +264,8 @@ out_free_sb: */ static inline void destroy_super(struct super_block *s) { + list_lru_destroy(&s->s_dentry_lru); + list_lru_destroy(&s->s_inode_lru); #ifdef CONFIG_SMP free_percpu(s->s_files); #endif @@ -323,8 +325,6 @@ void deactivate_locked_super(struct super_block *s) /* caches are now gone, we can safely kill the shrinker now */ unregister_shrinker(&s->s_shrink); - list_lru_destroy(&s->s_dentry_lru); - list_lru_destroy(&s->s_inode_lru); put_filesystem(fs); put_super(s); |