dexter submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
global_platform: fix command "delete"

The delete command formats a TPDU, not APDU, which leads to warning messages

Related: OS#6679
Change-Id: Id04c89acbd4f449cb974d3cb05024f11dba4684e
---
M pySim/global_platform/__init__.py
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pySim/global_platform/__init__.py b/pySim/global_platform/__init__.py
index 28cc01e..043a86a 100644
--- a/pySim/global_platform/__init__.py
+++ b/pySim/global_platform/__init__.py
@@ -791,7 +791,7 @@
self.delete(0x00, p2, cmd)

def delete(self, p1:int, p2:int, data:Hexstr) -> ResTuple:
- cmd_hex = "80E4%02x%02x%02x%s" % (p1, p2, len(data)//2, data)
+ cmd_hex = "80E4%02x%02x%02x%s00" % (p1, p2, len(data)//2, data)
return self._cmd.lchan.scc.send_apdu_checksw(cmd_hex)

est_scp02_parser = argparse.ArgumentParser()

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

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Id04c89acbd4f449cb974d3cb05024f11dba4684e
Gerrit-Change-Number: 39196
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>