<p>fixeria <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21073">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified
  fixeria: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">TLLI 0x00000000 is a valid TLLI, use 0xffffffff instead<br><br>The assumption that TLLI 0x00000000 is invalid and can be used<br>as the initializer is wrong.  Similar to TMSI, 0x00000000 is a<br>perfectly valid value, while 0xffffffff is reserved - use it.<br><br>According to 3GPP TS 23.003, section 2.4, a TMSI/P-TMSI with<br>all 32 bits equal to 1 is special and shall not be allocated by<br>the network.  The reason is that it must be stored on the SIM,<br>where 'ff'O represents the erased state.  According to section<br>2.6 of the same document, a local/foreign TLLI is derived from<br>P-TMSI, so the same rule applies to TLLI.<br><br>I manually checked and corrected all occurances of 'tlli' in the<br>code.  The test expectations have been adjusted with this command:<br><br>  $ find tests/ -name "*.err" | xargs sed -i "s/0x00000000/0xffffffff/g"<br><br>so there should be no behavior change.  The only exception is<br>the 'TypesTest', where TLLI 0xffffffff is being encoded and<br>expected in the hexdump, so I regenerated the test output.<br><br>Change-Id: Ie89fab75ecc1d8b5e238d3ff214ea7ac830b68b5<br>Related: OS#4844<br>---<br>M src/bts.h<br>M src/gprs_bssgp_pcu.cpp<br>M src/gprs_ms.cpp<br>M src/gprs_ms.h<br>M src/gprs_ms_storage.cpp<br>M src/gprs_ms_storage.h<br>M src/tbf.cpp<br>M src/tbf.h<br>M src/tbf_ul.cpp<br>M tests/alloc/AllocTest.err<br>M tests/app_info/AppInfoTest.err<br>M tests/ms/MsTest.err<br>M tests/tbf/TbfTest.err<br>M tests/types/TypesTest.cpp<br>M tests/types/TypesTest.err<br>M tests/types/TypesTest.ok<br>16 files changed, 196,751 insertions(+), 196,741 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/21073">change 21073</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-pcu/+/21073"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie89fab75ecc1d8b5e238d3ff214ea7ac830b68b5 </div>
<div style="display:none"> Gerrit-Change-Number: 21073 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>