<p>fixeria <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/26508">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">library/Osmocom_Types: fix maximum values passed to f_rnd_int()<br><br>It's clearly stated in a comment to this function that the returned<br>value is 0 <= x < max, so it's always less than the maximum.<br><br>Change-Id: If127fc6a2feeddbdd3ea636b56dadf8957941758<br>---<br>M library/Osmocom_Types.ttcn<br>1 file changed, 4 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/library/Osmocom_Types.ttcn b/library/Osmocom_Types.ttcn</span><br><span>index 3c0137c..967beb4 100644</span><br><span>--- a/library/Osmocom_Types.ttcn</span><br><span>+++ b/library/Osmocom_Types.ttcn</span><br><span>@@ -65,7 +65,7 @@</span><br><span> }</span><br><span> </span><br><span> /* return hexstring composed of random digits */</span><br><span style="color: hsl(0, 100%, 40%);">-function f_rnd_hexstring(in integer len, in integer max := 15) return hexstring {</span><br><span style="color: hsl(120, 100%, 40%);">+function f_rnd_hexstring(in integer len, in integer max := 16) return hexstring {</span><br><span>    var integer i;</span><br><span>       var hexstring ret := ''H;</span><br><span>    for (i := 0; i < len; i := i + 1) {</span><br><span>@@ -79,7 +79,7 @@</span><br><span>   var integer i;</span><br><span>       var octetstring ret := ''O;</span><br><span>  for (i := 0; i < len; i := i + 1) {</span><br><span style="color: hsl(0, 100%, 40%);">-          ret := ret & int2oct(f_rnd_int(255), 1);</span><br><span style="color: hsl(120, 100%, 40%);">+          ret := ret & int2oct(f_rnd_int(256), 1);</span><br><span>         }</span><br><span>    return ret;</span><br><span> }</span><br><span>@@ -91,7 +91,7 @@</span><br><span> }</span><br><span> </span><br><span> function f_rnd_imsi(in hexstring prefix) return hexstring {</span><br><span style="color: hsl(0, 100%, 40%);">-    return prefix & f_rnd_hexstring(15 - lengthof(prefix), 9);</span><br><span style="color: hsl(120, 100%, 40%);">+        return prefix & f_rnd_hexstring(15 - lengthof(prefix), 10);</span><br><span> }</span><br><span> </span><br><span> function f_rnd_msisdn(in octetstring prefix, integer len := 6) return octetstring {</span><br><span>@@ -249,7 +249,7 @@</span><br><span> </span><br><span> /* generate a random 11-bit RA (packet-switched only) */</span><br><span> function f_rnd_ra11_ps() return BIT11 {</span><br><span style="color: hsl(0, 100%, 40%);">-        var integer ra11 := f_rnd_int(bit2int('11111111111'B));</span><br><span style="color: hsl(120, 100%, 40%);">+       var integer ra11 := f_rnd_int(bit2int('11111111111'B) + 1);</span><br><span>  return int2bit(ra11, 11);</span><br><span> }</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/26508">change 26508</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-ttcn3-hacks/+/26508"/><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-Change-Id: If127fc6a2feeddbdd3ea636b56dadf8957941758 </div>
<div style="display:none"> Gerrit-Change-Number: 26508 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@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-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>