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

Change subject: pySim.commands: use _checksw during get_data() method
......................................................................

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

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved




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 https://gerrit.osmocom.org/c/pysim/+/38047?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ic784bf0c30b22e5e83843aa6694e2706b4b2ac48
Gerrit-Change-Number: 38047
Gerrit-PatchSet: 5
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>

Reply via email to