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

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">msc: Fix random error of TC_cl3_rnd_payload<br><br>The random length for that test could go out of bounds leading to a<br>Dynamic test case error when sending the message.<br>The limiting field here is the lengthIndicator of PDU_BSSAP which<br>includes the length of the PDU_BSSMAP mesageType, cellId as well as the<br>layer3 info IE and lenght indicator additionally to the l3info payload.<br>So maximum length for the payload can only be 240 bytes (if the cell ID<br>is encoded in the longest possible way as BSSMAP_FIELD_LAC_RNC_CI).<br><br>Change-Id: I7be33e261a11f03a80a6b770b6acf0a4be49b85b<br>---<br>M msc/MSC_Tests.ttcn<br>1 file changed, 5 insertions(+), 1 deletion(-)<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 1427fd0..4a6b822 100644</span><br><span>--- a/msc/MSC_Tests.ttcn</span><br><span>+++ b/msc/MSC_Tests.ttcn</span><br><span>@@ -1155,7 +1155,11 @@</span><br><span> private function f_tc_cl3_rnd_payload(charstring id, BSC_ConnHdlrPars pars) runs on BSC_ConnHdlr {</span><br><span>     f_init_handler(pars);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       var integer len := float2int(rnd() * 256.0);</span><br><span style="color: hsl(120, 100%, 40%);">+  /* length is limited by PDU_BSSAP length field which includes some</span><br><span style="color: hsl(120, 100%, 40%);">+     * other fields beside l3info payload. So payl can only be 240 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+   * Since rnd() returns values < 1 multiply with 241</span><br><span style="color: hsl(120, 100%, 40%);">+         */</span><br><span style="color: hsl(120, 100%, 40%);">+   var integer len := float2int(rnd() * 241.0);</span><br><span>         var octetstring payl := f_rnd_octstring(len);</span><br><span> </span><br><span>    /* Send Complete L3 Info with empty L3 frame */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10166">change 10166</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/10166"/><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: I7be33e261a11f03a80a6b770b6acf0a4be49b85b </div>
<div style="display:none"> Gerrit-Change-Number: 10166 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>