laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/33850 ) Change subject: pySim-shell: fix verify_adm command ...................................................................... pySim-shell: fix verify_adm command The comman verify_adm does no longer work since the verify_adm method is no longer available in the card base classes (cards.py). Let's use the verify_chv method from SimCardCommands instead. Change-Id: Ic87e1bff221b10d33d36da32b589e2737f6ca9cd --- M pySim-shell.py 1 file changed, 14 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/pySim-shell.py b/pySim-shell.py index 3b56bfd..2e332d3 100755 --- a/pySim-shell.py +++ b/pySim-shell.py @@ -781,7 +781,7 @@ "cannot find ADM-PIN for ICCID '%s'" % (self._cmd.iccid)) if pin_adm: - self._cmd.card.verify_adm(h2b(pin_adm)) + self._cmd.card._scc.verify_chv(self._cmd.card._adm_chv_num, h2b(pin_adm)) else: raise ValueError("error: cannot authenticate, no adm-pin!") -- To view, visit https://gerrit.osmocom.org/c/pysim/+/33850 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ic87e1bff221b10d33d36da32b589e2737f6ca9cd Gerrit-Change-Number: 33850 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-MessageType: merged