<p>fixeria <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/25674">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">rsl: prevent race condition during timeslot re-configuration<br><br>It may happen that the BSC requests logical channel activation on a<br>dynamic timeslot, which is in a process of switching from one pchan<br>type to another due to a preceding channel activation request.<br><br>In this case 'struct gsm_bts_trx_ts' already holds an msgb with the<br>preceding RSL CHANnel ACTIVation message, that is normally handled<br>once the PHY completes the process of timeslot re-configuration.<br><br>On receipt of subsequent RSL CHANnel ACTIVation messages, in function<br>dyn_ts_l1_reconnect() we overwrite the preceeding msgb (memleak), by<br>the most recent one.  And once the timeslot re-configuration is done,<br>only the most recent CHANnel ACTIVation message gets ACKed.<br><br>In order to avoid this, let's move the msgb ownership to 'struct<br>gsm_lchan', so it cannot be overwritten by the CHANnel ACTIVation<br>message that is related to a different lchan on the same timeslot.<br><br>Change-Id: Ia625c2827fca883ea712076706d5ef21ed793ba6<br>Related: I3b602ac9dbe0ab3e80eb30de573c9b48a79872d8<br>Fixes: OS#5245<br>---<br>M include/osmo-bts/gsm_data.h<br>M src/common/rsl.c<br>2 files changed, 22 insertions(+), 19 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/74/25674/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/25674">change 25674</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/c/osmo-bts/+/25674"/><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-Change-Id: Ia625c2827fca883ea712076706d5ef21ed793ba6 </div>
<div style="display:none"> Gerrit-Change-Number: 25674 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>