<p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22368">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22368/6/src/gprs_bssgp_rim.c">File src/gprs_bssgp_rim.c:</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/osmo-pcu/+/22368/6/src/gprs_bssgp_rim.c@69">Patch Set #6, Line 69:</a> <code style="font-family:monospace,monospace">         app_cont->si[app_cont->num_si] = bts->si1 + 2;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Add comment explaining what's this +2 about.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22368/6/src/gprs_bssgp_rim.c@266">Patch Set #6, Line 266:</a> <code style="font-family:monospace,monospace">              format_response_pdu(&resp_pdu, pdu, bts);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You are passing here the "dst BTS" and you are then using then using it to gather the information to fill the response message. However, you should probably instead be looking up the BTS based on CGI-PS from "reprt_cell" from NACC application in higher layers.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I agree though that in general both addresses should be the same, but I would then at least add some sanity check making sure both addresses match.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22368/6/src/pcu_l1_if.cpp">File src/pcu_l1_if.cpp:</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/osmo-pcu/+/22368/6/src/pcu_l1_if.cpp@300">Patch Set #6, Line 300:</a> <code style="font-family:monospace,monospace">  if (len == 1) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">switch (len) on all the possibilities in this function.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/22368">change 22368</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/+/22368"/><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: Id72118120c14984d2fb1b918b41fac4868150d41 </div>
<div style="display:none"> Gerrit-Change-Number: 22368 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </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-Comment-Date: Mon, 01 Feb 2021 11:03:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>