laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/35650?usp=email ) Change subject: osmo-smdpp: Handle case where client sends empty list of Pki Cert IDs ...................................................................... osmo-smdpp: Handle case where client sends empty list of Pki Cert IDs In this case we don't want to run in an "undefined variable" exception, so let's initialize ci_cert to None. Change-Id: I4541d11dc6f660b59a1ec5225428ddde8401817d --- M osmo-smdpp.py 1 file changed, 13 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/50/35650/1 diff --git a/osmo-smdpp.py b/osmo-smdpp.py index 2d1cd66..6f437e6 100755 --- a/osmo-smdpp.py +++ b/osmo-smdpp.py @@ -215,6 +215,7 @@ if 'euiccCiPKIdListForSigningV3' in euiccInfo1: pkid_list = pkid_list + euiccInfo1['euiccCiPKIdListForSigningV3'] # verify it supports one of the keys indicated by euiccCiPKIdListForSigning + ci_cert = None for x in pkid_list: ci_cert = self.ci_get_cert_for_pkid(x) if ci_cert: -- To view, visit https://gerrit.osmocom.org/c/pysim/+/35650?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I4541d11dc6f660b59a1ec5225428ddde8401817d Gerrit-Change-Number: 35650 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange