ocfs2: free 'dummy_sc' in sc_fop_release() to prevent memory leak
authorpiaojun <piaojun@huawei.com>
Thu, 6 Jul 2017 22:36:13 +0000 (15:36 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 6 Jul 2017 23:24:30 +0000 (16:24 -0700)
'sd->dbg_sock' is malloced in sc_common_open(), but not freed at the end
of sc_fop_release().

Link: http://lkml.kernel.org/r/594FB0A4.2050105@huawei.com
Signed-off-by: Jun Piao <piaojun@huawei.com>
Reviewed-by: Joseph Qi <jiangqi903@gmail.com>
Cc: Mark Fasheh <mfasheh@versity.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Junxiao Bi <junxiao.bi@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ocfs2/cluster/netdebug.c

index 564c504d6efd8c99f903e95d96a87b47ce7db36a..74a21f6695c8526ec6efe7aca48d5d75fec76e4d 100644 (file)
@@ -426,6 +426,7 @@ static int sc_fop_release(struct inode *inode, struct file *file)
        struct o2net_sock_container *dummy_sc = sd->dbg_sock;
 
        o2net_debug_del_sc(dummy_sc);
+       kfree(dummy_sc);
        return seq_release_private(inode, file);
 }