osmo-bts.git branch master updated. 0.7.0-22-g5b70bb6

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/osmocom-commitlog@lists.osmocom.org/.

gitosis at osmocom.org gitosis at osmocom.org
Tue Dec 5 14:55:15 UTC 2017


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  5b70bb673d5857cc2208712d3bbab29fd650cb09 (commit)
      from  70c4dc8d706973c65e6bf088156f5862f4e036b4 (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=5b70bb673d5857cc2208712d3bbab29fd650cb09

commit 5b70bb673d5857cc2208712d3bbab29fd650cb09
Author: Harald Welte <laforge at gnumonks.org>
Date:   Tue Oct 31 15:29:35 2017 -0400

    DTX: avoid illegal character contained in DTX FSM allocation which causes BTS crash
    
    Problem:
    
     lchan->tch.dtx.dl_amr_fsm struct failed to allocate in l1sap_chan_act routine
     in l1sap.c due to illegal characters contained in lchan->name which are passed to
     osmo_fsm_inst_alloc routine. As a result, lchan->tch.dtx.dl_amr_fsm is NULL
     causing BTS crashed (SEG FAULT) when trying to access this struct.
    
     Below is snapshot of crash log obtained by GDB:
     ...
     Fri Nov 24 18:13:55 2017 <0000> rsl.c:1653 payload type: 98
     Fri Nov 24 18:13:55 2017 <0000> rsl.c:1463 (bts=0,trx=0,ts=2,ss=0)
     RSL Tx IPAC_MDCX_ACK (local 127.0.0.1:11538, remote 127.0.0.1:30012)
     Program received signal SIGSEGV, Segmentation fault.
     0x00031930 in dtx_dl_amr_fsm_step (lchan=lchan at entry=0xb69592a8,
     rtp_pl=rtp_pl at entry=0x87ae8 " \024\351Y\363_\337\345\351f\177\373\300\210\201\200\210",
        rtp_pl_len=17, fn=1728481, l1_payload=0x10dd25 "", marker=marker at entry=true,
        len=len at entry=0x10ddc4 "\024", ft_out=0xbefff7d7 "\002",
        ft_out at entry=0xbefff7cf "\276\341_\032") at msg_utils.c:233
     233     msg_utils.c: No such file or directory.
     ...
    
    Fix:
    * Use different formatting for lchan name passed to osmo_fsm_inst_alloc routine
    * Refuse channel activation if FSM could not be generated (as opposed to crash)
    
    Related: OS#2606
    Reported-by: Minh-Quang Nguyen <minh-quang.nguyen at nutaq.com>
    Change-Id: I929ce3703dc57acf8db569ae0e346265644d0b3c

-----------------------------------------------------------------------

Summary of changes:
 src/common/l1sap.c | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Osmocom BTS-side code (Abis, scheduling, ...)



More information about the osmocom-commitlog mailing list