<p><a href="https://gerrit.osmocom.org/12924">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/12924/1/src/gsm/gsm0808.c">File src/gsm/gsm0808.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/12924/1/src/gsm/gsm0808.c@200">Patch Set #1, Line 200:</a> <code style="font-family:monospace,monospace">        msgb_v_put(msg, BSS_MAP_MSG_CLEAR_CMD);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">In the commit description you state you are fixing some length, but I also see some parts of this me […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">please note the sublte difference of first putting and then pushing at the end.  So the order in the code is reversed, but the order of the message is not.  Adding the header at the end alows us to use msgb_length(msg) for dynamically computing the length, which is exactly what prevents this kind of bug (hard-coded length values at start of message generation).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12924">change 12924</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/12924"/><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: I4d07d25fb03ca0f89fd7b94226c54309c77a010a </div>
<div style="display:none"> Gerrit-Change-Number: 12924 </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: Mon, 18 Feb 2019 12:48:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>