Vadim Yanitskiy has uploaded this change for review. (
https://gerrit.osmocom.org/12043
Change subject: libmsc/gsm_04_11.c: fix: always use SAPI 3 for MT SMS
......................................................................
libmsc/gsm_04_11.c: fix: always use SAPI 3 for MT SMS
In I4a07ece80d8dd40b23da6bb1ffc9d3d745b54092 I've introduced a
regression. According to GSM TS 04.11, section 2.3, SAPI 3
shall be used for both MO/MT SMS transmissions.
Change-Id: I08d0801a89d377441e95fb8e3dd27c8d587f89e9
---
M src/libmsc/gsm_04_11.c
1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/43/12043/1
diff --git a/src/libmsc/gsm_04_11.c b/src/libmsc/gsm_04_11.c
index 88dc433..b602344 100644
--- a/src/libmsc/gsm_04_11.c
+++ b/src/libmsc/gsm_04_11.c
@@ -149,8 +149,6 @@
case GSM_PAGING_SUCCEEDED:
/* Associate transaction with established connection */
trans->conn = msc_subscr_conn_get(conn, MSC_CONN_USE_TRANS_SMS);
- /* FIXME: specify SACCH in case we already have active TCH */
- trans->dlci = 0x03;
/* Confirm successful connection establishment */
gsm411_smc_recv(&trans->sms.smc_inst,
GSM411_MMSMS_EST_CNF, NULL, 0);
@@ -1017,6 +1015,9 @@
trans->sms.sm_rp_mr = conn->next_rp_ref++;
}
+ /* Use SAPI 3 (see GSM 04.11, section 2.3) */
+ trans->dlci = UM_SAPI_SMS;
+
return trans;
}
--
To view, visit https://gerrit.osmocom.org/12043
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I08d0801a89d377441e95fb8e3dd27c8d587f89e9
Gerrit-Change-Number: 12043
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <[email protected]>
