mirror of
https://gitea.osmocom.org/cellular-infrastructure/osmo-mgw.git
synced 2025-10-23 08:12:01 +00:00
gsm_04_08.c: Don't set msg->lchan nor msg->dst
the BSC-side of the API behind gsm0808_submit_dtap() is doing this resolving again anyway. So let's avoid doing it twice, and avoid having more dependency of the MSC down into the lchan details. Conflicts: openbsc/src/libmsc/gsm_04_08.c Change-Id: I14254be68ee1a48e9f1ce968233414d86c6ba9d5
This commit is contained in:
committed by
Neels Hofmeyr
parent
a81fbf0091
commit
9c138f4785
@@ -126,27 +126,6 @@ static int gsm48_conn_sendmsg(struct msgb *msg, struct gsm_subscriber_connection
|
||||
* work that the caller no longer has to do */
|
||||
if (trans) {
|
||||
gh->proto_discr = trans->protocol | (trans->transaction_id << 4);
|
||||
msg->lchan = trans->conn->lchan;
|
||||
}
|
||||
|
||||
if (msg->lchan) {
|
||||
struct e1inp_sign_link *sign_link =
|
||||
msg->lchan->ts->trx->rsl_link;
|
||||
|
||||
msg->dst = sign_link;
|
||||
if (gsm48_hdr_pdisc(gh) == GSM48_PDISC_CC)
|
||||
DEBUGP(DCC, "(bts %d trx %d ts %d ti %02x) "
|
||||
"Sending '%s' to MS.\n",
|
||||
sign_link->trx->bts->nr,
|
||||
sign_link->trx->nr, msg->lchan->ts->nr,
|
||||
gh->proto_discr & 0xf0,
|
||||
gsm48_cc_msg_name(gh->msg_type));
|
||||
else
|
||||
DEBUGP(DCC, "(bts %d trx %d ts %d pd %02x) "
|
||||
"Sending 0x%02x to MS.\n",
|
||||
sign_link->trx->bts->nr,
|
||||
sign_link->trx->nr, msg->lchan->ts->nr,
|
||||
gh->proto_discr, gh->msg_type);
|
||||
}
|
||||
|
||||
return gsm0808_submit_dtap(conn, msg, 0, 0);
|
||||
@@ -266,8 +245,6 @@ int gsm0408_loc_upd_rej(struct gsm_subscriber_connection *conn, uint8_t cause)
|
||||
return -1;
|
||||
}
|
||||
|
||||
msg->lchan = conn->lchan;
|
||||
|
||||
LOGP(DMM, LOGL_INFO, "Subscriber %s: LOCATION UPDATING REJECT "
|
||||
"LAC=%u BTS=%u\n", vlr_subscr_name(conn->vsub),
|
||||
bts->location_area_code, bts->nr);
|
||||
@@ -284,8 +261,6 @@ static int gsm0408_loc_upd_acc(struct gsm_subscriber_connection *conn,
|
||||
struct gsm48_loc_area_id *lai;
|
||||
uint8_t *mid;
|
||||
|
||||
msg->lchan = conn->lchan;
|
||||
|
||||
gh = (struct gsm48_hdr *) msgb_put(msg, sizeof(*gh));
|
||||
gh->proto_discr = GSM48_PDISC_MM;
|
||||
gh->msg_type = GSM48_MT_MM_LOC_UPD_ACCEPT;
|
||||
@@ -330,8 +305,6 @@ static int mm_tx_identity_req(struct gsm_subscriber_connection *conn, uint8_t id
|
||||
struct msgb *msg = gsm48_msgb_alloc_name("GSM 04.08 ID REQ");
|
||||
struct gsm48_hdr *gh;
|
||||
|
||||
msg->lchan = conn->lchan;
|
||||
|
||||
gh = (struct gsm48_hdr *) msgb_put(msg, sizeof(*gh) + 1);
|
||||
gh->proto_discr = GSM48_PDISC_MM;
|
||||
gh->msg_type = GSM48_MT_MM_ID_REQ;
|
||||
@@ -508,8 +481,6 @@ int gsm48_tx_mm_info(struct gsm_subscriber_connection *conn)
|
||||
int tzunits;
|
||||
int dst = 0;
|
||||
|
||||
msg->lchan = conn->lchan;
|
||||
|
||||
gh = (struct gsm48_hdr *) msgb_put(msg, sizeof(*gh));
|
||||
gh->proto_discr = GSM48_PDISC_MM;
|
||||
gh->msg_type = GSM48_MT_MM_INFO;
|
||||
@@ -659,7 +630,6 @@ int gsm48_tx_mm_auth_req(struct gsm_subscriber_connection *conn, uint8_t *rand,
|
||||
if (autn)
|
||||
DEBUGP(DMM, " AUTH REQ (autn = %s)\n", osmo_hexdump_nospc(autn, 16));
|
||||
|
||||
msg->lchan = conn->lchan;
|
||||
gh->proto_discr = GSM48_PDISC_MM;
|
||||
gh->msg_type = GSM48_MT_MM_AUTH_REQ;
|
||||
|
||||
@@ -1206,8 +1176,6 @@ int gsm48_send_rr_app_info(struct gsm_subscriber_connection *conn, uint8_t apdu_
|
||||
struct msgb *msg = gsm48_msgb_alloc_name("GSM 04.08 APP INF");
|
||||
struct gsm48_hdr *gh;
|
||||
|
||||
msg->lchan = conn->lchan;
|
||||
|
||||
DEBUGP(DRR, "TX APPLICATION INFO id=0x%02x, len=%u\n",
|
||||
apdu_id, apdu_len);
|
||||
|
||||
@@ -1299,8 +1267,6 @@ static int gsm48_tx_simple(struct gsm_subscriber_connection *conn,
|
||||
struct msgb *msg = gsm48_msgb_alloc_name("GSM 04.08 TX SIMPLE");
|
||||
struct gsm48_hdr *gh = (struct gsm48_hdr *) msgb_put(msg, sizeof(*gh));
|
||||
|
||||
msg->lchan = conn->lchan;
|
||||
|
||||
gh->proto_discr = pdisc;
|
||||
gh->msg_type = msg_type;
|
||||
|
||||
|
Reference in New Issue
Block a user