<p><a href="https://gerrit.osmocom.org/11236">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11236/1//COMMIT_MSG">Commit Message:</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/11236/1//COMMIT_MSG@12">Patch Set #1, Line 12:</a> <code style="font-family:monospace,monospace">due to miss-configuration</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think there's something seriously wrong with osmo-bsc, if such "mis-configuration" can happen</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11236/1/src/osmo-bsc/gsm_08_08.c">File src/osmo-bsc/gsm_08_08.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/11236/1/src/osmo-bsc/gsm_08_08.c@501">Patch Set #1, Line 501:</a> <code style="font-family:monospace,monospace">LOGL_DEBUG</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">debug for something that will effectively make any voice call fail?   I think we should definitely have this at NOTICE if not ERROR level.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, the best is if we don't even allow the user to shoot himself into the foot, i.e. do whatever is needed to avoid being able to create such invalid configurations in the first place.  So for sure, we should not permit an empty codec supprt list to be created from the VTY in the MSC section.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess the only remainig case here is that the BTS supported list (like hard-coded for nanoBTS with no HRv1) doesn't have any intersection with the list specified for the "msc" node in the VTY?  Maybe in that case also already print an error message after reading the config file?  Or while reading it?  Intersect the "msc" codecs with those of each BTS, and print a big fat warning/error message if for any BTS there's no intersection?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11236">change 11236</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/11236"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I119607047a132b75b3077bbe56c97936d8ae6c96 </div>
<div style="display:none"> Gerrit-Change-Number: 11236 </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: Fri, 05 Oct 2018 09:40:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>