mirror of
https://github.com/fluencelabs/redis
synced 2025-03-16 15:40:49 +00:00
Memory leak fixed: release client's bpop->keys dictionary.
Refactoring performed after issue #801 resolution (see commit 2f87cf8b0162bd9d78c3a89860c0971cd71d39db) introduced a memory leak that is fixed by this commit. I simply forgot to free the new allocated dictionary in the client structure trusting the output of "make test" on OSX. However due to changes in the "leaks" utility the test was no longer testing memory leaks. This problem was also fixed. Fortunately the CI test running at ci.redis.io spotted the bug in the valgrind run. The leak never ended into a stable release.
This commit is contained in:
parent
a18ca73681
commit
ab2924cff3
@ -605,6 +605,7 @@ void freeClient(redisClient *c) {
|
||||
c->querybuf = NULL;
|
||||
if (c->flags & REDIS_BLOCKED)
|
||||
unblockClientWaitingData(c);
|
||||
dictRelease(c->bpop.keys);
|
||||
|
||||
/* UNWATCH all the keys */
|
||||
unwatchAllKeys(c);
|
||||
|
Loading…
x
Reference in New Issue
Block a user