- Add doxygen docs on return values on check_user_full()

- Simplify code in handle_request_subscribe (mailbox subscription)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@9105 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 6c6abb6..df59ded 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -2259,6 +2259,10 @@ static void __sip_destroy(struct sip_pvt *p, int lockowner)
  * 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 @@ static int handle_request_subscribe(struct sip_pvt *p, struct sip_request *req,
 
        if (!p->lastinvite) {
                char mailboxbuf[256]="";
-               int found = 0;
                char *mailbox = NULL;
                int mailboxsize = 0;
 
@@ -10877,10 +10880,6 @@ static int handle_request_subscribe(struct sip_pvt *p, struct sip_request *req,
                                
                                */
                                if (!ast_strlen_zero(mailbox)) {
-                                       found++;
-                               }
-
-                               if (found){
                                        transmit_response(p, "200 OK", req);
                                        ast_set_flag(p, SIP_NEEDDESTROY);       
                                } else {