From 51da6263b77dd8c1325b44dbc31e4ac46c9fd1da Mon Sep 17 00:00:00 2001 From: Harald Welte Date: Tue, 13 Jan 2026 23:11:43 +0100 Subject: [PATCH] Fix esim.saip.ProfileElementSequence.remove_naas_of_type This method did not work at all at the moment, likely due to API churn over time. This change makes the following exception go away: Traceback (most recent call last): File "projects/git/pysim/contrib/saip-tool.py", line 473, in do_remove_naa(pes, opts) ~~~~~~~~~~~~~^^^^^^^^^^^ File "projects/git/pysim/contrib/saip-tool.py", line 203, in do_remove_naa pes.remove_naas_of_type(naa) ~~~~~~~~~~~~~~~~~~~~~~~^^^^^ File "projects/git/pysim/contrib/pySim/esim/saip/__init__.py", line 1748, in remove_naas_of_type if template in hdr.decoded['eUICC-Mandatory-GFSTEList']: ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "projects/git/pysim/contrib/pySim/esim/saip/oid.py", line 48, in __eq__ return (self.intlist == other.intlist) ^^^^^^^^^^^^^ AttributeError: 'str' object has no attribute 'intlist' A subsequent patch should introduce unit tests to avoid such breakage in the future. Change-Id: I88d862d751198c3d1648ab7f11d6e6a8fdbc41c9 --- pySim/esim/saip/__init__.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pySim/esim/saip/__init__.py b/pySim/esim/saip/__init__.py index 5ca87aa8..6b6c01cf 100644 --- a/pySim/esim/saip/__init__.py +++ b/pySim/esim/saip/__init__.py @@ -1751,8 +1751,7 @@ class ProfileElementSequence: del hdr.decoded['eUICC-Mandatory-services'][service] # remove any associated mandatory filesystem templates for template in naa.templates: - if template in hdr.decoded['eUICC-Mandatory-GFSTEList']: - hdr.decoded['eUICC-Mandatory-GFSTEList'] = [x for x in hdr.decoded['eUICC-Mandatory-GFSTEList'] if not template.prefix_match(x)] + hdr.decoded['eUICC-Mandatory-GFSTEList'] = [x for x in hdr.decoded['eUICC-Mandatory-GFSTEList'] if not template.prefix_match(x)] # determine the ADF names (AIDs) of all NAA ADFs naa_adf_names = [] if naa.pe_types[0] in self.pe_by_type: