laforge has submitted this change. (
https://gerrit.osmocom.org/c/pysim/+/31125 )
Change subject: ts_51_011: Fix decoding/encoding of EF_LOCIGPRS
......................................................................
ts_51_011: Fix decoding/encoding of EF_LOCIGPRS
The P-TMSI signature is a 3-byte value, not a 1-byte value.
Change-Id: I06e8d3efe0b3cf3970159c913acfd2f72280302d
---
M pySim/ts_51_011.py
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py
index 4d819dd..80182af 100644
--- a/pySim/ts_51_011.py
+++ b/pySim/ts_51_011.py
@@ -824,7 +824,8 @@
class EF_LOCIGPRS(TransparentEF):
def __init__(self, fid='6f53', sfid=None, name='EF.LOCIGPRS',
desc='GPRS Location Information', size=(14, 14)):
super().__init__(fid, sfid=sfid, name=name, desc=desc, size=size)
- self._construct = Struct('ptmsi'/HexAdapter(Bytes(4)),
'ptmsi_sig'/Int8ub, 'rai'/HexAdapter(Bytes(6)),
+ self._construct = Struct('ptmsi'/HexAdapter(Bytes(4)),
'ptmsi_sig'/HexAdapter(Bytes(3)),
+ 'rai'/HexAdapter(Bytes(6)),
'rau_status'/Enum(Byte, updated=0,
not_updated=1, plmn_not_allowed=2,
routing_area_not_allowed=3))
--
To view, visit
https://gerrit.osmocom.org/c/pysim/+/31125
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I06e8d3efe0b3cf3970159c913acfd2f72280302d
Gerrit-Change-Number: 31125
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged