laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
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(-)

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 change 27333. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I74bcf10b0a8977af0f2844044a812c5780af1706
Gerrit-Change-Number: 27333
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged