aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSha Zhengju <[email protected]>2012-05-10 13:01:45 -0700
committerLinus Torvalds <[email protected]>2012-05-10 15:06:44 -0700
commit8c7577637ca31385e92769a77e2ab5b428e8b99c (patch)
treeecf350b4defdd9ad31a9d1eb6353b890c5951eb9
parent5e2bf0142231194d36fdc9596b36a261ed2b9fe7 (diff)
memcg: free spare array to avoid memory leak
When the last event is unregistered, there is no need to keep the spare array anymore. So free it to avoid memory leak. Signed-off-by: Sha Zhengju <[email protected]> Acked-by: KAMEZAWA Hiroyuki <[email protected]> Reviewed-by: Kirill A. Shutemov <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
-rw-r--r--mm/memcontrol.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index 31ab9c3f0178..b659260c56ad 100644
--- a/mm/memcontrol.c
+++ b/mm/memcontrol.c
@@ -4507,6 +4507,12 @@ static void mem_cgroup_usage_unregister_event(struct cgroup *cgrp,
swap_buffers:
/* Swap primary and spare array */
thresholds->spare = thresholds->primary;
+ /* If all events are unregistered, free the spare array */
+ if (!new) {
+ kfree(thresholds->spare);
+ thresholds->spare = NULL;
+ }
+
rcu_assign_pointer(thresholds->primary, new);
/* To be sure that nobody uses thresholds */