From: Younger Liu <younger.liu@huawei.com>
Date: Wed, 3 Jul 2013 22:00:54 +0000 (-0700)
Subject: fs/ocfs2/cluster/tcp.c: free sc->sc_page in sc_kref_release()
X-Git-Tag: firefly_0821_release~176^2~5786^2~432
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=d3e3b41b3dffff50c66651d60146b155d6ce0484;p=firefly-linux-kernel-4.4.55.git

fs/ocfs2/cluster/tcp.c: free sc->sc_page in sc_kref_release()

There is a memory leak in sc_kref_release().  When free struct
o2net_sock_container (sc), we should release sc->sc_page.

Signed-off-by: Younger Liu <younger.liu@huawei.com>
Reviewed-by: Jie Liu <jeff.liu@oracle.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Mark Fasheh <mfasheh@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---

diff --git a/fs/ocfs2/cluster/tcp.c b/fs/ocfs2/cluster/tcp.c
index aa88bd8bcedc..bb9a48bfbd01 100644
--- a/fs/ocfs2/cluster/tcp.c
+++ b/fs/ocfs2/cluster/tcp.c
@@ -406,6 +406,9 @@ static void sc_kref_release(struct kref *kref)
 	sc->sc_node = NULL;
 
 	o2net_debug_del_sc(sc);
+
+	if (sc->sc_page)
+		__free_page(sc->sc_page);
 	kfree(sc);
 }