<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/10182">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;">vlr_access_req_fsm.c: use osmo_strlcpy and avoid compiler error<br><br>gcc 8.1.0:<br>../../../../src/osmo-msc/src/libvlr/vlr_access_req_fsm.c:679:3: error: ‘strncpy’ output may be truncated copying 15 bytes from a string of length 31 [-Werror=stringop-truncation]<br>   strncpy(par->imsi, mi_string, sizeof(par->imsi)-1);<br>   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br><br>The Mobile Identity is a union of various kinds, but the IMSI is at most 15<br>digits, so truncation is "intended". I hope other layers validate the correct<br>length of an IMSI MI.<br><br>Change-Id: I0a17a188fc91e42e252ae4bf1d6cd0bf0e5eb077<br>---<br>M src/libvlr/vlr_access_req_fsm.c<br>1 file changed, 1 insertion(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libvlr/vlr_access_req_fsm.c b/src/libvlr/vlr_access_req_fsm.c</span><br><span>index 79eb75f..3a0760d 100644</span><br><span>--- a/src/libvlr/vlr_access_req_fsm.c</span><br><span>+++ b/src/libvlr/vlr_access_req_fsm.c</span><br><span>@@ -676,8 +676,7 @@</span><br><span>    mi_type = mi_lv[1] & GSM_MI_TYPE_MASK;</span><br><span>   switch (mi_type) {</span><br><span>   case GSM_MI_TYPE_IMSI:</span><br><span style="color: hsl(0, 100%, 40%);">-          strncpy(par->imsi, mi_string, sizeof(par->imsi)-1);</span><br><span style="color: hsl(0, 100%, 40%);">-               par->imsi[sizeof(par->imsi)-1] = '\0';</span><br><span style="color: hsl(120, 100%, 40%);">+          osmo_strlcpy(par->imsi, mi_string, sizeof(par->imsi));</span><br><span>                 par->by_tmsi = false;</span><br><span>             break;</span><br><span>       case GSM_MI_TYPE_TMSI:</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10182">change 10182</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/10182"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I0a17a188fc91e42e252ae4bf1d6cd0bf0e5eb077 </div>
<div style="display:none"> Gerrit-Change-Number: 10182 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>