<p>Pau Espin Pedrol <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/11339">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp: osmux: Improve checks around activating and using enabled osmux<br><br>* Refactor code to have unified checks on all paths activating Osmux.<br>* Improve checkings at activation time and add logging.<br>* Code now enforces endp osmux status to be enabled before processing<br>the frame through endp->osmux.out. Before, a delayed or bad pkt could<br>arrive and be processed by an endp with osmux not enabled, using<br>endp->osmux.out that was not initialized and ended up crashing:<br>libosmo-netif/src/osmux.c:281:3: runtime error: member access within null pointer of type 'struct msgb'<br><br>This could also happen if a BSC started sending or we received (non legacy dummy) osmux<br>frames before we received the BSC CRCX ACK agreeing on osmux negotiation<br>and switching to ACTIVATING state.<br><br>Related: SYS#4350<br>Change-Id: I3536169c94e65f999aaa9c9e43cc7dab8551d30f<br>---<br>M openbsc/src/libmgcp/mgcp_osmux.c<br>1 file changed, 43 insertions(+), 42 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/39/11339/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/11339">change 11339</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/11339"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: openbsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I3536169c94e65f999aaa9c9e43cc7dab8551d30f </div>
<div style="display:none"> Gerrit-Change-Number: 11339 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>