dexter has uploaded this change for review. ( https://gerrit.osmocom.org/13339


Change subject: commands: correct case of a TLV tag (A5 => a5)
......................................................................

commands: correct case of a TLV tag (A5 => a5)

The hexadecimal tag defintions of pytlv are cas sensitive strings. So
'A5' is something different than 'a5'. Pytlv uses lower case letters for
the upper hexadecimal digits. Lets correct this.

Change-Id: I41a9933707783f6b1b68ebd91a365405ac0892d0
Related: OS#3850
---
M pySim/commands.py
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/39/13339/1

diff --git a/pySim/commands.py b/pySim/commands.py
index 73c2247..913f394 100644
--- a/pySim/commands.py
+++ b/pySim/commands.py
@@ -35,7 +35,7 @@
                # see also: ETSI TS 102 221, chapter 11.1.1.3.1 Response for MF,
                # DF or ADF
                 from pytlv.TLV import TLV
-               tlvparser = TLV(['82', '83', '84', 'A5', '8a', '8b', '8c', 
'80', 'ab', 'c6', '81', '88'])
+               tlvparser = TLV(['82', '83', '84', 'a5', '8a', '8b', '8c', 
'80', 'ab', 'c6', '81', '88'])

                # pytlv is case sensitive!
                fcp = fcp.lower()

--
To view, visit https://gerrit.osmocom.org/13339
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I41a9933707783f6b1b68ebd91a365405ac0892d0
Gerrit-Change-Number: 13339
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>

Reply via email to