fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/35881?usp=email )


Change subject: msc_a_up_call_assignment_complete(): handle all CSD modes
......................................................................

msc_a_up_call_assignment_complete(): handle all CSD modes

Change-Id: I92f7ab11ebc2e53700b27c38e91679c2c6f72050
---
M src/libmsc/msc_a.c
1 file changed, 17 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/81/35881/1

diff --git a/src/libmsc/msc_a.c b/src/libmsc/msc_a.c
index a933bd2..8751819 100644
--- a/src/libmsc/msc_a.c
+++ b/src/libmsc/msc_a.c
@@ -1485,13 +1485,18 @@
                        return;
                }

-               /* Check for unexpected codec with CSD */
-               if (cc_trans->bearer_cap.transfer == 
GSM48_BCAP_ITCAP_UNR_DIG_INF &&
-                   codec_if_known->type != GSM0808_SCT_CSD) {
+               switch (cc_trans->bearer_cap.transfer) {
+               case GSM48_BCAP_ITCAP_FAX_G3:
+               case GSM48_BCAP_ITCAP_3k1_AUDIO:
+               case GSM48_BCAP_ITCAP_UNR_DIG_INF:
+                       if (codec_if_known->type == GSM0808_SCT_CSD)
+                               break; /* we're good */
                        LOG_TRANS(cc_trans, LOGL_ERROR, "Unexpected codec in 
Assignment Complete for CSD: %s\n",
                                  
gsm0808_speech_codec_type_name(codec_if_known->type));
                        call_leg_release(msc_a->cc.call_leg);
                        return;
+               default:
+                       break;
                }

                /* Update RAN-side endpoint CI from Assignment result -- unless 
it is forced by the ran_infra, in which

--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/35881?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I92f7ab11ebc2e53700b27c38e91679c2c6f72050
Gerrit-Change-Number: 35881
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-MessageType: newchange

Reply via email to