laforge has uploaded this change for review.

View Change

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):

To view, visit change 30440. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I9e4170721be30342bdce7fb4beeefd1927263ca6
Gerrit-Change-Number: 30440
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-MessageType: newchange