Add todo comment about handle_incoming() calling assumption.
authorRichard Mudgett <rmudgett@digium.com>
Mon, 11 Oct 2010 19:07:59 +0000 (19:07 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Mon, 11 Oct 2010 19:07:59 +0000 (19:07 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@291115 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_sip.c

index 79408c0..26f0a38 100644 (file)
@@ -23799,6 +23799,7 @@ static void process_request_queue(struct sip_pvt *p, int *recount, int *nounlock
        struct sip_request *req;
 
        while ((req = AST_LIST_REMOVE_HEAD(&p->request_queue, next))) {
+               /*! \todo XXX if nounlock is nonzero we do not have the channel lock anymore.  handle_incoming() assumes that it is locked. */
                if (handle_incoming(p, req, &p->recv, recount, nounlock) == -1) {
                        /* Request failed */
                        ast_debug(1, "SIP message could not be handled, bad request: %-70.70s\n", p->callid[0] ? p->callid : "<no callid>");
@@ -24024,6 +24025,7 @@ static int handle_request_do(struct sip_request *req, struct ast_sockaddr *addr)
                process_request_queue(p, &recount, &nounlock);
        }
 
+       /*! \todo XXX if nounlock is nonzero we do not have the channel lock anymore.  handle_incoming() assumes that it is locked. */
        if (handle_incoming(p, req, addr, &recount, &nounlock) == -1) {
                /* Request failed */
                ast_debug(1, "SIP message could not be handled, bad request: %-70.70s\n", p->callid[0] ? p->callid : "<no callid>");