<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body bgcolor="#FFFFFF" fpstyle="1" ocsi="0">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">Hi Daniel,
<div><br>
</div>
<div>please take a look at msg_presentity_clean function in publish.c module presence.</div>
<div><br>
</div>
<div>if the initial query returns a result</div>
<div>if pres_notifier_processes = 0 (immediate notify)</div>
<div>it calls publ_notify function</div>
<div>in publ_notify function in notify.c </div>
<div>
<div>if(p->event->agg_nbody) // dialoginfo aggregates the results</div>
<div>{<span class="Apple-tab-span" style="white-space:pre"> </span></div>
<div><span class="Apple-tab-span" style="white-space:pre"></span>notify_body = <b>
get_p_notify_body</b>(pres_uri, p->event , offline_etag, NULL);</div>
</div>
<div> .....</div>
<div><b>get_p_notify_body </b>queries the <b>presentity </b>table to get other records to aggregate body</div>
<div>which will override the result in the connection.</div>
<div><br>
</div>
<div>so the process is the same but it calls the query 2 times in a row without freeing the 1st result.</div>
<div><br>
</div>
<div><br>
</div>
<div><br>
</div>
<div><br>
</div>
<div>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div id="divRpF523141" style="direction: ltr;"><font face="Tahoma" size="2" color="#000000"><b>From:</b> sr-dev-bounces@lists.sip-router.org [sr-dev-bounces@lists.sip-router.org] on behalf of Daniel-Constantin Mierla [miconda@gmail.com]<br>
<b>Sent:</b> Friday, September 12, 2014 12:15 PM<br>
<b>To:</b> Kamailio (SER) - Development Mailing List<br>
<b>Subject:</b> Re: [sr-dev] kazoo module updates & documentation<br>
</font><br>
</div>
<div></div>
<div>Hello,<br>
<br>
changes to kazoo modules are ok, you can merge them.<br>
<br>
But the one for db_text I don't find it (yet) necessary, because it may solve an issue in a wrong way.<br>
<br>
I asked you to push the other patch on db_text related to blob type handling in freeing the value. That can be done. For the one storing the result on db_res variable, I will comment separately.<br>
<br>
Daniel<br>
<br>
<div class="moz-cite-prefix">On 12/09/14 20:55, Luis Azedo wrote:<br>
</div>
<blockquote type="cite"><style type="text/css" id="owaParaStyle"></style>
<div style="direction:ltr; font-family:Tahoma; color:#000000; font-size:10pt">Hi,
<div><br>
</div>
<div>i would like to propose the small changes to the kazoo module in branch lazedo/kazoo to go into master.</div>
<div>it's mostly code cleanup & dependency removal.</div>
<div><br>
</div>
<div>also added the documentation on that branch and would appreciate any feedback.</div>
<div><br>
</div>
<div><br>
</div>
<div>there is also a patch to the db_text module. the comment on that commit tells the reason for that patch.</div>
<div><br>
</div>
<div>Thanks</div>
<div><br>
</div>
</div>
<br>
<fieldset class="mimeAttachmentHeader" target="_blank"></fieldset> <br>
<pre>_______________________________________________
sr-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sr-dev@lists.sip-router.org" target="_blank">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" target="_blank">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a>
</pre>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Daniel-Constantin Mierla
<a class="moz-txt-link-freetext" href="http://twitter.com/#!/miconda" target="_blank">http://twitter.com/#!/miconda</a> - <a class="moz-txt-link-freetext" href="http://www.linkedin.com/in/miconda" target="_blank">http://www.linkedin.com/in/miconda</a>
Next Kamailio Advanced Trainings 2014 - <a class="moz-txt-link-freetext" href="http://www.asipto.com" target="_blank">http://www.asipto.com</a>
Sep 22-25, Berlin, Germany</pre>
</div>
</div>
</div>
</div>
</body>
</html>