[asterisk-commits] trunk r9105 - /trunk/channels/chan_sip.c

[asterisk-commits at lists.digium.com]

Author: oej
Date: Thu Feb  2 14:28:57 2006
New Revision: 9105

URL: http://svn.digium.com/view/asterisk?rev=9105&view=rev
Log:
- Add doxygen docs on return values on check_user_full()
- Simplify code in handle_request_subscribe (mailbox subscription)

Modified:
    trunk/channels/chan_sip.c

Modified: trunk/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_sip.c?rev=9105&r1=9104&r2=9105&view=diff
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Thu Feb  2 14:28:57 2006
@@ -2259,6 +2259,10 @@
  * is *two* devices in Asterisk, not one.
  *
  * Thought: For realtime, we should propably update storage with inuse counter... 
+ *
+ * \return 0 if call is ok (no call limit, below treshold)
+ *	-1 on rejection of call
+ *		
  */
 static int update_call_counter(struct sip_pvt *fup, int event)
 {
@@ -10801,7 +10805,6 @@
 
 	if (!p->lastinvite) {
 		char mailboxbuf[256]="";
-		int found = 0;
 		char *mailbox = NULL;
 		int mailboxsize = 0;
 
@@ -10877,10 +10880,6 @@
 				
 				*/
 				if (!ast_strlen_zero(mailbox)) {
-					found++;
-				}
-
-				if (found){
 					transmit_response(p, "200 OK", req);
 					ast_set_flag(p, SIP_NEEDDESTROY);	
 				} else {