<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/15719">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/libosmocore/+/15719/5/src/gsm/gsm0502.c">File src/gsm/gsm0502.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/libosmocore/+/15719/5/src/gsm/gsm0502.c@181">Patch Set #5, Line 181:</a> <code style="font-family:monospace,monospace">struct fn_remap_table *table = fn_remap_table_ptr[channel];</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks for implementing the table-lookup-table as per my earlier review comment.  However, my review comment also contained something like ... you just have to do OSMO_ASSERT(channel > ARRAY_SIZE(fn_remap_table)).  That part is missing here and we can have out-of-bound reads if some code should ever call us with some broken value for 'channel'.  At runtime, the enum is just an int, and nothing guarantees that this value is within bounds.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/15719">change 15719</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/libosmocore/+/15719"/><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-Change-Id: I3d71c66f8c401f5afbad9b1c86c24580dab9e0ce </div>
<div style="display:none"> Gerrit-Change-Number: 15719 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 23 Oct 2019 21:40:31 +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>