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."""