<p><a href="https://gerrit.osmocom.org/13081">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/13081/1/src/gsm/gsm0808_utils.c">File src/gsm/gsm0808_utils.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/13081/1/src/gsm/gsm0808_utils.c@1417">Patch Set #1, Line 1417:</a> <code style="font-family:monospace,monospace">            return -EINVAL;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">EINVAL should be returned if the input data was wrong. However, here it looks more like the check is reporting an implementation error of this function.  If some input data is given that results in more than 4 codecs to be chosen, then this function is broken because it didn't constrain itself to four.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Wouldn't it make more sense to simply check after each count++ if we now have 4.  And if we have four, return those four rather than adding more and then returning EINVAL?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13081">change 13081</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/13081"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I6fd7f4073b84093742c322752f2fd878d1071e15 </div>
<div style="display:none"> Gerrit-Change-Number: 13081 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 28 Feb 2019 15:47:50 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>