[Kamailio-Devel] presentity.c patch breaks event matching

Juha Heinanen jh at tutpro.com
Thu Sep 11 18:22:56 CEST 2008


klaus,

i noticed that with very new svn version of kamailio, event matching is
broken.  it appears like this:

Sep 11 19:05:01 lohi /usr/sbin/kamailio[985]: INFO: Handling PUBLISH to <sip:test at vm.tutpro.com> 
Sep 11 19:05:01 lohi /usr/sbin/kamailio[985]: ERROR:presence:check_if_dialog: failed to parse xml document 
Sep 11 19:05:01 lohi /usr/sbin/kamailio[985]: ERROR:presence:update_presentity: failed to check if dialog stored 
Sep 11 19:05:01 lohi /usr/sbin/kamailio[985]: ERROR:presence:handle_publish: when updating presentity 

of course xml parsing of message-summary body fails, because it is not
an xml document.

check_if_dialog() is called by presentity.c/update_presentity() only if  

	if(presentity->event->evp->parsed == EVENT_DIALOG)

evaluates to true.  the above test must not be true if event is
MSGSUM_EVENT.  the test used to be

	if(presentity->event->evp->parsed & EVENT_DIALOG)

so obviously something got broken by your change.  i have not yet had
time to look more closely what went wrong.

-- juha



More information about the Devel mailing list