<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">did you actually encounter any specific bug/problem before this patch?</p><p style="white-space: pre-wrap; word-wrap: break-word;">In general, I'm not aware of any USIM/ISIM specific requirements for the CLAss byte.</p><p style="white-space: pre-wrap; word-wrap: break-word;">IMHO, the underlying question is whether the card is a classic SIM (0xA0) or an UICC (0x00). But an UICC can have various other Applications, not  just USIM/ISIM.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, why reset the card after changing the class byte?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this needs more explanation in commit log and/or comments.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The link you quote is some APDU trace and not a specification reference.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, without this commit, it was not possible to select an AID on Sysmocom ISIM and USIM (also any operation related to USIM/ISIM AID) as pySim was throwing various forms 6Exx or 6Axx errors. Yeah are right about SIM being classic SIM (0xA0) or an UICC (0x00), I think the commit message here may be a bit misleading.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Regarding resetting of card, without doing that any changes done to CLA or P1 or P2 was not taken into effect resulting in failure of selecting AIDs or any operation related to USIM/ISIM AID.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, this commit was inspired by the implementation in sysmo-usim-tool, which uses the same CLA, P1 and P2 to read a USIM.</p><p style="white-space: pre-wrap; word-wrap: break-word;">P.S: With this commit i was even able to read a commercial USIM from Vodafone DE (previously it was not possible).</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I am seeing a similar problem/need with the CLA byte, shouldn't it be better addressed with a new SimCard class?</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/17880">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/17880">change 17880</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/+/17880"/><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: Ifea328eff3a381d7b82118e22d2bc0ec5f8a87e4 </div>
<div style="display:none"> Gerrit-Change-Number: 17880 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: herlesupreeth <herlesupreeth@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: herlesupreeth <herlesupreeth@gmail.com> </div>
<div style="display:none"> Gerrit-CC: guilly@gmail.com <guilly@gmail.com> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 26 Apr 2020 09:08:02 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>