<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/4.2.3">
</HEAD>
<BODY>
Hi,<BR>
<BR>
I just tried this updated presence module and ran into some problems.<BR>
<BR>
The scenario I use is presence and RLS.&nbsp; The client SUBSCRIBEs to a single resource list on the RLS and to presence.winfo on the PS.&nbsp; The client PUBLISHes to the PS.<BR>
<BR>
With the new code I get two error messages out a lot:<BR>
<BLOCKQUOTE>
    Feb 29 11:31:18 pd-laptop-linux ./kamailio[22566]: ERROR: presence [subscribe.c:799]: wrong status<BR>
    Feb 29 11:32:39 pd-laptop-linux ./kamailio[22566]: ERROR: presence [subscribe.c:1314]: wrong sequence number received: 4 - stored: 10<BR>
</BLOCKQUOTE>
<BR>
I also get segmentation faults when I change presence state or log out.&nbsp; Here is a fragment of the back-trace:<BR>
<BLOCKQUOTE>
    #0&nbsp; 0x00007f0447191438 in get_subs_db (pres_uri=0x7fff0ce09ea0, event=0x7f0443f15728, sender=&lt;optimized out&gt;, s_array=0x7fff0ce09bf0, n=0x7fff0ce09c00) at notify.c:1120<BR>
    #1&nbsp; 0x00007f044719375f in get_subs_dialog (pres_uri=0x7fff0ce09ea0, event=0x7f0443f15728, sender=0x0) at notify.c:1238<BR>
    #2&nbsp; 0x00007f04471994ce in query_db_notify (pres_uri=0x7fff0ce09ea0, event=0x7f0443f15728, watcher_subs=0x0) at notify.c:1353<BR>
    #3&nbsp; 0x00007f04471b024e in update_subscription (msg=0x7f044d36ec38, subs=0x7fff0ce09ea0, to_tag_gen=&lt;optimized out&gt;, sent_reply=0x7fff0ce09ff0) at subscribe.c:452<BR>
    #4&nbsp; 0x00007f04471be82a in handle_subscribe (msg=0x7f044d36ec38, str1=&lt;optimized out&gt;, str2=&lt;optimized out&gt;) at subscribe.c:805<BR>
    #5&nbsp; 0x00000000004e50c5 in do_action (h=0x7fff0ce0c950, a=&lt;optimized out&gt;, msg=0x7f044d36ec38) at action.c:1116<BR>
</BLOCKQUOTE>
I rolled back to the previous version of presence and it all works correctly as before.<BR>
<BR>
Thanks,<BR>
<BR>
Peter<BR>
<BR>
On Wed, 2012-02-15 at 13:45 +0100, Anca Vamanu wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
Module: sip-router
Branch: master
Commit: ae86ca3611398ce365ac4a1776ff0c7e95476bbe
URL:    <A HREF="http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ae86ca3611398ce365ac4a1776ff0c7e95476bbe">http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ae86ca3611398ce365ac4a1776ff0c7e95476bbe</A>

Author: Anca Vamanu &lt;<A HREF="mailto:anca.vamanu@1and1.ro">anca.vamanu@1and1.ro</A>&gt;
Committer: Anca Vamanu &lt;<A HREF="mailto:anca.vamanu@1and1.ro">anca.vamanu@1and1.ro</A>&gt;
Date:   Wed Feb 15 13:39:55 2012 +0200

modules_k/presence Fixed DB Storage Modes

- removed db_mode and fallback2db parameters and added two new
  parameters: subs_db_mode and publ_cache
- fixed and extended the storage modes for subscriptions: Memory Only,
  Write Through, Write Back, DB Only
- publ_cache parameter offers the possibility to disable publish cache
- some other fixes:
    - delete subscription only for 481 or 408 reply for Notify
    - call child_init also for main process (no shutdown DB flush was
      being performed)

---

 modules_k/presence/README                 |  190 ++++++-----
 modules_k/presence/bind_presence.c        |    4 +-
 modules_k/presence/bind_presence.h        |    4 +-
 modules_k/presence/doc/presence_admin.xml |  127 +++++---
 modules_k/presence/doc/presence_devel.xml |    2 +-
 modules_k/presence/event_list.c           |    2 +-
 modules_k/presence/event_list.h           |    2 +-
 modules_k/presence/hash.c                 |   25 +--
 modules_k/presence/hash.h                 |    2 +-
 modules_k/presence/notify.c               |  253 ++++++---------
 modules_k/presence/notify.h               |    2 +-
 modules_k/presence/presence.c             |  108 +++----
 modules_k/presence/presence.h             |   19 +-
 modules_k/presence/presentity.c           |   30 +--
 modules_k/presence/presentity.h           |    2 +-
 modules_k/presence/publish.c              |   52 ++--
 modules_k/presence/publish.h              |    2 +-
 modules_k/presence/subscribe.c            |  495 +++++++++++++++++++----------
 modules_k/presence/subscribe.h            |    7 +-
 modules_k/presence/utils_func.c           |    2 +-
 modules_k/presence/utils_func.h           |    2 +-
 modules_k/pua/hash.h                      |    1 +
 22 files changed, 736 insertions(+), 597 deletions(-)

Diff:   <A HREF="http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=ae86ca3611398ce365ac4a1776ff0c7e95476bbe">http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=ae86ca3611398ce365ac4a1776ff0c7e95476bbe</A>

_______________________________________________
sr-dev mailing list
<A HREF="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</A>
<A HREF="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</A>
</PRE>
</BLOCKQUOTE>
<BR>
<TABLE CELLSPACING="0" CELLPADDING="0" WIDTH="100%">
<TR>
<TD>
<PRE>
-- 
Peter Dunkley
Technical Director
Crocodile RCS Ltd
</PRE>
</TD>
</TR>
</TABLE>
</BODY>
</HTML>