Change in pysim[master]: sysmocom_sja2: Properly decode EF.USIM_SQN freshness

[laforge]

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

Change subject: sysmocom_sja2: Properly decode EF.USIM_SQN freshness
......................................................................

sysmocom_sja2: Properly decode EF.USIM_SQN freshness

The freshness parameter is not one opaque bytestring, but an array
of 6-byte integer values.

before:
    "freshness": "000000000020000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000"

after:
    "freshness": [ 32, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 ]

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

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



diff --git a/pySim/sysmocom_sja2.py b/pySim/sysmocom_sja2.py
index 337eca7..263999d 100644
--- a/pySim/sysmocom_sja2.py
+++ b/pySim/sysmocom_sja2.py
@@ -166,7 +166,7 @@
                           'aus_concealed'/Bit, 'autn_concealed'/Bit)
         self._construct = Struct('flag1'/Flag1, 'flag2'/Flag2,
                                  'delta_max'/BytesInteger(6), 'age_limit'/BytesInteger(6),
-                                 'freshness'/HexAdapter(GreedyBytes))
+                                 'freshness'/GreedyRange(BytesInteger(6)))
 
 class EF_USIM_AUTH_KEY(TransparentEF):
     def __init__(self, fid='af20', name='EF.USIM_AUTH_KEY'):

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

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I53edbcdb40652eb0e2bc4c982cf7b6f7d585cab9
Gerrit-Change-Number: 25807
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211016/5eec8e98/attachment.htm>