diff options
author | oej <oej@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-08 10:02:12 +0000 |
---|---|---|
committer | oej <oej@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-08 10:02:12 +0000 |
commit | 456644d619a1eddbb4504541d0161ce518541057 (patch) | |
tree | b75322532172bd4c45c0f9a06ef02372b8df72da /channels/chan_sip.c | |
parent | 0d63a42dce1f160ef6b1120ab5e5213b75c826b7 (diff) |
Merged revisions 128950 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r128950 | oej | 2008-07-08 11:52:21 +0200 (Tis, 08 Jul 2008) | 11 lines
Don't hangup the call if we can't resolve the Contact if there's a proxy
route set for the call.
----
This comment was added a while ago and today it hit me badly.
/* OEJ: Possible issue that may need a check:
If we have a proxy route between us and the device,
should we care about resolving the contact
or should we just send it?
*/
........
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@128951 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels/chan_sip.c')
-rw-r--r-- | channels/chan_sip.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index df545f834..4f71133d2 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -15182,21 +15182,17 @@ static void handle_response_invite(struct sip_pvt *p, int resp, char *rest, stru if (outgoing) { update_call_counter(p, DEC_CALL_RINGING); parse_ok_contact(p, req); + /* Save Record-Route for any later requests we make on this dialogue */ + if (!reinvite) + build_route(p, req, 1); + if(set_address_from_contact(p)) { /* Bad contact - we don't know how to reach this device */ /* We need to ACK, but then send a bye */ - /* OEJ: Possible issue that may need a check: - If we have a proxy route between us and the device, - should we care about resolving the contact - or should we just send it? - */ - if (!req->ignore) + if (!p->route && !req->ignore) ast_set_flag(&p->flags[0], SIP_PENDINGBYE); } - /* Save Record-Route for any later requests we make on this dialogue */ - if (!reinvite) - build_route(p, req, 1); } if (p->owner && (p->owner->_state == AST_STATE_UP) && (bridgepeer = ast_bridged_channel(p->owner))) { /* if this is a re-invite */ |