dexter has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/pysim/+/34897?usp=email )


Change subject: runtime: fix tracking of selected_adf
......................................................................

runtime: fix tracking of selected_adf

The class property selected_adf is not updated in all locations where an
ADF is selected, this means that we may loose track of the currently
selected ADF in some locations

Change-Id: I4cc0c58ff887422b4f3954d35c8380ddc00baa1d
Related: OS#5418
---
M pySim/runtime.py
1 file changed, 16 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/97/34897/1

diff --git a/pySim/runtime.py b/pySim/runtime.py
index a54a1b6..1c58f17 100644
--- a/pySim/runtime.py
+++ b/pySim/runtime.py
@@ -111,6 +111,7 @@
         for f in sorted(set(apps_profile) - set(apps_taken), key=str):
             try:
                 data, sw = self.card.select_adf_by_aid(f.aid)
+                self.selected_adf = f
                 if sw == "9000":
                     print(" %s: %s" % (f.name, f.aid))
                     apps_taken.append(f)
@@ -295,7 +296,6 @@
             raise RuntimeError('Cannot determine path from %s to %s' % 
(self.selected_file, file))

         self._select_pre(cmd_app)
-
         for p in inter_path:
             try:
                 if isinstance(p, CardADF):
@@ -344,6 +344,7 @@
             try:
                 if isinstance(f, CardADF):
                     (data, sw) = self.rs.card.select_adf_by_aid(f.aid, 
scc=self.scc)
+                    self.selected_adf = f
                 else:
                     (data, sw) = self.scc.select_file(f.fid)
                 self.selected_file = f

--
To view, visit https://gerrit.osmocom.org/c/pysim/+/34897?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I4cc0c58ff887422b4f3954d35c8380ddc00baa1d
Gerrit-Change-Number: 34897
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to