<p>pespin <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24812">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">pdch: Fix heap-use-after-free in pdch->ulc<br><br>In existing previous code, pdch->ulc would be freed in<br>gprs_rlcmac_pdch::free_resources() when  it became disabled as per PCUIF<br>info_ind (for instance, when a DYN TS is switched PDCH->SDCCH8).<br>However, pdch->ulc was so far only allocated during pdch_init, which is<br>only called during bts_alloc() time.<br>Hence, after first info_ind disabling it, if it became again enabled<br>(again by info_ind re-enabling it after SDCCH8 was not longer in use),<br>the pdch->ulc would be used again but it would point to freed memory.<br><br>Let's rearrange how/when resources are freed to make it more logical.<br>With this patch, pdch internal resources are freed upon ->disable(), and<br>re-allocated upon ->enable().<br><br>Change-Id: Id51f5f6a54ac9f24b784c17bc360ac38f5726fc7<br>---<br>M src/osmobts_sock.c<br>M src/pcu_l1_if.cpp<br>M src/pdch.cpp<br>M src/pdch.h<br>M tests/tbf/TbfTest.cpp<br>M tests/ulc/PdchUlcTest.cpp<br>6 files changed, 36 insertions(+), 19 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/12/24812/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/24812">change 24812</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-pcu/+/24812"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Id51f5f6a54ac9f24b784c17bc360ac38f5726fc7 </div>
<div style="display:none"> Gerrit-Change-Number: 24812 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>