Hi Anca,

I just tried this change on my system.  With the change present and two presence/RLS clients signed in I get some error messages displayed when I log one of the clients out:
Mar 13 15:36:09 pd-laptop-linux ./kamailio[1493]: ERROR: presence_xml [notify_body.c:176]: while parsing the xml body message
Mar 13 15:36:09 pd-laptop-linux ./kamailio[1493]: ERROR: presence_xml [notify_body.c:133]: in function get_final_notify_body
Mar 13 15:36:09 pd-laptop-linux ./kamailio[1493]: ERROR: presence [notify.c:1396]: in function apply_auth_nbody
Mar 13 15:36:09 pd-laptop-linux ./kamailio[1493]: ERROR: presence [notify.c:1583]: sending Notify not successful
Mar 13 15:36:09 pd-laptop-linux ./kamailio[1493]: ERROR: presence [notify.c:1288]: Could not send notify for presence
I reversed out the change and the errors stopped happening, they started up again as soon as I put it back.

Thanks,

Peter

On Tue, 2012-03-13 at 10:40 +0100, Anca Vamanu wrote: 
Module: sip-router
Branch: master
Commit: 42f2d990d022cb67b051ac563ee8cde916a0d14a
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=42f2d990d022cb67b051ac563ee8cde916a0d14a

Author: Dragos Dinu <dragos.dinu@1and1.ro>
Committer: Anca Vamanu <anca.vamanu@1and1.ro>
Date:   Tue Mar 13 11:33:06 2012 +0200

modules_k/presence Used affected_rows SQL statement

In case affected_rows function is available in the DB driver, this
function is used instead of an extra query in presentity table that only
verified if the record existed.

Patch author Dragos Dinu <dragos.dinu@1and1.ro>

---

 modules_k/presence/presentity.c |  436 ++++++++++++++++++++++----------------
 1 files changed, 252 insertions(+), 184 deletions(-)

Diff:   http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=42f2d990d022cb67b051ac563ee8cde916a0d14a

_______________________________________________
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev


-- 
Peter Dunkley
Technical Director
Crocodile RCS Ltd