summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNiels de Vos <ndevos@redhat.com>2017-04-14 17:19:47 +0200
committerJeff Darcy <jeff@pl.atyp.us>2017-04-14 17:35:34 -0400
commitd7f0ead43969e755d07ae92c669d63527a58c572 (patch)
treec49ff3557d6b9a2c76f44d85c4b31883a9fd7c58
parent73f66bf422d6184c72c01493920687e880ad8918 (diff)
meta: free this->private in fini()
For every graph switch or de-init (includes calling `glfs_fini`) the meta_priv_t structure (80 bytes on x86_64) leaks. Because it is allocated in the `init()` funtion, it should be free'd in `fini()`. Change-Id: Ie45a9c9a520d55bebd0eb04c2ac667bc8c1c5acb BUG: 1442411 Signed-off-by: Niels de Vos <ndevos@redhat.com> Reviewed-on: https://review.gluster.org/17061 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: Jeff Darcy <jeff@pl.atyp.us>
-rw-r--r--xlators/meta/src/meta.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/xlators/meta/src/meta.c b/xlators/meta/src/meta.c
index 25720136714..7ea01ab147b 100644
--- a/xlators/meta/src/meta.c
+++ b/xlators/meta/src/meta.c
@@ -243,6 +243,7 @@ out:
int
fini (xlator_t *this)
{
+ GF_FREE (this->private);
return 0;
}