Attention is currently required from: laforge, pespin, dexter. Christian Amsüss has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/29033 )
Change subject: Add new pySim.ota library, implement SIM OTA crypto ...................................................................... Patch Set 5: (3 comments) Patchset: PS5: I've verified this module's encryption and decryption against what SJA2 and one other SIM implementation. Most comments are on an "it's not broken" level and could just as well be addressed later, but the PCNT should not get overlooked, so I'm leaving this at +1 until that is fixed to avoid accidental merging (if a +2 from me even carried the weight to make things mergable). File pySim/ota.py: https://gerrit.osmocom.org/c/pysim/+/29033/comment/2a3b12c8_25dcdd2e PS5, Line 402: def decode_resp(self, otak: OtaKeyset, spi: dict, data: bytes) -> bytes: `-> bytes` is currently inaccurate, should be `CompactRemoteResp` (but see below) https://gerrit.osmocom.org/c/pysim/+/29033/comment/ce9cbba8_295545de PS5, Line 452: return dec There will need to be an API change later here, as in cases of `res.response_status == 'actual_response_sms_submit'`, we'll rather need the res (and the CompactRemoteResp parsing fails in these cases, currently) rather than the dec. A concrete suggestion for a change is in I1c488a19ad75114255c8ff1c59d903388451b37a in the chrysn/for-29033 branch. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/29033 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I193ff4712c8503279c017b4b1324f0c3d38b9f84 Gerrit-Change-Number: 29033 Gerrit-PatchSet: 5 Gerrit-Owner: laforge <[email protected]> Gerrit-Reviewer: Christian Amsüss <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Attention: dexter <[email protected]> Gerrit-Comment-Date: Fri, 19 Aug 2022 20:50:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
