diff options
| author | Richard Wareing <rwareing@fb.com> | 2016-03-18 14:20:53 -0700 |
|---|---|---|
| committer | Shreyas Siravara <sshreyas@fb.com> | 2017-08-30 02:33:00 +0000 |
| commit | 9f9da37e3afa1f9394fb5edf49334ef9d6a6dd00 (patch) | |
| tree | 035b4f7d5170741bcb1eaa09bfbb21d5eebd4240 /libglusterfs | |
| parent | 9bc7c9df41f4e7fc0b8462cbc56298d421cb8722 (diff) | |
libglusterfs: Fix leak in client_t destroy struct
Summary:
- People tell me free'ing authentication data after we are done with it
is a good thing to do.
- This is a port of D3071688.
Test Plan:
- Ran valgrind w/ looping FUSE mount requests and watches "definitely
lost", it no longer goes up the more we mount/umount
Reviewers: kvigor, sshreyas
Subscribers: moox, dld, dph
Change-Id: Ia3d4a5bdd431006bd2d39b957cfe27f1ba3ef16e
Reviewed-on: https://review.gluster.org/18142
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shreyas Siravara <sshreyas@fb.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/client_t.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/libglusterfs/src/client_t.c b/libglusterfs/src/client_t.c index 3e0e5936ae2..b3eb4e4df8c 100644 --- a/libglusterfs/src/client_t.c +++ b/libglusterfs/src/client_t.c @@ -366,6 +366,8 @@ client_destroy (client_t *client) } } GF_FREE (client->auth.data); + GF_FREE (client->auth.username); + GF_FREE (client->auth.passwd); GF_FREE (client->scratch_ctx.ctx); GF_FREE (client->client_uid); GF_FREE (client); |