andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

meta: free this->private in fini()

Browse Source 
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>
This commit is contained in:
Niels de Vos 2017-04-14 17:19:47 +02:00 committed by Jeff Darcy
parent 73f66bf422
commit d7f0ead439
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
xlators/meta/src/meta.c
View File

@ -243,6 +243,7 @@ out:
int
fini (xlator_t *this)
{
GF_FREE (this->private);
return 0;
}