<p>fixeria <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18907">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">assignment_fsm: fix assignment_count(): conn->lchan may be NULL<br><br>Starting from ttcn3-bsc-test-sccplite build #777, it was noticed<br>that osmo-bsc crashes with the following message:<br><br>  Assert failed conn->lchan include/osmocom/bsc/gsm_data.h:1376<br><br>Here is a backtrace:<br><br>  Program received signal SIGABRT<br>  pwndbg> bt<br>    0x0000555555be6e52 in conn_get_bts (conn=0x622000057160) at include/osmocom/bsc/gsm_data.h:1376<br>    0x0000555555c1edc8 in assignment_fsm_timer_cb (fi=0x612000060220) at assignment_fsm.c:758<br>    0x00007ffff72b1104 in fsm_tmr_cb (data=0x612000060220) at libosmocore/src/fsm.c:325<br>    0x00007ffff72ab062 in osmo_timers_update () at libosmocore/src/timer.c:257<br>    0x00007ffff72ab5d2 in _osmo_select_main (polling=0) at libosmocore/src/select.c:260<br>    0x00007ffff72abd2f in osmo_select_main_ctx (polling=<optimized out>) at libosmocore/src/select.c:291<br>    0x0000555555e1b81b in main (argc=3, argv=0x7fffffffe1b8) at osmo_bsc_main.c:953<br>    0x00007ffff6752002 in __libc_start_main () from /usr/lib/libc.so.6<br>    0x0000555555b61bbe in _start ()<br><br>Apparently conn->lchan can be NULL, so assignment_count() must be<br>safe against this. Let's add a guard 'if' statement there.<br><br>Yes, we miss a chance to increase a counter in such case, but this<br>is a separate problem. The main point is that osmo-bsc must not crash.<br><br>Change-Id: Id681dfb0ad654bdb4b71805d1ad4f39a8bf6bbd1<br>Fixes: I0009e51d4caf68e762138d98e2e23d49acc3cc1a<br>Related: OS#4620, OS#4619<br>---<br>M src/osmo-bsc/assignment_fsm.c<br>1 file changed, 8 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/07/18907/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18907">change 18907</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bsc/+/18907"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Id681dfb0ad654bdb4b71805d1ad4f39a8bf6bbd1 </div>
<div style="display:none"> Gerrit-Change-Number: 18907 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>