9 Mar
2018
9 Mar
'18
1:47 p.m.
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Osmocom BTS-side code (Abis, scheduling, ...)".
The branch, master has been updated
via c2045474b1b500a6cc38c00cc807978ce9e8c5e5 (commit)
from 7eed026efc0b75608b37521147db140c06fca5b1 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
http://cgit.osmocom.org/osmo-bts/commit/?id=c2045474b1b500a6cc38c00cc807978…
commit c2045474b1b500a6cc38c00cc807978ce9e8c5e5
Author: Neels Hofmeyr <neels(a)hofmeyr.de>
Date: Thu Mar 8 20:04:42 2018 +0100
fix handover: handle_ph_ra_ind(): evaluate ra_ind before msgb_trim()
Commit c2b4c668f3510b7b0baace749c5a310959010e90
I3b989580cb38082e3fd8fc50a11fedda13991092 introduces evaluation of ra_ind
members below the msgb_trim() call that actually invalidates ra_ind.
A symptom is that it breaks detection of Handover RACH, wich always ends up
with lchan == NULL and interpreting all RACH as chan_nr == 0x88.
Fix: do all evaluation of ra_ind before the msgb_trim(), for osmo-bts-sysmo,
litecell-15 and octphy.
To guard against similar mistakes in the future, set ra_ind = NULL before the
msgb_trim() call.
Related: OS#3045
Change-Id: I203021ee57f49cb963679ba8bec5943e2abb67fb
-----------------------------------------------------------------------
Summary of changes:
src/osmo-bts-litecell15/l1_if.c | 83 +++++++++++++++++++++------------------
src/osmo-bts-octphy/l1_if.c | 46 +++++++++++++---------
src/osmo-bts-sysmo/l1_if.c | 86 ++++++++++++++++++++++-------------------
3 files changed, 118 insertions(+), 97 deletions(-)
hooks/post-receive
--
Osmocom BTS-side code (Abis, scheduling, ...)