laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/pysim/+/38962?usp=email )

Change subject: pySim-shell: reset card in method equip
......................................................................

pySim-shell: reset card in method equip

When the equip method is running, all kinds of states in pySim-shell are reset.
To be sure that the card state is also reset (normally this is the case because
usually init_card is called before equip), we should send an explicit reset to
the card as well.

Related: OS#6640
Change-Id: I622a2df2c9184841f72abd18483bfbfd00b2f464
---
M pySim-shell.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  osmith: Looks good to me, but someone else must approve
  Jenkins Builder: Verified
  laforge: Looks good to me, approved




diff --git a/pySim-shell.py b/pySim-shell.py
index 08f00f9..f2b3ad2 100755
--- a/pySim-shell.py
+++ b/pySim-shell.py
@@ -155,6 +155,7 @@
         # When a card object and a runtime state is present, (re)equip 
pySim-shell with everything that is
         # needed to operate on cards.
         if self.card and self.rs:
+            self.rs.reset()
             self.lchan = self.rs.lchan[0]
             self._onchange_conserve_write(
                 'conserve_write', False, self.conserve_write)

--
To view, visit https://gerrit.osmocom.org/c/pysim/+/38962?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I622a2df2c9184841f72abd18483bfbfd00b2f464
Gerrit-Change-Number: 38962
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>

Reply via email to