dexter has submitted this change. (
https://gerrit.osmocom.org/c/pysim/+/28170 )
Change subject: filesystem: also return the encoded FCP from probe_file
......................................................................
filesystem: also return the encoded FCP from probe_file
he method probe_file returns the decoded FCP after it managed to
successfully probe the file. Lets also return the encoded FCP string, as
it is needed by the caller.
Change-Id: Ia5659e106fb0d6fb8b77506a10eba309e764723e
---
M pySim/filesystem.py
1 file changed, 3 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, approved
diff --git a/pySim/filesystem.py b/pySim/filesystem.py
index 5bbd57b..7c5d49a 100644
--- a/pySim/filesystem.py
+++ b/pySim/filesystem.py
@@ -1402,7 +1402,7 @@
self.selected_file.add_files([f])
self.selected_file = f
- return select_resp
+ return select_resp, data
def _select_pre(self, cmd_app):
# unregister commands of old file
@@ -1487,7 +1487,8 @@
raise RuntimeError("%s: %s - %s" % (swm.sw_actual, k[0],
k[1]))
select_resp = f.decode_select_response(data)
else:
- select_resp = self.probe_file(name, cmd_app)
+ (select_resp, data) = self.probe_file(name, cmd_app)
+
# store the raw + decoded FCP for later reference
self.selected_file_fcp_hex = data
self.selected_file_fcp = select_resp
--
To view, visit
https://gerrit.osmocom.org/c/pysim/+/28170
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ia5659e106fb0d6fb8b77506a10eba309e764723e
Gerrit-Change-Number: 28170
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: merged