<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Ops, sorry, I send a wrong patch, that one creates an API for
ndb_redis module.<br>
<br>
The patch attached now is the correct one.<br>
<br>
Regards,<br>
Vicente.<br>
<br>
<br>
<br>
On 05/02/2012 09:33 PM, Vicente Hernando wrote:
<blockquote cite="mid:4FA18C10.5000206@systemonenoc.com" type="cite">Hello,
<br>
<br>
looking at ndb_redis module source code I see replies are not
freed when module is destroyed.
<br>
<br>
Also a reply is only freed when another one with same name is
performed. This could cause problems if an user would ask for
<br>
a lot of different named replies.
<br>
<br>
IMHO freeing reply list when module is destroyed and creating a
new redisc_free_reply function would solve this.
<br>
<br>
<br>
I attach a patch that compiles but not tested to show the idea.
<br>
<br>
If you agree I can perform the task, I suppose redisc_free_reply
function should also be exported in cmd_export_t module structure.
<br>
<br>
<br>
This solution would be compatible with old ndb redis module
configuration.
<br>
<br>
<br>
<br>
Regards,
<br>
Vicente.
<br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
sr-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a>
<a class="moz-txt-link-freetext" href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a>
</pre>
</blockquote>
<br>
</body>
</html>