Change in osmocom-bb[master]: mobile: fix gsm_recv_voice: pull l1ctl header from TCH payload

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/.

laforge gerrit-no-reply at lists.osmocom.org
Fri Mar 6 21:51:34 UTC 2020


laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/17379 )

Change subject: mobile: fix gsm_recv_voice: pull l1ctl header from TCH payload
......................................................................

mobile: fix gsm_recv_voice: pull l1ctl header from TCH payload

That function encapsulates the RTP payload in an MNCC header, but the l1ctl dl
header has to be removed first to get only the RTP payload in the MNCC
structure.

Change-Id: Id6ddc9b1da43e88c5b9468d4397a39953bdf533a
---
M src/host/layer23/src/mobile/voice.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved
  laforge: Looks good to me, but someone else must approve



diff --git a/src/host/layer23/src/mobile/voice.c b/src/host/layer23/src/mobile/voice.c
index b767833..5106063 100644
--- a/src/host/layer23/src/mobile/voice.c
+++ b/src/host/layer23/src/mobile/voice.c
@@ -38,6 +38,8 @@
 
 	/* distribute and then free */
 	if (ms->mncc_entity.mncc_recv && ms->mncc_entity.ref) {
+		/* Drop the l1ctl_info_dl header */
+		msgb_pull_to_l2(msg);
 		/* push mncc header in front of data */
 		mncc = (struct gsm_data_frame *)
 			msgb_push(msg, sizeof(struct gsm_data_frame));

-- 
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/17379
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Id6ddc9b1da43e88c5b9468d4397a39953bdf533a
Gerrit-Change-Number: 17379
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200306/565602a2/attachment.htm>


More information about the gerrit-log mailing list