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

Change subject: ts_31_102: Further decode TAI in EF.OPL5G
......................................................................

ts_31_102: Further decode TAI in EF.OPL5G

The TAI is not just an opaque bytestring but it consists of 3 fields.

Change-Id: Ie5a5ce74713deb0e151218ae553d3f3d96cef17d
---
M pySim/ts_31_102.py
1 file changed, 2 insertions(+), 1 deletion(-)

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



diff --git a/pySim/ts_31_102.py b/pySim/ts_31_102.py
index 231d387..fec81bf 100644
--- a/pySim/ts_31_102.py
+++ b/pySim/ts_31_102.py
@@ -1033,7 +1033,8 @@
 class EF_OPL5G(LinFixedEF):
     def __init__(self, fid='6f08', sfid=0x08, name='EF.OPL5G', desc='5GS 
Operator PLMN List', **kwargs):
         super().__init__(fid=fid, sfid=sfid, name=name, desc=desc, 
rec_len={10, None}, **kwargs)
-        self._construct = Struct('tai'/Bytes(9), 'pnn_record_id'/Int8ub)
+        Tai = Struct('mcc_mnc'/BcdAdapter(Bytes(3)), 'tac_min'/Bytes(3), 
'tac_max'/Bytes(3))
+        self._construct = Struct('tai'/Tai, 'pnn_record_id'/Int8ub)

 # TS 31.102 Section 4.4.11.10
 class EF_SUPI_NAI(TransparentEF):

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

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

Reply via email to