<p>laforge has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/23580">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">ts_31_102: Fix decode_select_response() for DF.5GS<br><br>In Change-Id I848a766e6d00be497c7db905475e0681cce197ac we added a CardDF<br>instance for DF_5GS.  That DF should not have provided a<br>decode_select_response() method, and instead fall back to that of the<br>base class, which calls the method of the parent directory (ADF_USIM).<br><br>The difference is illustrated below<br><br>pySIM-shell (MF/ADF.USIM/EF.IMSI)> select DF.5GS<br>"622e8202782183025fc0a509800171830400018d088a01058c056611111111c60f90017083010183018183010a83010b"<br><br>vs. (with this patch):<br><br>pySIM-shell (MF/ADF.USIM)> select DF.5GS<br>{<br>    "file_descriptor": {<br>        "shareable": true,<br>        "file_type": "df",<br>        "structure": "no_info_given"<br>    },<br>    "file_identifier": "5FC0",<br>    "proprietary_info": {<br>        "uicc_characteristics": "71",<br>        "available_memory": 101640<br>    },<br>    "life_cycle_status_int": "operational_activated",<br>    "security_attrib_compact": "6611111111",<br>    "pin_status_template_do": "90017083010183018183010A83010B"<br>}<br><br>Change-Id: I80612711bbc8c47285a828a0759b20beea6619f1<br>---<br>M pySim/ts_31_102.py<br>1 file changed, 0 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/80/23580/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/pySim/ts_31_102.py b/pySim/ts_31_102.py</span><br><span>index 9febaf7..cefc9d4 100644</span><br><span>--- a/pySim/ts_31_102.py</span><br><span>+++ b/pySim/ts_31_102.py</span><br><span>@@ -479,9 +479,6 @@</span><br><span> </span><br><span>         self.add_files(files)</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-    def decode_select_response(self, data_hex):</span><br><span style="color: hsl(0, 100%, 40%);">-        return data_hex</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span> class ADF_USIM(CardADF):</span><br><span>     def __init__(self, aid='a0000000871002', name='ADF.USIM', fid=None, sfid=None,</span><br><span>                  desc='USIM Application'):</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/23580">change 23580</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/+/23580"/><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: I80612711bbc8c47285a828a0759b20beea6619f1 </div>
<div style="display:none"> Gerrit-Change-Number: 23580 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>