laforge submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: Jenkins Builder: Verified lynxis lazus: Looks good to me, approved
[cosmetic] fix typos in comments

Change-Id: I549ef7002e6ebef3f13af620cad8d03c7f4d891a
---
M osmo-smdpp.py
M pySim/esim/rsp.py
2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/osmo-smdpp.py b/osmo-smdpp.py
index d7fc872..9551396 100755
--- a/osmo-smdpp.py
+++ b/osmo-smdpp.py
@@ -444,7 +444,7 @@

ss.host_id = b'mahlzeit'

- # Generate Session Keys using the CRT, opPK.eUICC.ECKA and otSK.DP.ECKA according to annex G
+ # Generate Session Keys using the CRT, otPK.eUICC.ECKA and otSK.DP.ECKA according to annex G
euicc_public_key = ec.EllipticCurvePublicKey.from_encoded_point(ss.smdp_ot.curve, ss.euicc_otpk)
ss.shared_secret = ss.smdp_ot.exchange(ec.ECDH(), euicc_public_key)
print("shared_secret: %s" % b2h(ss.shared_secret))
diff --git a/pySim/esim/rsp.py b/pySim/esim/rsp.py
index a032031..85dacee 100644
--- a/pySim/esim/rsp.py
+++ b/pySim/esim/rsp.py
@@ -37,7 +37,7 @@
def __init__(self, transactionId: str, serverChallenge: bytes, ci_cert_id: bytes):
self.transactionId = transactionId
self.serverChallenge = serverChallenge
- # used at a later point between API calsl
+ # used at a later point between API calls
self.ci_cert_id = ci_cert_id
self.euicc_cert: Optional[x509.Certificate] = None
self.eum_cert: Optional[x509.Certificate] = None

To view, visit change 36971. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I549ef7002e6ebef3f13af620cad8d03c7f4d891a
Gerrit-Change-Number: 36971
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-MessageType: merged