[SR-Users] Possible memory leak in parse_pai.c

Daniel-Constantin Mierla miconda at gmail.com
Mon Apr 22 09:49:43 CEST 2013


Hello,

On 4/19/13 12:42 PM, Tais Plougmann Hansen wrote:
> I had added $ai to acc log_extra like this:
>
> modparam("acc", "log_extra",
>     "src_user=$fU;src_domain=$fd;src_ip=$si;"
>     "dst_ouser=$tU;dst_user=$rU;dst_domain=$rd"
>     "p_asserted_identity=$ai;remote_party_id=$re")
>
> Once I commented out the last line, the leak vanished.
>
> Apr 17 16:55:41 gws1 /usr/sbin/kamailio[10649]: ALERT: qm_status: 
> used= 646728, used+overhead=1099696, free=7288912
> Apr 19 12:28:04 gws1 /usr/sbin/kamailio[10649]: ALERT: qm_status: 
> used= 646752, used+overhead=1100104, free=7288504
>
> I'll test the avp intermediary in the next few days.
ok, I will look over acc module for this case.

Do you still have the xlog lines printing these variables?

Cheers,
Daniel

>
>
>
>
> On Thu, Apr 18, 2013 at 10:13 AM, Daniel-Constantin Mierla 
> <miconda at gmail.com <mailto:miconda at gmail.com>> wrote:
>
>     Hello,
>
>     I looked quickly to the core and seems ok, I will check the
>     accounting later.
>
>     Do you simply account $ai or is any other attribute related to it?
>
>     If you have time to test, can you put the $ai in an avp during the
>     request processing and account that avp? If the leak is not shown
>     anymore in that case, it narrows down a lot where to look at.
>
>     Cheers,
>     Daniel
>
>
>     On 4/17/13 5:19 PM, Tais Plougmann Hansen wrote:
>>     Hi,
>>
>>     I'm seeing a possible leak in parsing the P-Asserted-Identity,
>>     and possibly P-Preferred-Identity headers in kamailio-3.3.4.
>>
>>     I have a server running in which one thread has currently most of
>>     the 8 MB private memory allocated. At restart qm_sums reports
>>     parse_pai.c as sitting on most of the memory:
>>
>>     Apr 16 16:38:58 gws1 /usr/sbin/kamailio[9398]: INFO: qm_status:
>>     heap size= 8388608
>>     Apr 16 16:38:58 gws1 /usr/sbin/kamailio[9398]: INFO: qm_status:
>>     used= 5369240, used+overhead=6417216, free=1971392
>>     [...]
>>     Apr 16 16:39:03 gws1 /usr/sbin/kamailio[9398]: INFO: qm_sums:
>>      count=  6198 size=   4721776 bytes from <core>: parse_pai.c:
>>     parse_pai_header(58)
>>
>>     I am currently using the header through $ai for xlog statements,
>>     the accounting module and a validity check (if (!($(ai{uri.user})
>>     =~ "^\+[1-9]+")) {...}).
>>
>>     From digging through the logs it could seem like this happens
>>     when using $ai in the accounting module. I'll try removing it
>>     from accounting (log_extra) and see if it stops leaking.
>>
>>     -- 
>>     Tais Plougmann Hansen
>>
>>     	OSD Consulting ApS
>>     osd.dk <http://osd.dk/>
>>     Tel: +45 78101078 <tel:%2B45%2078101078>
>>     CVR: DK31332737
>>
>>
>>
>>     _______________________________________________
>>     SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
>>     sr-users at lists.sip-router.org  <mailto:sr-users at lists.sip-router.org>
>>     http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
>
>     -- 
>     Daniel-Constantin Mierla -http://www.asipto.com
>     http://twitter.com/#!/miconda  <http://twitter.com/#%21/miconda>  -http://www.linkedin.com/in/miconda
>     Kamailio World Conference, April 16-17, 2013, Berlin
>       -http://conference.kamailio.com  -
>
>
>     _______________________________________________
>     SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing
>     list
>     sr-users at lists.sip-router.org <mailto:sr-users at lists.sip-router.org>
>     http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
>
>
>
>
> -- 
> Tais Plougmann Hansen
> OSD Consulting ApS
>

-- 
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-users/attachments/20130422/ce208e3a/attachment-0001.htm>


More information about the sr-users mailing list