Change in pysim[master]: ts_102_221: fix fixup_fcp_proprietary_tlv_map()

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

dexter gerrit-no-reply at lists.osmocom.org
Thu Mar 11 16:47:34 UTC 2021


dexter has uploaded a new patch set (#3) to the change originally created by laforge. ( https://gerrit.osmocom.org/c/pysim/+/23211 )

Change subject: ts_102_221: fix fixup_fcp_proprietary_tlv_map()
......................................................................

ts_102_221: fix fixup_fcp_proprietary_tlv_map()

According to its description, the function
fixup_fcp_proprietary_tlv_map() is supposed to add propritary TLV tags
to satisfy the pyTlv parser, which can't handle unexpected tags.
However, the function adds tags but it does only fill up the gaps from
0xd0 onwards. This makes no sense, lets extended it so that it adds a
propritary tag for each tag that it is not already in the dictionary so
that we get a continious dictionary that starts at 0x00 and ends at 0xff

Change-Id: Ib06041361851f05dd990576fff47048dc1aa6290
Related: OS#4963
---
M pySim/ts_102_221.py
1 file changed, 4 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/11/23211/3
-- 
To view, visit https://gerrit.osmocom.org/c/pysim/+/23211
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ib06041361851f05dd990576fff47048dc1aa6290
Gerrit-Change-Number: 23211
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210311/b4e476c3/attachment.htm>


More information about the gerrit-log mailing list