<p>Hoernchen <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/21071">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">7816 fsm: move to static msgb<br><br>Error handling was difficult due to the need to pass msgb pointers<br>attached to the sub fsms back to the cb that is polled from the main<br>loop to be able free them, while ensuring they never get lost, because<br>memory leakage is deadly.<br><br>This is now fixed by using static pseudo-msgbs for the fsms that are<br>never deallocated. This only adds one tpdu tx copy, the ccid response<br>msgbs were already being copied anyway, so memory usage has not changed<br>except for the "unused slots" case that is not really important since<br>the octsim was designed around concurrent slot operation anway.<br><br>All of this allows convenient error handling in the allstate function<br>instead of having to spread it all over the sub fsms - in practice<br>handling errors mostly consists of card deactivation + returning a<br>proper failure message that matches the ccid command anyway.<br><br>Change-Id: I65e77c376aca9ed50e234a0b58a7450a8bbd4fe0<br>---<br>M ccid_common/ccid_slot_fsm.c<br>M ccid_common/iso7816_fsm.c<br>M ccid_common/iso7816_fsm.h<br>3 files changed, 216 insertions(+), 151 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ccid-firmware refs/changes/71/21071/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/21071">change 21071</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-ccid-firmware/+/21071"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ccid-firmware </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I65e77c376aca9ed50e234a0b58a7450a8bbd4fe0 </div>
<div style="display:none"> Gerrit-Change-Number: 21071 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Jenkins Builder </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>