laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
runtime: Reset selected_file_fcp[_hex] if SELECT returns no data

In case SELECT doesn't return any response data, we must reset
the lchan.selected_file_fcp* members to None to prevent pySim-shell
preventing stale data from the previously selected file.

Change-Id: Ia04b8634e328e604e8df7e8d59b7fd532242d2ca
---
M pySim/runtime.py
1 file changed, 16 insertions(+), 0 deletions(-)

diff --git a/pySim/runtime.py b/pySim/runtime.py
index f836ec8..3aeaab1 100644
--- a/pySim/runtime.py
+++ b/pySim/runtime.py
@@ -304,6 +304,9 @@
if select_resp_data:
self.selected_file_fcp_hex = select_resp_data
self.selected_file_fcp = self.selected_file.decode_select_response(select_resp_data)
+ else:
+ self.selected_file_fcp_hex = None
+ self.selected_file_fcp = None

# register commands of new file
if cmd_app and self.selected_file.shell_commands:

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

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ia04b8634e328e604e8df7e8d59b7fd532242d2ca
Gerrit-Change-Number: 35723
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged