diff options
| author | Niels de Vos <ndevos@redhat.com> | 2017-04-14 17:19:47 +0200 |
|---|---|---|
| committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-04-14 17:35:34 -0400 |
| commit | d7f0ead43969e755d07ae92c669d63527a58c572 (patch) | |
| tree | c49ff3557d6b9a2c76f44d85c4b31883a9fd7c58 | |
| parent | 73f66bf422d6184c72c01493920687e880ad8918 (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.c | 1 |
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; } |