[Devel] Re: [Users] Out of Memory

[Bogdan-Andrei Iancu]

Hi Marcos,

I manage to find and hopefully fix the problem. I just committed a patch 
on the cvs (for devel branch) - please update and give it a try to see 
if the problem is fixed.

Regards,
Bogdan


Marcos Hack wrote:

>Hi Bogdan,
>
>More one "little" mistake:
>
>Problem occur only when using add_local_rport() and forward() (that
>calls build_req_buf_from_sip_req()) functions together, and not
>force_rport() as I said before.
>
>In other words, problem occur when enter in block
>
> if(msg->msg_flags&FL_FORCE_LOCAL_RPORT) {
>   ...
>   extra_params.s = (char*)pkg_malloc(extra_params.len+1); /*line 1340*/
>   ...
> }
>
>
>Full debug of this scenario in
>http://vorigon.com/mhack/openser/20060725-1438/
>
>Regards,
>
>Marcos.
>
>(I'm trying, I'm trying... ;)
>
>
>
>Bogdan-Andrei Iancu wrote:
>  
>
>>Hi Marcos,
>>
>>thanks for the debugging - I will try to take a look over it in the next
>>day.
>>
>>regards,
>>bogdan
>>
>>Marcos Hack wrote:
>>
>>    
>>
>>>Hi Bogdan,
>>>
>>>Sorry, my fault. I believed that function force_rport() was of module
>>>nathelper and I took off it of openser config.
>>>
>>>I came back nathelper module, commented out just force_rport() function
>>>and the problem don't occur.
>>>
>>>I turned on debug=9 and see the following ( complete log in [1] ):
>>>
>>>---
>>>
>>>6(5032) qm_malloc(0x8146fc0, 7) called from msg_translator.c:
>>>build_req_buf_from_sip_req(1340)
>>>6(5032) qm_malloc(0x8146fc0, 8) returns address 0x817ea48 frag.
>>>0x817ea30 (size=8) on 1 -th hit
>>>6(5032) build_req_from_req: extra param added: <;rport>
>>>
>>>---
>>>
>>>Frag 0x817ea30 allocated here are not freed anywhere.
>>>
>>>Well, I tried to find the error but after breaking everything I think
>>>that I need your help! ;)
>>>
>>>
>>>      
>>>
>
>  
>