laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/37532?usp=email )

Change subject: pySim.apdu: Fix APDU CLA matching ......................................................................

pySim.apdu: Fix APDU CLA matching

The cla values as hex strings must be compared in case insensitive manner

Change-Id: I890bc385d6209e6cfe9b0c38bd9deee7ae50e5f5 --- M pySim/apdu/__init__.py 1 file changed, 13 insertions(+), 2 deletions(-)

Approvals: fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified dexter: Looks good to me, approved; Verified

diff --git a/pySim/apdu/__init__.py b/pySim/apdu/__init__.py index 6613e91..fbe7e6b 100644 --- a/pySim/apdu/__init__.py +++ b/pySim/apdu/__init__.py @@ -272,7 +272,7 @@ """Does the given CLA match the CLA list of the command?.""" if not isinstance(cla, str): cla = '%02X' % cla - cla = cla.lower() + cla = cla.upper() # see https://github.com/PyCQA/pylint/issues/7219 # pylint: disable=no-member for cla_match in cls._cla: @@ -282,7 +282,7 @@ cla_masked += 'X' else: cla_masked += cla[i] - if cla_masked == cla_match: + if cla_masked == cla_match.upper(): return True return False