laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/30440 )
Change subject: pySim-trace: Also consider SW 91xx as successful ......................................................................
pySim-trace: Also consider SW 91xx as successful
Change-Id: I9e4170721be30342bdce7fb4beeefd1927263ca6 --- M pySim/apdu/__init__.py 1 file changed, 6 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/40/30440/1
diff --git a/pySim/apdu/__init__.py b/pySim/apdu/__init__.py index 3961fe9..1d1174d 100644 --- a/pySim/apdu/__init__.py +++ b/pySim/apdu/__init__.py @@ -135,7 +135,12 @@ if callable(method): return method() # default case: only 9000 is success - return self.sw == b'\x90\x00' + if self.sw == b'\x90\x00': + return True + # This is not really a generic positive APDU SW but specific to UICC/SIM + if self.sw[0] == 0x91: + return True + return False
class ApduCommand(Apdu, metaclass=ApduCommandMeta):
-
laforge