[Users] memory leak in presence module?

[Anca-Maria Vamanu]

Hello,

I have made a fix. Please test again with sipp and say if it works now.

regards,

Anca

Klaus Darilion wrote:

> Hi!
>
> Openser 1.2 runs out of memory during presence load tests with sipp.
>
> Config snippet:
>
> if (is_method("PUBLISH")) {
>   xlog("L_INFO","$ci PUBLISH detected ... outside t_newtran\n");
>   if (t_newtran()) {
>     xlog("L_INFO","$ci PUBLISH detected ... inside t_newtran\n");
>     handle_publish();
>     xlog("L_INFO","$ci PUBLISH detected, handle_publish ... done\n");
>     exit;
>   } else {
>     sl_reply_error();
>     exit;
>   }
> }
>
> Memory log after openser stop (after 20 minutes of inactivity):
> http://pernau.at/kd/openser/debug-stop.txt
>
>
> regards
> klaus   
>
>
> Klaus Darilion wrote:
>
>> Hi Cesc!
>>
>> Where do you have t_newtran() - in ser or openser?
>>
>> I have it here in openser just before handle_publish(). It works but 
>> now openser has run out of mem during my sipp performance tests :-(
>>
>> ...more to come....
>>
>> Cesc wrote:
>>
>>> I think I can answer that one ... stateless?
>>>
>>> I have a presence openser proxied by a SER, both on same machine ...
>>> For easier configuration purposes, openser and ser communicate via
>>> 127.0.0.1 ...
>>>
>>> The result is that OK from the PUAs do not get to openser (i guess SER
>>> gets confussed by the vias of openser being 0.0.0.0:5060 and the
>>> 127.0.0.1 stuff) ... So, openser gets no OK, but it does not matter
>>> ...
>>> BTW, i have a t_newtran() in there ...
>>>
>>> Oh, for sure my configuration is messy and improvable ... but for now
>>> it works steady :) But the OKs not going through is worrying me a bit
>>> ... and also openser not "timing out" or giving an error.
>>>
>>> Cesc
>>>
>>> On 4/25/07, Klaus Darilion <klaus.mailinglists at pernau.at> wrote:
>>>
>>>> Hi!
>>>>
>>>> Does presence module handles PUBLISH/SUBSCRIBE stateful or stateless?
>>>>
>>>> Thus, how should retransmission be handled. Will they be detected by
>>>> presence module or is it possible to wrap the handle_publish inside
>>>> t_newtran()?
>>>>
>>>> regards
>>>> klaus
>>>>
>>>> _______________________________________________
>>>> Users mailing list
>>>> Users at openser.org
>>>> http://openser.org/cgi-bin/mailman/listinfo/users
>>>>
>>
>> _______________________________________________
>> Users mailing list
>> Users at openser.org
>> http://openser.org/cgi-bin/mailman/listinfo/users
>
>
> _______________________________________________
> Users mailing list
> Users at openser.org
> http://openser.org/cgi-bin/mailman/listinfo/users
>