<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/9740">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">check for osmo-msc crash upon overlong IMSI<br><br>Overlong IMSIs used to trigger an assertion failure in osmo-msc.<br>This problem has been fixed but there was no test for it yet.<br><br>A lazy way of testing for this problem is to send an overlong IMSI<br>from an existing test which already verifies related behaviour<br>and would fail if the MSC crashed: TC_lu_by_tmsi_noauth_unknown<br><br>However, osmo-msc currently accepts overlong IMSIs and silently<br>truncates them, so this change as-is currently breaks this test.<br>But I would argue that osmo-msc's current behaviour is unreasonable<br>anyway and have proposed a patch to change it:<br>https://gerrit.osmocom.org/#/c/osmo-msc/+/9739/<br>With that patch applied to osmo-msc, this test keeps passing.<br><br>Change-Id: I2c472bee76086f6c84ec684d2e58b3351ebc3147<br>Depends: I785c994f41a646d8d83d3d82f5a9ae6b572eb641<br>Related: OS#2864<br>Related: g#9739<br>---<br>M msc/MSC_Tests.ttcn<br>1 file changed, 3 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/msc/MSC_Tests.ttcn b/msc/MSC_Tests.ttcn</span><br><span>index dc5303e..dc3ad1e 100644</span><br><span>--- a/msc/MSC_Tests.ttcn</span><br><span>+++ b/msc/MSC_Tests.ttcn</span><br><span>@@ -807,6 +807,8 @@</span><br><span> </span><br><span> /* Test LU by TMSI with unknown TMSI, expect (and answer) ID REQ. */</span><br><span> private function f_tc_lu_tmsi_noauth_unknown(charstring id, BSC_ConnHdlrPars pars) runs on BSC_ConnHdlr {</span><br><span style="color: hsl(120, 100%, 40%);">+   /* We piggyback a test for an MSC crash on overlong IMSI (OS#2864) onto this test. */</span><br><span style="color: hsl(120, 100%, 40%);">+ var hexstring overlong_imsi := '012345789ABCDEF0123456789ABCDEF'H;</span><br><span>   f_init_handler(pars);</span><br><span> </span><br><span>    var PDU_ML3_MS_NW l3_lu := f_build_lu_tmsi('01020304'O); /* FIXME: Random */</span><br><span>@@ -823,6 +825,7 @@</span><br><span> </span><br><span>       /* Wait for + respond to ID REQ (IMSI) */</span><br><span>    BSSAP.receive(tr_PDU_DTAP_MT(tr_ML3_MT_MM_ID_Req('001'B)));</span><br><span style="color: hsl(120, 100%, 40%);">+   BSSAP.send(ts_PDU_DTAP_MO(ts_ML3_MO_MM_ID_Rsp_IMSI(overlong_imsi))); /* test for OS#2864 */</span><br><span>  BSSAP.send(ts_PDU_DTAP_MO(ts_ML3_MO_MM_ID_Rsp_IMSI(g_pars.imsi)));</span><br><span> </span><br><span>       /* Expect MSC to do UpdateLocation to HLR; respond to it */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9740">change 9740</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/9740"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I2c472bee76086f6c84ec684d2e58b3351ebc3147 </div>
<div style="display:none"> Gerrit-Change-Number: 9740 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>