<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hello,<br>
<br>
I missed the commit of that functions, was done by Peter Dunkley, I
had in mind to add a PV to return the sdp body and other attributes.<br>
<br>
You can commit the second patch, the one without
pkg_malloc()/_free(), because avps duplicate the value in shared
memory, cloning in pkg first makes no difference.<br>
<br>
Cheers,<br>
Daniel<br>
<br>
<div class="moz-cite-prefix">On 8/31/12 2:07 PM, Vicente Hernando
wrote:<br>
</div>
<blockquote cite="mid:5040A916.1090009@systemonenoc.com" type="cite">Sorry,
previous attachment did not work.
<br>
<br>
On 08/31/2012 02:06 PM, Vicente Hernando wrote:
<br>
<blockquote type="cite">Hello,
<br>
<br>
within w_get_sdp function in modules/sdpops/sdpops_mod.c
<br>
<br>
} else {
<br>
avp_val.s.s = pkg_malloc(sdp->raw_sdp.len);
<br>
avp_val.s.len = sdp->raw_sdp.len;
<br>
if (avp_val.s.s == NULL)
<br>
<br>
I do not find anywhere where avp_val.s.s variable is freed.
<br>
<br>
<br>
So I attach two possible patches to solve that in two different
ways.
<br>
<br>
Kind regards,
<br>
Vicente.
<br>
<br>
_______________________________________________
<br>
sr-dev mailing list
<br>
<a class="moz-txt-link-abbreviated" href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a>
<br>
<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>
<br>
<br>
</blockquote>
<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>
<pre class="moz-signature" cols="72">--
Daniel-Constantin Mierla - <a class="moz-txt-link-freetext" href="http://www.asipto.com">http://www.asipto.com</a>
<a class="moz-txt-link-freetext" href="http://twitter.com/#!/miconda">http://twitter.com/#!/miconda</a> - <a class="moz-txt-link-freetext" href="http://www.linkedin.com/in/miconda">http://www.linkedin.com/in/miconda</a>
Kamailio Advanced Training, Berlin, Nov 5-8, 2012 - <a class="moz-txt-link-freetext" href="http://asipto.com/u/kat">http://asipto.com/u/kat</a></pre>
</body>
</html>