laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/33849 ) Change subject: cards: fix swapped PIN mapping number ...................................................................... cards: fix swapped PIN mapping number The constant for _adm_chv_num is swapped. It should be 0x0A, rather than 0xA0 Change-Id: I5680d2deee855ef316a98058e8c8ff8cf4edbbb2 --- M pySim/cards.py 1 file changed, 13 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/pySim/cards.py b/pySim/cards.py index c6a68fc..c87c488 100644 --- a/pySim/cards.py +++ b/pySim/cards.py @@ -101,7 +101,7 @@ self._scc.cla_byte = "00" self._scc.sel_ctrl = "0004" # request an FCP # See also: ETSI TS 102 221, Table 9.3 - self._adm_chv_num = 0xA0 + self._adm_chv_num = 0x0A def probe(self) -> bool: # EF.DIR is a mandatory EF on all ICCIDs; however it *may* also exist on a TS 51.011 SIM -- To view, visit https://gerrit.osmocom.org/c/pysim/+/33849 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I5680d2deee855ef316a98058e8c8ff8cf4edbbb2 Gerrit-Change-Number: 33849 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged