<p><a href="https://gerrit.osmocom.org/12873">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/12873/1/src/common/oml.c">File src/common/oml.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/12873/1/src/common/oml.c@1453">Patch Set #1, Line 1453:</a> <code style="font-family:monospace,monospace">         msgb_l3trim(msg, oh->length);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">That's weird imho, I'd drop it with oml_tx_failure_event_rep(), since attempting decoding potentiall […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">To me it's quite clear: The OML protocol states there's 10 bytes of payload (IEs), and that's what we have to parse.  Extra data at the end of the message should not concern us.  In Ethernet, small frames get a trailer at the end.  In LLC, there's a CRC at the end. I think this is generally customary that you use the length value from the protocol/header you currently work on and ignore any extra data at the end.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12873">change 12873</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/12873"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib98f0d7c2cff9172714ed18667c02564540d65d7 </div>
<div style="display:none"> Gerrit-Change-Number: 12873 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 10 Feb 2019 22:23:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>