[asterisk-commits] branch oej/subscribemwi r10652 - /team/oej/subscribemwi/channels/chan_sip.c

[asterisk-commits at lists.digium.com]

Author: oej
Date: Tue Feb 21 12:59:21 2006
New Revision: 10652

URL: http://svn.digium.com/view/asterisk?rev=10652&view=rev
Log:
Update

Modified:
    team/oej/subscribemwi/channels/chan_sip.c

Modified: team/oej/subscribemwi/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/team/oej/subscribemwi/channels/chan_sip.c?rev=10652&r1=10651&r2=10652&view=diff
==============================================================================
--- team/oej/subscribemwi/channels/chan_sip.c (original)
+++ team/oej/subscribemwi/channels/chan_sip.c Tue Feb 21 12:59:21 2006
@@ -10887,7 +10887,8 @@
  		} else if (!strcmp(event, "message-summary") && !strcmp(accept, "application/simple-message-summary")) {
 			/* Looks like they actually want a mailbox status 
 			  This version of Asterisk supports mailbox subscriptions
-
+			  The subscribed URI needs to exist in the dial plan
+			  In most devices, this is configurable to the voicemailmain extension you use
 			*/
 			if (!authpeer || ast_strlen_zero(authpeer->mailbox)) {
 				transmit_response(p, "404 Not found (no mailbox)", req);
@@ -10897,9 +10898,10 @@
 			}
 
  			p->subscribed = MWI_NOTIFICATION;
-			if (authpeer->mwipvt && authpeer->mwipvt != p)
+			if (authpeer->mwipvt && authpeer->mwipvt != p)	/* Destroy old PVT if this is a new one */
+				/* We only allow one subscription per peer */
 				sip_destroy(authpeer->mwipvt);
-			authpeer->mwipvt = p;	/* Link from peer to pvt */
+			authpeer->mwipvt = p;		/* Link from peer to pvt */
 			p->relatedpeer = authpeer;	/* Link from pvt to peer */
 		} else { /* At this point, Asterisk does not understand the specified event */
 			transmit_response(p, "489 Bad Event", req);