Bugs item #2798154, was opened at 2009-05-29 01:30
Message generated for change (Comment added) made by miconda
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2798154&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: None
Status: Open
Resolution: None
Priority: 4
Private: No
Submitted By: Iñaki Baz Castillo (ibc_sf)
Assigned to: Nobody/Anonymous (nobody)
Summary: [dialog] Crash when receiving empry "Record-Route"
Initial Comment:
Kamailio 1.5 rev 5834.
I use 'dialog" module and "uac" module (this one to change From username).
Kamailio receives an INVITE, changes From (so addes a long "vsf" parameter to Record-Route) and forwards it to a gateway.
The gateway replies 183 and 200 containing a "Record-Route" with no value (empty).
Then Kamailio shows this error and crashes:
ERROR:core:print_rr_body: failed to parse RR
ERROR:dialog:populate_leg_info: failed to print route records
ERROR:dialog:dlg_onreply: could not add further info to the dialog
CRITICAL:core:receive_fd: EOF on 12
INFO:core:handle_sigs: child process 4505 exited by a signal 11
INFO:core:handle_sigs: core was not generated
INFO:core:handle_sigs: terminating due to SIGCHLD
INFO:core:sig_usr: signal 15 received
I attach the output of the coredump and also the SIP trace showing it.
----------------------------------------------------------------------
>Comment By: Daniel-Constantin Mierla (miconda)
Date: 2009-06-01 12:33
Message:
I did a fix for this particular case, can you test and see if is fine now?
There is more code to review, though...
----------------------------------------------------------------------
Comment By: Iñaki Baz Castillo (ibc_sf)
Date: 2009-05-29 09:28
Message:
Note that the crash also occur with:
modparam("dialog", "db_mode", 0)
This is: without storing dialogs in database.
----------------------------------------------------------------------
Comment By: Daniel-Constantin Mierla (miconda)
Date: 2009-05-29 09:20
Message:
Seems similar to what Alex Hermann reported in #2797928. I will check.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2798154&group_…
Revision: 5864
http://openser.svn.sourceforge.net/openser/?rev=5864&view=rev
Author: miconda
Date: 2009-06-01 09:31:18 +0000 (Mon, 01 Jun 2009)
Log Message:
-----------
- init local variables and stafety checks to avoid crash when headers are missing (reported by Inaki Baz Castillo, #2798154)
Modified Paths:
--------------
branches/1.5/modules/dialog/dlg_handlers.c
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.