summaryrefslogtreecommitdiffstats
path: root/xlators/cluster
diff options
context:
space:
mode:
authorJoseph Fernandes <josferna@redhat.com>2015-12-03 15:40:16 +0530
committerDan Lambright <dlambrig@redhat.com>2015-12-09 09:30:36 -0800
commit117dce7a489624ad7a7ab6b9299b3e2c1aa0bab6 (patch)
tree40ef3e8464e967adf94b465a8ce08745b94d31e2 /xlators/cluster
parent10d2b90770a09b283c7141c3c422a6349c0c30b4 (diff)
tier/dht: Fix mem leak from lookup response dict
Fixing memory leak from response dict during a parent lookup to get the path. Backport of http://review.gluster.org/12867 > Change-Id: I60c23d0b25e7f763f0e53c40e71ee053aba6d555 > BUG: 1288019 > Signed-off-by: Joseph Fernandes <josferna@redhat.com> > Reviewed-on: http://review.gluster.org/12867 > Tested-by: Gluster Build System <jenkins@build.gluster.com> > Reviewed-by: Joseph Fernandes > Tested-by: Joseph Fernandes Signed-off-by: Joseph Fernandes <josferna@redhat.com> Change-Id: I77dfbb097e0200ace4c1adcf48560caf283007d7 BUG: 1288992 Reviewed-on: http://review.gluster.org/12889 Tested-by: NetBSD Build System <jenkins@build.gluster.org> Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Dan Lambright <dlambrig@redhat.com> Tested-by: Dan Lambright <dlambrig@redhat.com>
Diffstat (limited to 'xlators/cluster')
-rw-r--r--xlators/cluster/dht/src/tier.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/xlators/cluster/dht/src/tier.c b/xlators/cluster/dht/src/tier.c
index 8685d84f7c6..426a09a71af 100644
--- a/xlators/cluster/dht/src/tier.c
+++ b/xlators/cluster/dht/src/tier.c
@@ -526,6 +526,12 @@ abort:
loc_wipe (&loc);
loc_wipe (&p_loc);
+
+ if (xdata_response) {
+ dict_unref (xdata_response);
+ xdata_response = NULL;
+ }
+
if ((total_files > defrag->tier_conf.max_migrate_files)
|| (total_migrated_bytes >
defrag->tier_conf.max_migrate_bytes)) {