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! ;)





_______________________________________________
Devel mailing list
[email protected]
http://openser.org/cgi-bin/mailman/listinfo/devel

Reply via email to