Change in osmo-msc[master]: libmsc/mncc_sock.c: Add lchan_type_offset in queue_hello()

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 Whyte gerrit-no-reply at lists.osmocom.org
Fri Aug 10 14:10:21 UTC 2018


Keith Whyte has uploaded this change for review. ( https://gerrit.osmocom.org/10428


Change subject: libmsc/mncc_sock.c: Add lchan_type_offset in queue_hello()
......................................................................

libmsc/mncc_sock.c: Add lchan_type_offset in queue_hello()

lchan_type was removed from gsm_mncc and the hello message on initial import from legacy OpenBSC
in Change-Id: Id3705236350d5f69e447046b0a764bbabc3d493c

This patch follows on from Change-Id: Ia02373a36df7605507ee3de49173a9fd6547b726 which reintroduced
lchan_type to the gsm_mncc struct. This patch restores the lchan_type_offset to the hello protocol message

Without this patch, LCR will issue an error and disconnect from the MNCC socket.

Change-Id: I65312082fa5dc0721170f923840e992ef9481a63
Closes: OS#3461
---
M src/libmsc/mncc_sock.c
1 file changed, 1 insertion(+), 0 deletions(-)



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

diff --git a/src/libmsc/mncc_sock.c b/src/libmsc/mncc_sock.c
index 74ea9d2..57b4bd8 100644
--- a/src/libmsc/mncc_sock.c
+++ b/src/libmsc/mncc_sock.c
@@ -228,6 +228,7 @@
 	hello->called_offset = offsetof(struct gsm_mncc, called);
 	hello->signal_offset = offsetof(struct gsm_mncc, signal);
 	hello->emergency_offset = offsetof(struct gsm_mncc, emergency);
+	hello->lchan_type_offset = offsetof(struct gsm_mncc, lchan_type);
 
 	msgb_enqueue(&mncc->net->upqueue, msg);
 	mncc->conn_bfd.when |= BSC_FD_WRITE;

-- 
To view, visit https://gerrit.osmocom.org/10428
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: I65312082fa5dc0721170f923840e992ef9481a63
Gerrit-Change-Number: 10428
Gerrit-PatchSet: 1
Gerrit-Owner: Keith Whyte <keith at rhizomatica.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180810/45e1c108/attachment.htm>


More information about the gerrit-log mailing list