laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/33208 ) Change subject: Fix result parsing of "suspend_uicc" ...................................................................... Fix result parsing of "suspend_uicc" prior to this patch, the suspend_uicc command would always cause a python exception as a list of integers was returned by decode_duration rather than a single integer (that can be used with %u format string). Change-Id: I981e9d46607193176b28cb574564e6da546501ba --- M pySim/commands.py 1 file changed, 13 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/pySim/commands.py b/pySim/commands.py index b0f8b39..6acdb2e 100644 --- a/pySim/commands.py +++ b/pySim/commands.py @@ -610,7 +610,7 @@ def decode_duration(enc: Hexstr) -> int: time_unit = enc[:2] - length = h2i(enc[2:4]) + length = h2i(enc[2:4])[0] if time_unit == '04': return length * 10*24*60*60 elif time_unit == '03': -- To view, visit https://gerrit.osmocom.org/c/pysim/+/33208 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I981e9d46607193176b28cb574564e6da546501ba Gerrit-Change-Number: 33208 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge@osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier@sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-MessageType: merged