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

Change subject: global_platform: fix command "delete"
......................................................................

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

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




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 https://gerrit.osmocom.org/c/pysim/+/39196?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: Id04c89acbd4f449cb974d3cb05024f11dba4684e
Gerrit-Change-Number: 39196
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>

Reply via email to