laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27333 
)

Change subject: ts_51_011: Properly decode EF.OPL
......................................................................

ts_51_011: Properly decode EF.OPL

The OPL has 7 bytes "LAI" as the LAI actually contains a LAC
range (so two more bytes for the end of the 16bit range).

Change-Id: I74bcf10b0a8977af0f2844044a812c5780af1706
---
M pySim/ts_51_011.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py
index ddfed95..9a6aff9 100644
--- a/pySim/ts_51_011.py
+++ b/pySim/ts_51_011.py
@@ -889,7 +889,7 @@
 class EF_OPL(LinFixedEF):
     def __init__(self, fid='6fc6', sfid=None, name='EF.OPL', rec_len={8, 8}, 
desc='Operator PLMN List', **kwargs):
         super().__init__(fid, sfid=sfid, name=name, desc=desc, 
rec_len=rec_len, **kwargs)
-        self._construct = Struct('lai'/Bytes(5), 'pnn_record_id'/Int8ub)
+        self._construct = Struct('lai'/Bytes(7), 'pnn_record_id'/Int8ub)

 # TS 51.011 Section 10.3.44 + TS 31.102 4.2.62
 class EF_MBI(LinFixedEF):

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

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I74bcf10b0a8977af0f2844044a812c5780af1706
Gerrit-Change-Number: 27333
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged

Reply via email to