[S] Change in pysim[master]: euicc: Fix decoding of SubjectKeyIdentifier.

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/35523?usp=email ) Change subject: euicc: Fix decoding of SubjectKeyIdentifier. ...................................................................... euicc: Fix decoding of SubjectKeyIdentifier. There's actually no additional TLV structure inside the Tag 0x04. Change-Id: Ic922355308747a888083c5b26765d272b6b20bd0 --- M pySim/euicc.py 1 file changed, 14 insertions(+), 5 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/pySim/euicc.py b/pySim/euicc.py index e405f30..81fbd1e 100644 --- a/pySim/euicc.py +++ b/pySim/euicc.py @@ -94,13 +94,11 @@ # SGP.22 Section 5.7.8: GetEUICCInfo class SVN(BER_TLV_IE, tag=0x82): _construct = VersionType -class SubjectKeyIdentifier(BER_TLV_IE, tag=0x81): +class SubjectKeyIdentifier(BER_TLV_IE, tag=0x04): _construct = HexAdapter(GreedyBytes) -class SubjectKeyIdentifierSeq(BER_TLV_IE, tag=0x04, nested=[SubjectKeyIdentifier]): +class EuiccCiPkiListForVerification(BER_TLV_IE, tag=0xa9, nested=[SubjectKeyIdentifier]): pass -class EuiccCiPkiListForVerification(BER_TLV_IE, tag=0xa9, nested=[SubjectKeyIdentifierSeq]): - pass -class EuiccCiPkiListForSigning(BER_TLV_IE, tag=0xaa, nested=[SubjectKeyIdentifierSeq]): +class EuiccCiPkiListForSigning(BER_TLV_IE, tag=0xaa, nested=[SubjectKeyIdentifier]): pass class EuiccInfo1(BER_TLV_IE, tag=0xbf20, nested=[SVN, EuiccCiPkiListForVerification, EuiccCiPkiListForSigning]): pass -- To view, visit https://gerrit.osmocom.org/c/pysim/+/35523?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ic922355308747a888083c5b26765d272b6b20bd0 Gerrit-Change-Number: 35523 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged