<p>daniel has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/10166">View Change</a></p><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 to out of bounds leading to a<br>Dynamic test case error.<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, 4 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/66/10166/1</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..5078292 100644</span><br><span>--- a/msc/MSC_Tests.ttcn</span><br><span>+++ b/msc/MSC_Tests.ttcn</span><br><span>@@ -1155,8 +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%);">+  var integer len := float2int(rnd() * 241.0);</span><br><span>         var octetstring payl := f_rnd_octstring(len);</span><br><span style="color: hsl(120, 100%, 40%);">+ log("Length:", len);</span><br><span> </span><br><span>   /* Send Complete L3 Info with empty L3 frame */</span><br><span>      BSSAP.send(ts_BSSAP_Conn_Req(g_pars.sccp_addr_peer, g_pars.sccp_addr_own,</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: newchange </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: 1 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@sysmocom.de> </div>