<p>neels <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/19768">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">paging: pass struct osmo_mobile_identity, not encoded IE bytes<br><br>In get_paging_mi(), before this, an encoded buffer of Mobile Identity bytes is<br>returned. Code paths following this repeatedly decode the Mobile Identity<br>bytes, e.g. for logging. Also, in get_paging_mi(), since the TMSI is read in<br>from a different encoding than a typical Mobile Identity IE, the TMSI was<br>manually encoded into a typical Mobile Identity IE. This is essentially a code<br>dup of osmo_mobile_identity_encode(). Stop this madness.<br><br>Instead, in get_paging_mi(), return a decoded struct osmo_mobile_identity. Code<br>paths after this use the struct osmo_mobile_identity directly without repeated<br>decoding.<br><br>At the point of finally needing an encoded Mobile Identity IE (in<br>Encoding::write_paging_request()), do a proper osmo_mobile_identity_encode().<br><br>Since this may return errors, add an rc check for the caller of<br>write_paging_request(), gprs_rlcmac_paging_request().<br><br>A side effect is stricter validation of the Mobile Identity passing through the<br>Paging code path. Before, invalid MI might have passed through unnoticed.<br><br>Change-Id: Iad845acb0096b75dc453105c9c16b2252879b4ca<br>---<br>M src/bts.cpp<br>M src/bts.h<br>M src/encoding.cpp<br>M src/encoding.h<br>M src/gprs_bssgp_pcu.cpp<br>M src/gprs_rlcmac.cpp<br>M src/gprs_rlcmac.h<br>M src/pcu_l1_if.cpp<br>M src/pdch.cpp<br>M src/pdch.h<br>10 files changed, 51 insertions(+), 40 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/68/19768/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/19768">change 19768</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/+/19768"/><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: Iad845acb0096b75dc453105c9c16b2252879b4ca </div>
<div style="display:none"> Gerrit-Change-Number: 19768 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Jenkins Builder </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>