[sr-dev] Possible memory leakage in ndb_redis module

[Vicente Hernando]

Ops, sorry, I send a wrong patch, that one creates an API for ndb_redis 
module.

The patch attached now is the correct one.

Regards,
Vicente.



On 05/02/2012 09:33 PM, Vicente Hernando wrote:
> Hello,
>
> looking at ndb_redis module source code I see replies are not freed 
> when module is destroyed.
>
> Also a reply is only freed when another one with same name is 
> performed. This could cause problems if an user would ask for
> a lot of different named replies.
>
> IMHO freeing reply list when module is destroyed and creating a new 
> redisc_free_reply function would solve this.
>
>
> I attach a patch that compiles but not tested to show the idea.
>
> If you agree I can perform the task, I suppose redisc_free_reply 
> function should also be exported in cmd_export_t module structure.
>
>
> This solution would be compatible with old ndb redis module 
> configuration.
>
>
>
> Regards,
> Vicente.
>
>
>
> _______________________________________________
> sr-dev mailing list
> sr-dev at lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20120502/1dcf60c9/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ndb_redis_free_reply.patch
Type: text/x-diff
Size: 1637 bytes
Desc: not available
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20120502/1dcf60c9/attachment-0001.patch>