<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Please also update osmo-gsm-manuals.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, I have it as checklist item in the issue:<br>https://osmocom.org/issues/3733</p><p><a href="https://gerrit.osmocom.org/12409">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12409/2/include/osmocom/gsm/gsup.h">File include/osmocom/gsm/gsup.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12409/2/include/osmocom/gsm/gsup.h@277">Patch Set #2, Line 277:</a> <code style="font-family:monospace,monospace">osmo_gsup_imei_result</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">You are basically introducing three types: […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The response to CHECK_IMEI could be, as intended by the standard:</p><ul><li>ACK: this IMEI is okay, allow it to be used</li><li>NACK: this IMEI is not allowed to be used (e.g. because it is on a stolen device blacklist)</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">We don't check against a blacklist or whitelist in the Osmocom stack, and this is not planned IIRC. Right now, always ACK is sent back. But this could come in the future, so having the option to say ACK or NACK here is closer to the spec.</p><p style="white-space: pre-wrap; word-wrap: break-word;">ERROR is sent back when the IMEI can not be decoded.</p><p style="white-space: pre-wrap; word-wrap: break-word;">See also the related HLR changes using this message:<br>https://gerrit.osmocom.org/#/c/osmo-hlr/+/12411/1/src/hlr.c</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12409">change 12409</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/12409"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I085819df0ea7f3bfeb0cabebb5fd1942a23c6155 </div>
<div style="display:none"> Gerrit-Change-Number: 12409 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 21 Dec 2018 12:30:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>