laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
pySim.commands: use _checksw during get_data() method

All other methods use send_apdu_checksw, just get_data()
was missing the _checksw part.

Change-Id: Ic784bf0c30b22e5e83843aa6694e2706b4b2ac48
---
M pySim/commands.py
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/pySim/commands.py b/pySim/commands.py
index f13e18b..5c5933a 100644
--- a/pySim/commands.py
+++ b/pySim/commands.py
@@ -794,8 +794,9 @@
data, sw = self.send_apdu_checksw('8076010008' + token, apply_lchan = False)
return (data, sw)

+ # GPC_SPE_034 11.3
def get_data(self, tag: int, cla: int = 0x00):
- data, sw = self.send_apdu('%02xca%04x00' % (cla, tag))
+ data, sw = self.send_apdu_checksw('%02xca%04x00' % (cla, tag))
return (data, sw)

# TS 31.102 Section 7.5.2

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

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ic784bf0c30b22e5e83843aa6694e2706b4b2ac48
Gerrit-Change-Number: 38047
Gerrit-PatchSet: 5
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>