<p style="white-space: pre-wrap; word-wrap: break-word;">Please think about inbound message parsing in context of what I wrote below in the example of lchan lookup by chan_nr.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24430">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/osmo-bsc/+/24430/9/src/osmo-bsc/bts_trx.c">File src/osmo-bsc/bts_trx.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-bsc/+/24430/9/src/osmo-bsc/bts_trx.c@165">Patch Set #9, Line 165:</a> <code style="font-family:monospace,monospace"> case ABIS_RSL_CHAN_NR_CBITS_OSMO_VAMOS_Bm_ACCHs:</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't see anything that ensures those new osmocom-specific cbits are only used if the bts model is of type osmo-bts and has advertised VAMOS capability.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think your statement may be true for outbound messages that the BSC generates, but this kind of code change to generic/common code that is not guarded by a certain BTS model / feature flag is what I'm worried about.</p><p style="white-space: pre-wrap; word-wrap: break-word;">ASsuming some other BTS model/vendor has a different use of those cbits, we are no longer running into the "return NULL" but we're going to compute an index into the lchan array on something we don't know what it might be.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24430">change 24430</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-bsc/+/24430"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I957eff0d2c33ec795eda75a4bff21965b0179f73 </div>
<div style="display:none"> Gerrit-Change-Number: 24430 </div>
<div style="display:none"> Gerrit-PatchSet: 9 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 31 May 2021 10:08:05 +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>