Change in ...osmo-sip-connector[master]: Fix bug with MO calls not connecting.

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

keith gerrit-no-reply at lists.osmocom.org
Tue Aug 6 15:57:51 UTC 2019


keith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-sip-connector/+/15068


Change subject: Fix bug with MO calls not connecting.
......................................................................

Fix bug with MO calls not connecting.

Fixes a bug I introduced in 5f73c2033b6b4e690f30292020d7361f48b5f2c2
where we would not call mncc_call_leg_connect() on receiving 200 from
SIP side, and therefore never send MNCC_SETUP_RSP to the MS

Change-Id: Ic7cc56c0d68a27eb1229c0c4aa1fa54d00b660b6
---
M src/sip.c
1 file changed, 5 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-sip-connector refs/changes/68/15068/1

diff --git a/src/sip.c b/src/sip.c
index b652b4b..4eb7689 100644
--- a/src/sip.c
+++ b/src/sip.c
@@ -332,11 +332,13 @@
 		if (status == 180 || status == 183)
 			call_progress(leg, sip, status);
 		else if (status == 200) {
-			struct sip_call_leg *leg = sip_find_leg(nh);
-			if (leg)
+			if (leg->state == SIP_CC_CONNECTED || leg->state == SIP_CC_HOLD) {
+				/* This 200 is a response to our re-INVITE on
+				 * a connected call. We just need to ACK it. */
 				nua_ack(leg->nua_handle, TAG_END());
-			else
+			} else {
 				call_connect(leg, sip);
+			}
 		}
 		else if (status >= 300) {
 			struct call_leg *other = call_leg_other(&leg->base);

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15068
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: Ic7cc56c0d68a27eb1229c0c4aa1fa54d00b660b6
Gerrit-Change-Number: 15068
Gerrit-PatchSet: 1
Gerrit-Owner: keith <keith at rhizomatica.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190806/dee40345/attachment.htm>


More information about the gerrit-log mailing list