Change in pysim[master]: ts_51_011: fix encoder of EF.CNL

dexter gerrit-no-reply at lists.osmocom.org
Fri Apr 30 15:05:54 UTC 2021


dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/24000 )


Change subject: ts_51_011: fix encoder of EF.CNL
......................................................................

ts_51_011: fix encoder of EF.CNL

The encoder of EF.CNL is using "res" to reference some fields in the
input dictionary, this is wrong, it should use "in_json" instead

Change-Id: I6b07c91042ae3a87c561671689ec236918254ddf
---
M pySim/ts_51_011.py
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/00/24000/1

diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py
index 15a7907..14be6dc 100644
--- a/pySim/ts_51_011.py
+++ b/pySim/ts_51_011.py
@@ -649,8 +649,8 @@
         return res
     def _encode_record_hex(self, in_json):
         plmn = enc_plmn(in_json['mcc'], in_json['mnc'])
-        return b2h(pack('!3sBBB', h2b(plmn), res['network_subset'], res['service_provider_id'],
-                        res['corporate_id']))
+        return b2h(pack('!3sBBB', h2b(plmn), in_json['network_subset'], in_json['service_provider_id'],
+                        in_json['corporate_id']))
 
 # TS 51.011 Section 10.3.31
 class EF_NIA(LinFixedEF):

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

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I6b07c91042ae3a87c561671689ec236918254ddf
Gerrit-Change-Number: 24000
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210430/9e1d4fc8/attachment.htm>


More information about the gerrit-log mailing list