Change in pysim[master]: fix: Decoder may raise KeyError

[laforge]

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


Change subject: fix: Decoder may raise KeyError
......................................................................

fix: Decoder may raise KeyError

This fixes an issue where a KeyError may be raised when 'A5'
is not present in `fcp`

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



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/70/26370/1

diff --git a/pySim/ts_102_221.py b/pySim/ts_102_221.py
index adb40c1..f145d19 100644
--- a/pySim/ts_102_221.py
+++ b/pySim/ts_102_221.py
@@ -684,7 +684,7 @@
         fcp_tlv = TLV(FCP_TLV_MAP)
         fcp = fcp_tlv.parse(fcp_base['62'])
         # further decode the proprietary information
-        if fcp['A5']:
+        if 'A5' in fcp:
             prop_tlv = TLV(FCP_Proprietary_TLV_MAP)
             prop = prop_tlv.parse(fcp['A5'])
             fcp['A5'] = tlv_val_interpret(FCP_prorietary_interpreter_map, prop)

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

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I5bb6131bd76c7bae2a70034c429cae2b380d164f
Gerrit-Change-Number: 26370
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211125/aad93b38/attachment.htm>