Neels Hofmeyr has uploaded this change for review. (
https://gerrit.osmocom.org/11700
Change subject: always directly send BSSMAP Clear Request
......................................................................
always directly send BSSMAP Clear Request
When a gscon wants to send a BSSMAP Clear Request, it makes no sense to do it
conditionally depending on the current conn state. Just send it: don't call
gscon_sigtran_send(), directly go for osmo_bsc_sigtran_send().
In particular, if an incoming inter-BSC handover ends in failure, the gscon
state is still ST_INIT, but if the MSC fails to give us a Clear Command, we may
want to ask with a BSSMAP Clear Request.
Change-Id: I39fae24260a4bb7a6af704ebe760f93fff566536
---
M src/osmo-bsc/bsc_subscr_conn_fsm.c
1 file changed, 8 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/00/11700/1
diff --git a/src/osmo-bsc/bsc_subscr_conn_fsm.c
b/src/osmo-bsc/bsc_subscr_conn_fsm.c
index bca9e1f..2202ecf 100644
--- a/src/osmo-bsc/bsc_subscr_conn_fsm.c
+++ b/src/osmo-bsc/bsc_subscr_conn_fsm.c
@@ -134,7 +134,14 @@
enum gsm0808_cause cause)
{
struct msgb *resp = gsm0808_create_clear_rqst(cause);
- gscon_sigtran_send(conn, resp);
+ int rc;
+ if (!resp) {
+ LOGPFSML(conn->fi, LOGL_ERROR, "Unable to compose BSSMAP Clear
Request message\n");
+ return;
+ }
+ rc = osmo_bsc_sigtran_send(conn, resp);
+ if (rc < 0)
+ LOGPFSML(conn->fi, LOGL_ERROR, "Unable to deliver BSSMAP Clear
Request message\n");
}
/* forward MO DTAP from RSL side to BSSAP side */
--
To view, visit https://gerrit.osmocom.org/11700
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I39fae24260a4bb7a6af704ebe760f93fff566536
Gerrit-Change-Number: 11700
Gerrit-PatchSet: 1
Gerrit-Owner: Neels Hofmeyr <[email protected]>
