diff options
author | Philipp Maier <pmaier@sysmocom.de> | 2017-03-21 12:05:30 +0100 |
---|---|---|
committer | Philipp Maier <pmaier@sysmocom.de> | 2017-04-06 00:38:22 +0200 |
commit | 4146086d2fd2e1ffae06dfa5c1dece3b742e7648 (patch) | |
tree | 81491a305ce953efc04221e9fd8575b33a3421ec /pySim/commands.py | |
parent | 3ce84d9a4478768a619a0875dc87319415fdf0a1 (diff) |
Fix select control parameter
sysmo-usim-sjs1 requires P2 to be set to 0x0C (request FCI) when
using the USIM application commands. The FCI is not used by pysim
anyway and might even cause problems with other cards.
This commit adds a pair of get/set methods to the SimCardCommands
class in order to set a default for the selection control
parameters (P1, P2). (Similar to the set/get methods for the class
byte)
The SysmoUSIMSJS1 class now calls the setter method for the
selection control parameters inside of its constructuor and sets
the selection control parameter default to "000C". This way we
can be sure that we only change the behaviour for sysmo-usim-sjs1
and do not break support for any other cards.
Change-Id: I1993a267c952bf37d5de1cb4e1107f445614c17b
Diffstat (limited to 'pySim/commands.py')
-rw-r--r-- | pySim/commands.py | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/pySim/commands.py b/pySim/commands.py index b7fb77f..777dd24 100644 --- a/pySim/commands.py +++ b/pySim/commands.py @@ -29,6 +29,7 @@ class SimCardCommands(object): def __init__(self, transport): self._tp = transport; self._cla_byte = "a0" + self.sel_ctrl = "0000" @property def cla_byte(self): @@ -37,11 +38,17 @@ class SimCardCommands(object): def cla_byte(self, value): self._cla_byte = value + @property + def sel_ctrl(self): + return self._sel_ctrl + @sel_ctrl.setter + def sel_ctrl(self, value): + self._sel_ctrl = value def select_file(self, dir_list): rv = [] for i in dir_list: - data, sw = self._tp.send_apdu_checksw(self.cla_byte + "a4000002" + i) + data, sw = self._tp.send_apdu_checksw(self.cla_byte + "a4" + self.sel_ctrl + "02" + i) rv.append(data) return rv |