Skip to content
Success

Changes

Summary

  1. pylint: personalization.py: fix E1135: permitted_len (details)
  2. pylint: ota.py: fix E0606 possibly-used-before-assignment (details)
  3. filesystem: do not decode short TransRecEF records (details)
  4. es8p: fix typo (details)
Commit 19e1330ce8856cc6e0e28aeed1da20dc0fe4efa8 by Neels Hofmeyr
pylint: personalization.py: fix E1135: permitted_len unsupported-membership-test

pre-empt this from coming up in patch
I60ea8fd11fb438ec90ddb08b17b658cbb789c051:

E1135: Value 'self.permitted_len' doesn't support membership test (unsupported-membership-test) pickermitted_len

Change-Id: I0343f8dbbffefb4237a1cb4dd40b576f16111073
The file was modifiedpySim/esim/saip/personalization.py
Commit cabb8edd53a0676902176f904e53ba7bb608f03b by Neels Hofmeyr
pylint: ota.py: fix E0606 possibly-used-before-assignment

************* Module pySim.ota
pySim/ota.py:430:24: E0606: Possibly using variable 'cpl' before assignment (possibly-used-before-assignment)

Change-Id: Ibbae851e458bbe7426a788b0784d553753c1056f
The file was modifiedpySim/ota.py
Commit 74be2e202fd8c503eb00a870519fdf5c14b39fb7 by laforge
filesystem: do not decode short TransRecEF records

A TransRecEF is based on a TransparentEF. This means that a TransRecEF
is basically normal TransparentEF that holds a record oriented data
structure. This also requires that the total length of the TransRecEF
is a multiple of the record length of the data structure that is stored
in it. When this is not the case, the last record will be cut short and
the decoding will fail. We should guard against this case.

Related: OS#6598
Change-Id: Ib1dc4d7ce306f1f0b080bb4b6abc36e72431d3fa
The file was modifiedpySim/filesystem.py
Commit 8da8b20f58f2fedbc0412a6741b12dbb2772cada by laforge
es8p: fix typo

Change-Id: I241efe0c7ceab190b7729a6d88101501ca37652e
The file was modifiedpySim/esim/es8p.py