summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeff Darcy <jdarcy@redhat.com>2017-02-08 19:45:46 -0500
committerShyamsundar Ranganathan <srangana@redhat.com>2017-02-09 16:01:45 -0500
commit226d7c442509172b2209515841ef499ec12fc9f2 (patch)
tree414d519761643d75bd6aedb94f2c014bd8000474
parent94374de6984da45e668d81a3794aa8137aabeb0f (diff)
libglusterfs: fix serious leak of xlator_t structures
There's a lot of logic (and some long comments) around how to free these structures safely, but then we didn't do it. Now we do. Backport of: > Change-Id: I9731ae75c60e99cc43d33d0813a86912db97fd96 > BUG: 1420571 > Reviewed-on: https://review.gluster.org/16570 Change-Id: I54415b614b277224196f5723bce5a4c5a404d881 BUG: 1420810 Signed-off-by: Jeff Darcy <jdarcy@redhat.com> Reviewed-on: https://review.gluster.org/16583 Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
-rw-r--r--libglusterfs/src/xlator.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/libglusterfs/src/xlator.c b/libglusterfs/src/xlator.c
index 4702ea3eb77..0d09b3fbc82 100644
--- a/libglusterfs/src/xlator.c
+++ b/libglusterfs/src/xlator.c
@@ -752,6 +752,7 @@ xlator_tree_free_memacct (xlator_t *tree)
while (prev) {
trav = prev->next;
xlator_memrec_free (prev);
+ GF_FREE (prev);
prev = trav;
}