laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/36925?usp=email ) Change subject: pySim/commands: STATUS: Use indeterminate length Le/P3 == '00' ...................................................................... pySim/commands: STATUS: Use indeterminate length Le/P3 == '00' Let's have the card tell us what the length is by indicating '00' instead of stating 'FF'. This is better aligned with general practice and won't break assumptions in other parts of the code like SCP transport. Change-Id: Ied63c6e1970e3dfc675da5e5f94579fbb06fea51 --- M pySim/commands.py 1 file changed, 15 insertions(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/pySim/commands.py b/pySim/commands.py index 7b21faf..c0ac1d7 100644 --- a/pySim/commands.py +++ b/pySim/commands.py @@ -618,7 +618,7 @@ def status(self) -> ResTuple: """Execute a STATUS command as per TS 102 221 Section 11.1.2.""" - return self.send_apdu_checksw(self.cla4lchan('80') + 'F20000ff') + return self.send_apdu_checksw(self.cla4lchan('80') + 'F2000000') def deactivate_file(self) -> ResTuple: """Execute DECATIVATE FILE command as per TS 102 221 Section 11.1.14.""" -- To view, visit https://gerrit.osmocom.org/c/pysim/+/36925?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: Ied63c6e1970e3dfc675da5e5f94579fbb06fea51 Gerrit-Change-Number: 36925 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged