search

typpo / rfc5766-turn-server

Automatically exported from code.google.com/p/rfc5766-turn-server
1 stars 0 forks source link

Redis reply objects not freed #61

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. use redis as the user database

What is the expected output? What do you see instead?
When left idle, but with a populated database, the memory used should not 
increase.

What version of the product are you using? On what operating system?
2.6.3.1 on Linux Fedora 19

Please provide any additional information below.
All objects returned from redisCommand() need to be freed with freeReplyObject()

Original issue reported on code.google.com by kevindem...@gmail.com on 8 Oct 2013 at 2:10

GoogleCodeExporter commented 9 years ago 
I should have said that all non null pointers returned by redisCommand() need 
to be passed to freeReplyObject(). Also the redis contexts from each 
get_redis_connection() need to be releases with redisFree()

Original comment by kevindem...@gmail.com on 8 Oct 2013 at 2:13

GoogleCodeExporter commented 9 years ago

Original comment by mom040...@gmail.com on 8 Oct 2013 at 3:31

GoogleCodeExporter commented 9 years ago 
The result of get_redis_connection() must not be freed because this is a static 
singleton value (in this program)

Original comment by mom040...@gmail.com on 8 Oct 2013 at 7:34

GoogleCodeExporter commented 9 years ago 
Fixed in the SVN repository; will be included into 2.6.5.1, soon.

Original comment by mom040...@gmail.com on 8 Oct 2013 at 7:49

GoogleCodeExporter commented 9 years ago 
Fixed in 2.6.5.1

Original comment by mom040...@gmail.com on 8 Oct 2013 at 8:10

GoogleCodeExporter commented 9 years ago

Original comment by mom040...@gmail.com on 8 Oct 2013 at 8:11

GoogleCodeExporter commented 9 years ago 
https://groups.google.com/forum/#!topic/turn-server-project-rfc5766-turn-server/
1gI4jFuVinE

Original comment by mom040...@gmail.com on 9 Oct 2013 at 7:27