diff options
author | Kinglong Mee <kinglongmee@gmail.com> | 2017-04-20 02:08:32 -0400 |
---|---|---|
committer | Niels de Vos <ndevos@redhat.com> | 2017-04-27 10:44:49 +0000 |
commit | 232882ac6f93ebac847715f522a99960bcfb11a6 (patch) | |
tree | a61bd0426b0f0e636b8175afddb13a71a7b94cfa | |
parent | 9f5e59abfbf529b91d699143b0c69c8748ac6253 (diff) | |
download | glusterfs-232882ac6f93ebac847715f522a99960bcfb11a6.tar.gz glusterfs-232882ac6f93ebac847715f522a99960bcfb11a6.tar.xz glusterfs-232882ac6f93ebac847715f522a99960bcfb11a6.zip |
io-stats: fix some memory leak after fini
Change-Id: Id057b711d7f615c6d2db1ebbca2f17d0afacb2ae
BUG: 1444023
Signed-off-by: Kinglong Mee <mijinlong@open-fs.com>
Reviewed-on: https://review.gluster.org/17089
Reviewed-by: Niels de Vos <ndevos@redhat.com>
Tested-by: Niels de Vos <ndevos@redhat.com>
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>
-rw-r--r-- | xlators/debug/io-stats/src/io-stats.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/xlators/debug/io-stats/src/io-stats.c b/xlators/debug/io-stats/src/io-stats.c index ac99f98d85..d9d1e1dff1 100644 --- a/xlators/debug/io-stats/src/io-stats.c +++ b/xlators/debug/io-stats/src/io-stats.c @@ -3663,6 +3663,7 @@ ios_destroy_top_stats (struct ios_conf *conf) GF_FREE (list); list_head->members--; } + GF_FREE (list_head->iosstats); } for (i = 0; i < IOS_STATS_THRU_MAX; i++) { @@ -3678,6 +3679,7 @@ ios_destroy_top_stats (struct ios_conf *conf) GF_FREE (list); list_head->members--; } + GF_FREE (list_head->iosstats); } UNLOCK (&conf->lock); @@ -3880,7 +3882,9 @@ ios_conf_destroy (struct ios_conf *conf) ios_destroy_top_stats (conf); _ios_destroy_dump_thread (conf); + ios_destroy_sample_buf (conf->ios_sample_buf); LOCK_DESTROY (&conf->lock); + GF_FREE(conf->dnscache); GF_FREE(conf); } |