Fix ACK routing for non-2xx responses.

When we send an ACK for a 2xx response to an INVITE, we are supposed
to use the learned route set. However, when we receive a non-2xx final
response to an INVITE, we are supposed to send the ACK to the same place
we initially sent the INVITE.

We had been doing this up until the changes went in that would build a route
set from provisional responses. That introduced a regression where we would
use the learned route set under all circumstances.

With this change, we now will set the destination of our ACK based on the
invitestate. If it is INV_COMPLETED then that means that we have received
a non-2xx final response (INV_TERMINATED indicates a 2xx response was received).
If it is INV_CANCELLED, then that means the call is being canceled, which
means that we should be ACKing a 487 response.

The other change introduced here is setting the invitestate to INV_CONFIRMED
when we send an ACK *after* the reqprep instead of before. This way, we can
tell in reqprep more easily what the invitestate is prior to sending the ACK.

(closes issue ASTERISK-19389)
reported by Karsten Wemheuer
patches:
    ASTERISK-19389v2.patch uploaded by Mark Michelson (license #5049)
	(with some slight modifications prior to commit)

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

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 790a46c6e31302a89f97834e72e59f437a7e2c54..7e2d255aa11dc4677091db9aec5f28e63420d46c 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -10354,10 +10354,14 @@ static int reqprep(struct sip_request *req, struct sip_pvt *p, int sipmethod, ui
 
        add_header(req, "Via", p->via);
        /*
-        * Use the learned route set unless this is a CANCEL. For a CANCEL
-        * we have to send to the same destination as the original INVITE.
+        * Use the learned route set unless this is a CANCEL on an ACK for a non-2xx
+        * final response. For a CANCEL or ACK, we have to send to the same destination
+        * as the original INVITE.
         */
-       if (p->route && sipmethod != SIP_CANCEL) {
+       if (sipmethod == SIP_CANCEL ||
+                       (sipmethod == SIP_ACK && (p->invitestate == INV_COMPLETED || p->invitestate == INV_CANCELLED))) {
+               set_destination(p, ast_strdupa(p->uri));
+       } else if (p->route) {
                set_destination(p, p->route->hop);
                add_route(req, is_strict ? p->route->next : p->route);
        }
@@ -13584,15 +13588,15 @@ static int transmit_request(struct sip_pvt *p, int sipmethod, uint32_t seqno, en
 {
        struct sip_request resp;
        
-       if (sipmethod == SIP_ACK) {
-               p->invitestate = INV_CONFIRMED;
-       }
-
        reqprep(&resp, p, sipmethod, seqno, newbranch);
        if (sipmethod == SIP_CANCEL && p->answered_elsewhere) {
                add_header(&resp, "Reason", "SIP;cause=200;text=\"Call completed elsewhere\"");
        }
 
+       if (sipmethod == SIP_ACK) {
+               p->invitestate = INV_CONFIRMED;
+       }
+
        return send_request(p, &resp, reliable, seqno ? seqno : p->ocseq);
 }