<p>dexter <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/10334">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">GSCON: call api of a_reset.c with msc object directly<br><br>The API of a_reset.c is currently called with a pointer to struct<br>reset_ctx. This puts the responsibility of checking the presence of<br>msc->a.reset_fsm to the caller. It would be much more effective if the<br>caller would check if msc->a.reset_fsm before dereferencing it.<br><br>This also fixes at least one segfault that ocurrs when gscon_timer_cb()<br>is called but no sccp connection is present yet. Therefore the pointer<br>to bsc_msc_data would not be populated. This is now detected by<br>a_reset.c itsself.<br><br>- minor code cleanups<br>- call a_reset.c functions with msc (struct bsc_msc_data)<br><br>Change-Id: I0802aaadf0af4e58e41c98999e8c6823838adb61<br>Related: OS#3447<br>---<br>M include/osmocom/bsc/a_reset.h<br>M src/osmo-bsc/a_reset.c<br>M src/osmo-bsc/bsc_subscr_conn_fsm.c<br>M src/osmo-bsc/gsm_08_08.c<br>M src/osmo-bsc/osmo_bsc_bssap.c<br>M src/osmo-bsc/osmo_bsc_sigtran.c<br>6 files changed, 54 insertions(+), 39 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/34/10334/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/10334">change 10334</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/10334"/><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: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I0802aaadf0af4e58e41c98999e8c6823838adb61 </div>
<div style="display:none"> Gerrit-Change-Number: 10334 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>