diff options
| author | Raghavendra G <raghavendra@gluster.com> | 2009-12-06 04:39:13 +0000 |
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2009-12-06 02:30:21 -0800 |
| commit | bf23224e5dc2af09efd0888490f7af191052c874 (patch) | |
| tree | 606280628bb506ad819af5df691ae2fe70fb5341 /xlators | |
| parent | 4d8c682181822834f46adaa6afea029556a0bb59 (diff) | |
performance/stat-prefetch: fix memory leak.
- rbt_hash_remove only removes entry from the hash table. It does not free
the data.
Signed-off-by: Raghavendra G <raghavendra@gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 438 (Memory leak)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=438
Diffstat (limited to 'xlators')
| -rw-r--r-- | xlators/performance/stat-prefetch/src/stat-prefetch.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/xlators/performance/stat-prefetch/src/stat-prefetch.c b/xlators/performance/stat-prefetch/src/stat-prefetch.c index 729924ff..7722c7bc 100644 --- a/xlators/performance/stat-prefetch/src/stat-prefetch.c +++ b/xlators/performance/stat-prefetch/src/stat-prefetch.c @@ -377,6 +377,7 @@ sp_cache_remove_entry (sp_cache_t *cache, char *name, char remove_all) rbthash_table_t *table = NULL; xlator_t *this; sp_private_t *priv = NULL; + void *data = NULL; if ((cache == NULL) || ((name == NULL) && !remove_all)) { goto out; @@ -408,7 +409,9 @@ sp_cache_remove_entry (sp_cache_t *cache, char *name, char remove_all) ret = 0; } } else { - rbthash_remove (cache->table, name, strlen (name)); + data = rbthash_remove (cache->table, name, + strlen (name)); + FREE (data); ret = 0; } } |