<p>Stefan Sperling has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/10650">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">properly handle mandatory cli param of gsm0808_create_paging2<br><br>The cell identifier list parameter is mandatory. Document it as<br>such, and tweak code which treated it like an optional parameter.<br><br>No functional change. The existing code already asserts that<br>a non-NULL value is passed for this parameter.<br><br>Change-Id: I3716f9d5b210e0a7e6f45c9fe3fc34024e5234ad<br>Related: OS#3021<br>---<br>M src/gsm/gsm0808.c<br>1 file changed, 4 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/50/10650/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/gsm/gsm0808.c b/src/gsm/gsm0808.c</span><br><span>index a90aa22..fec3d25 100644</span><br><span>--- a/src/gsm/gsm0808.c</span><br><span>+++ b/src/gsm/gsm0808.c</span><br><span>@@ -588,7 +588,7 @@</span><br><span> /*! Create BSSMAP PAGING message</span><br><span>  *  \param[in] imsi Mandatory paged IMSI in string representation</span><br><span>  *  \param[in] tmsi Optional paged TMSI</span><br><span style="color: hsl(0, 100%, 40%);">- *  \param[in] cil Cell Identity List (where to page)</span><br><span style="color: hsl(120, 100%, 40%);">+ *  \param[in] cil Mandatory Cell Identity List (where to page)</span><br><span>  *  \param[in] chan_needed Channel Type needed</span><br><span>  *  \returns callee-allocated msgb with BSSMAP PAGING message */</span><br><span> struct msgb *gsm0808_create_paging2(const char *imsi, const uint32_t *tmsi,</span><br><span>@@ -615,7 +615,7 @@</span><br><span>        /* Message Type 3.2.2.1 */</span><br><span>   msgb_v_put(msg, BSS_MAP_MSG_PAGING);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        /* IMSI 3.2.2.6 */</span><br><span style="color: hsl(120, 100%, 40%);">+    /* mandatory IMSI 3.2.2.6 */</span><br><span>         mid_len = gsm48_generate_mid_from_imsi(mid_buf, imsi);</span><br><span>       msgb_tlv_put(msg, GSM0808_IE_IMSI, mid_len - 2, mid_buf + 2);</span><br><span> </span><br><span>@@ -626,9 +626,8 @@</span><br><span>                           (uint8_t *) & tmsi_sw);</span><br><span>     }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   /* Cell Identifier List 3.2.2.27 */</span><br><span style="color: hsl(0, 100%, 40%);">-     if (cil)</span><br><span style="color: hsl(0, 100%, 40%);">-                gsm0808_enc_cell_id_list2(msg, cil);</span><br><span style="color: hsl(120, 100%, 40%);">+  /* mandatory Cell Identifier List 3.2.2.27 */</span><br><span style="color: hsl(120, 100%, 40%);">+ gsm0808_enc_cell_id_list2(msg, cil);</span><br><span> </span><br><span>     /* Channel Needed 3.2.2.36 */</span><br><span>        if (chan_needed) {</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10650">change 10650</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/10650"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I3716f9d5b210e0a7e6f45c9fe3fc34024e5234ad </div>
<div style="display:none"> Gerrit-Change-Number: 10650 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>