dexter has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/13478 )

Change subject: wavemobile-sim: write mnc-length field ine EF.AD
......................................................................

wavemobile-sim: write mnc-length field ine EF.AD

The length field in wavemobile sim cards is not set, so that the field
stays at its initial value, which is 0xFF. Lets write the correct mnc
length here.

Change-Id: Ieda0ce864bf3e8c7b92f062eaa3a5482c98507e2
Related: OS#3850
---
M pySim/cards.py
1 file changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved
  Vadim Yanitskiy: Looks good to me, but someone else must approve



diff --git a/pySim/cards.py b/pySim/cards.py
index d912a7a..a341b71 100644
--- a/pySim/cards.py
+++ b/pySim/cards.py
@@ -896,6 +896,12 @@
                        if sw != '9000':
                                print("Programming OPLMNwAcT failed with code 
%s"%sw)

+                # EF.AD
+                if p.get('mcc') and p.get('mnc'):
+                       sw = self.update_ad(p['mnc'])
+                       if sw != '9000':
+                               print("Programming AD failed with code %s"%sw)
+
                 return None

        def erase(self):

--
To view, visit https://gerrit.osmocom.org/13478
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ieda0ce864bf3e8c7b92f062eaa3a5482c98507e2
Gerrit-Change-Number: 13478
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Vadim Yanitskiy <axilira...@gmail.com>
Gerrit-Reviewer: dexter <pma...@sysmocom.de>

Reply via email to