laforge has uploaded this change for review. ( 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(-)

git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/23/35523/1

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