<p>dexter <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/23819">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">utils: fix encoding of EF.MSISDN<br><br>The encoding of EF.MSISDN is a bit unstrutured. The encoder function<br>does not return a valid result since it lacks the parameters<br>Capability/Configuration2 Record Identifier and Extension5 Record<br>Identifier, which are mandatory but can be set to 0xFF. Also the<br>encoder gets its input from pySim-shell, so it should have some<br>more input validation, especially when the user encodes an empty<br>string. The encoder and decoder function also do not have unit-tests.<br><br>Since the encoder now adds the missing two bytes by isself this does<br>not have to be done manually anymore, so cards.py needs to be<br>re-aligned.<br><br>For pySim-shell.py the encoder is used from ts_51_011.py. Unfortunately<br>it is used wrongly there. The optional Alpha Identifier is required<br>here as well.<br><br>Related: OS#4963<br>Change-Id: Iee5369b3e3ba7fa1155facc8fa824bc60e33b55b<br>---<br>M pySim/cards.py<br>M pySim/ts_51_011.py<br>M pySim/utils.py<br>M tests/test_utils.py<br>4 files changed, 46 insertions(+), 6 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/19/23819/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/23819">change 23819</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/pysim/+/23819"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iee5369b3e3ba7fa1155facc8fa824bc60e33b55b </div>
<div style="display:none"> Gerrit-Change-Number: 23819 </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: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>