July 2026 - gerrit-log - lists.osmocom.org

[XS] Change in onomondo-eim[master]: esipa_asn1_handler_utils: use eimTransactionId in EuiccPackageErrorUn...
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43015?usp=email ) Change subject: esipa_asn1_handler_utils: use eimTransactionId in EuiccPackageErrorUnsigned ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43015?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Ibfbc6debcd36d2aae1f4530a608db823caf3a394 Gerrit-Change-Number: 43015 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:45:17 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in onomondo-eim[master]: restop.py, tryme_*.sh: improve log output and handling
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43014?usp=email ) Change subject: restop.py, tryme_*.sh: improve log output and handling ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43014?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Id257e7a541fc92386cecee5af3dad2077b62709c Gerrit-Change-Number: 43014 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:44:57 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-eim[master]: esipa_asn1_handler: store IPAd stateChangeCause
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43013?usp=email ) Change subject: esipa_asn1_handler: store IPAd stateChangeCause ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43013?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: I3fe64687786d9184ae82cbb12f9d1292fdc945c8 Gerrit-Change-Number: 43013 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:44:29 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in onomondo-eim[master]: doc/build.md: fix typo
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/42994?usp=email ) Change subject: doc/build.md: fix typo ...................................................................... Patch Set 3: (1 comment) Patchset: PS3: it's not really a typo, but a question whether you'd want american or british english? -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42994?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Ie6a066f3289fbdde40f55bd9f38a3dd90102e7e6 Gerrit-Change-Number: 42994 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:43:53 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[L] Change in onomondo-eim[master]: mnesia_db_euicc: renovate handling of euicc table (mnesia, REST)
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/42993?usp=email ) Change subject: mnesia_db_euicc: renovate handling of euicc table (mnesia, REST) ...................................................................... Patch Set 3: (1 comment) Patchset: PS3: as with other patches changing the JSON schema: shouldn't there be some kind of version in it, and that version being bumped before making incompatible changes? So that any user of the JSON interface could at runtime inquire about the schema version used by the eIM and decide if it can work with that version or raise an error? -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42993?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: I74c602555b194a28d6eb9dd67ec4c6a8865fcb77 Gerrit-Change-Number: 42993 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:42:58 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[L] Change in onomondo-ipa[master]: V1.2: Add new fields to ProfileInfo list
by Jenkins Builder 16 Jul '26

16 Jul '26

Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43032?usp=email ) Change subject: V1.2: Add new fields to ProfileInfo list ...................................................................... Patch Set 1: (3 comments) File src/ipa/libipa/es10c_get_prfle_info.c: Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43032/comment/2247934f_d9252347… : PS1, Line 78: if (conv_req && conv_req->fallback_attribute_removed) { braces {} are not necessary for single statement blocks Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43032/comment/8dc9f3f6_a02c5a59… : PS1, Line 109: struct ipa_es10c_get_prfle_info_req *conv_get_prfle_info_req(const struct ipa_es10c_get_prfle_info_req *req) { open brace '{' following function definitions go on the next line Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43032/comment/d5fb15a4_452e81f6… : PS1, Line 110: uint8_t default_tags[] = { 0x5A, 0x4F, 0x9F, 0x70, 0x91, 0x92, 0x95, 0x9F,0x7B, 0x9F,0x26, 0x9F,0x67 }; space required after that ',' (ctx:VxV) -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43032?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I6667dba8dd84817cfee69daa1c44b98239fa171c Gerrit-Change-Number: 43032 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Thu, 16 Jul 2026 14:23:21 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in onomondo-ipa[master]: V1.2: Add EimConfigurationData.indirectProfileDownload
by Jenkins Builder 16 Jul '26

16 Jul '26

Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43029?usp=email ) Change subject: V1.2: Add EimConfigurationData.indirectProfileDownload ...................................................................... Patch Set 1: (1 comment) File src/ipa/libipa/proc_eim_pkg_retr.c: Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43029/comment/d2cfef33_d353eb21… : PS1, Line 122: "Indirect profile download is not suppoted by eIM, we don't support direct profile download, -- cannot continue!\n"); 'suppoted' may be misspelled - perhaps 'supported'? -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43029?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: Ic40111531e6f35d5ea7f3789de6cae0f1c0d5ba7 Gerrit-Change-Number: 43029 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Thu, 16 Jul 2026 14:22:58 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in onomondo-ipa[master]: V1.2: Add function to set Device Capabilities
by Jenkins Builder 16 Jul '26

16 Jul '26

Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email ) Change subject: V1.2: Add function to set Device Capabilities ...................................................................... Patch Set 1: (2 comments) File include/onomondo/ipa/ipad.h: Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43060/comment/c3823fc2_cabc9083… : PS1, Line 90: /*! Set the Device Capabilites. 'Capabilites' may be misspelled - perhaps 'Capabilities'? File src/ipa/main.c: Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43060/comment/b2f7a23b_6e5266b4… : PS1, Line 67: printf(" -D ......................... set SGP.22 device capabilites other than the 3 that are required by this IPAd\n"); 'capabilites' may be misspelled - perhaps 'capabilities'? -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I866787b8cda9a87a9f2e0db4965fbb554d339a6c Gerrit-Change-Number: 43060 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Thu, 16 Jul 2026 14:22:12 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in onomondo-ipa[master]: V1.2: Add TERMINAL CAPABILITY for IoT support
by Jenkins Builder 16 Jul '26

16 Jul '26

Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43047?usp=email ) Change subject: V1.2: Add TERMINAL CAPABILITY for IoT support ...................................................................... Patch Set 1: (2 comments) File src/ipa/libipa/euicc.c: Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43047/comment/fbd21064_30159478… : PS1, Line 539: /* Set capabilites to enable support for eUICC functions. */ 'capabilites' may be misspelled - perhaps 'capabilities'? Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/onomondo-ipa/+/43047/comment/b2eb8cae_9c3fe749… : PS1, Line 544: /* Set capabilites to enable support for IoT functions. */ 'capabilites' may be misspelled - perhaps 'capabilities'? -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43047?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I1116bee93b31ee4a159125d840d7c5866be6fdb4 Gerrit-Change-Number: 43047 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Thu, 16 Jul 2026 14:21:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in onomondo-ipa[master]: V1.2: Cange remaining ASN1 definition to SGP.32 V1.2
by jolly 16 Jul '26

16 Jul '26

jolly has uploaded this change for review. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43073?usp=email ) Change subject: V1.2: Cange remaining ASN1 definition to SGP.32 V1.2 ...................................................................... V1.2: Cange remaining ASN1 definition to SGP.32 V1.2 These changes are cosmetic or have no impact on functionality. Related: SYS#8101 Change-Id: Ifa2546784a6d75170e4f5aae2a5e98edb2c0627a --- M asn1/SGP32Definitions.asn M src/ipa/libasn/EuiccPackageErrorCode.h M src/ipa/libasn/EuiccPackageErrorDataSigned.c M src/ipa/libasn/EuiccPackageErrorDataSigned.h M src/ipa/libasn/EuiccPackageSigned.c 5 files changed, 87 insertions(+), 40 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-ipa refs/changes/73/43073/1 diff --git a/asn1/SGP32Definitions.asn b/asn1/SGP32Definitions.asn index b9746ea..876dc3d 100644 --- a/asn1/SGP32Definitions.asn +++ b/asn1/SGP32Definitions.asn @@ -30,7 +30,7 @@ eimSignature [APPLICATION 55] OCTET STRING -- Tag '5F37' } EuiccPackageSigned ::= SEQUENCE { - eimId [0] UTF8String, + eimId [0] UTF8String (SIZE(1..128)), eidValue [APPLICATION 26] Octet16, -- Tag '5A' counterValue [1] INTEGER, eimTransactionId [2] TransactionId OPTIONAL, @@ -47,7 +47,7 @@ -- ASN1START EimConfigurationData ::= SEQUENCE { eimId [0] UTF8String (SIZE(1..128)), -- eimId of eIM - eimFqdn [1] UTF8String OPTIONAL, -- FQDN of eIM + eimFqdn [1] UTF8String OPTIONAL, -- FQDN of eIM or intermediate server, if used eimIdType [2] EimIdType OPTIONAL, counterValue [3] INTEGER OPTIONAL, -- initial counterValue for the eIM associationToken [4] INTEGER OPTIONAL, @@ -61,7 +61,7 @@ } OPTIONAL, eimSupportedProtocol [7] EimSupportedProtocol OPTIONAL, euiccCiPKId [8] SubjectKeyIdentifier OPTIONAL, -- CI Public Key Identifier supported on the eUICC for signature creation - indirectProfileDownload[9] NULL OPTIONAL -- support of Indirect Profile download with indicated EimSupportedProtocol + indirectProfileDownload [9] NULL OPTIONAL -- support of Indirect Profile download with indicated EimSupportedProtocol } EimIdType ::= INTEGER { eimIdTypeOid(1), @@ -113,7 +113,7 @@ -- Section 2.11.1.2 -- ASN1START IpaEuiccDataRequest ::= [82] SEQUENCE { -- Tag BF52 - tagList [APPLICATION 28] OCTET STRING, -- Tag '5C' + tagList [APPLICATION 28] OCTET STRING, -- Tag '5C' euiccCiPKIdentifierToBeUsed OCTET STRING OPTIONAL, -- CI Public Key Identifier (possibly truncated) supported on the eUICC for signature creation searchCriteriaNotification [1] CHOICE { seqNumber [0] INTEGER, @@ -195,10 +195,17 @@ EuiccPackageErrorDataSigned ::= SEQUENCE { eimId [0] UTF8String (SIZE(1..128)), counterValue [1] INTEGER, - transactionId [2] TransactionId OPTIONAL, + eimTransactionId [2] TransactionId OPTIONAL, euiccPackageErrorCode EuiccPackageErrorCode } -EuiccPackageErrorCode ::= INTEGER { invalidEid(3), replayError(4), counterValueOutOfRange(6), sizeOverflow(15), undefinedError(127)} +EuiccPackageErrorCode ::= INTEGER { + invalidEid(3), + replayError(4), + counterValueOutOfRange(6), + sizeOverflow(15), + ecallActive(104), + undefinedError(127) +} EuiccPackageErrorUnsigned ::= SEQUENCE { eimId [0] UTF8String (SIZE(1..128)), eimTransactionId [2] TransactionId OPTIONAL, @@ -369,8 +376,8 @@ activateIpae(0) -- IPAe activation } } --- ASN1START -- ASN1STOP +-- ASN1START IpaeActivationResponse ::= [66] SEQUENCE { -- Tag 'BF42' ipaeActivationResult INTEGER {ok(0), notSupported(1)} } @@ -501,8 +508,8 @@ } -- Definition of IpaMode IpaMode ::= INTEGER { -ipad (0), -- IPAd is active -ipae (1) -- IPAe is active + ipad (0), -- IPAd is active + ipae (1) -- IPAe is active } -- ASN1STOP @@ -536,20 +543,20 @@ -- (added manually since the PDF document lacks the ASN1START/ASN1STOP token for this function, which aused problems -- during the automatic parsing of the PDF document.) SGP32-EuiccMemoryResetRequest ::= [52] SEQUENCE { -- Tag 'BF34' - resetOptions [2] BIT STRING { - deleteOperationalProfiles(0), - deleteFieldLoadedTestProfiles(1), - resetDefaultSmdpAddress(2), - deletePreLoadedTestProfiles(3), - deleteProvisioningProfiles(4), - resetEimConfigData(5), - resetImmediateEnableConfig (6) - } + resetOptions [2] BIT STRING { + deleteOperationalProfiles(0), + deleteFieldLoadedTestProfiles(1), + resetDefaultSmdpAddress(2), + deletePreLoadedTestProfiles(3), + deleteProvisioningProfiles(4), + resetEimConfigData(5), + resetImmediateEnableConfig (6) + } } SGP32-EuiccMemoryResetResponse ::= [100] SEQUENCE { -- Tag 'BF64' - resetResult INTEGER {ok(0), nothingToDelete(1), catBusy(5), ecallActive(104), undefinedError(127)}, - resetEimResult INTEGER {ok(0), nothingToDelete(1), eimResetNotSupported(2), undefinedError(127)} OPTIONAL, - resetImmediateEnableConfigResult INTEGER {ok(0), resetIECNotSupported (1), undefinedError(127)} OPTIONAL + resetResult INTEGER {ok(0), nothingToDelete(1), catBusy(5), ecallActive(104), undefinedError(127)}, + resetEimResult INTEGER {ok(0), nothingToDelete(1), eimResetNotSupported(2), undefinedError(127)} OPTIONAL, + resetImmediateEnableConfigResult INTEGER {ok(0), resetIECNotSupported (1), undefinedError(127)} OPTIONAL } -- Section 5.9.10 @@ -631,8 +638,8 @@ } -- ASN1STOP -- ASN1START - ConfigureImmediateProfileEnablingResponse ::= [89] SEQUENCE { -- Tag 'BF59' - configImmediateEnableResult [0] INTEGER { +ConfigureImmediateProfileEnablingResponse ::= [89] SEQUENCE { -- Tag 'BF59' + configImmediateEnableResult [0] INTEGER { ok(0), insufficientMemory(1), associatedEimAlreadyExists(2), @@ -770,7 +777,7 @@ } CompactEuiccSigned2 ::= SEQUENCE { euiccOtpk [APPLICATION 73] OCTET STRING OPTIONAL, -- otPK.EUICC.ECKA, tag '5F49' euiccOtpk is always present except if bppEuiccOtpk was chosen by the eUICC - hashCc Octet32 OPTIONAL -- Hash of confirmation code, if not received from Eim + hashCc Octet32 OPTIONAL -- Hash of confirmation code, if not received from eIM } -- ASN1STOP @@ -779,9 +786,9 @@ SGP32-EuiccSigned1 ::= SEQUENCE { transactionId [0] TransactionId, serverAddress [3] UTF8String, - serverChallenge [4] Octet16, + serverChallenge [4] Octet16, -- The RSP Server Challenge euiccInfo2 [34] EUICCInfo2, - ctxParams1 CtxParams1 -- The RSP Server Challenge + ctxParams1 CtxParams1 } SGP32-AuthenticateResponseOk ::= SEQUENCE { euiccSigned1 SGP32-EuiccSigned1, -- Signed information @@ -811,6 +818,7 @@ -- Section 5.14.7 -- ASN1START +-- ASN1STOP SGP32-PendingNotification ::= CHOICE { profileInstallationResult [55] SGP32-ProfileInstallationResult, -- tag 'BF37' otherSignedNotification OtherSignedNotification, @@ -843,8 +851,6 @@ tbsOtherNotification NotificationMetadata, euiccNotificationSignature [APPLICATION 55] OCTET STRING -- eUICC signature of tbsOtherNotification, Tag '5F37' } --- ASN1STOP - -- Section 5.14.8 -- ASN1START SGP32-CancelSessionResponse ::= [65] CHOICE { -- Tag 'BF41' @@ -918,7 +924,7 @@ -- Section 6.3.2.2 -- ASN1START AuthenticateClientRequestEsipa ::= [59] SEQUENCE { -- Tag 'BF3B' - transactionId [0] TransactionId, + transactionId [0] TransactionId, -- The TransactionID generated by the SM-DP+/SM-DS authenticateServerResponse [56] SGP32-AuthenticateServerResponse -- This is the response from ES10b.AuthenticateServer, possibly in compact format } AuthenticateClientResponseEsipa ::= [59] CHOICE { -- Tag 'BF3B' @@ -942,7 +948,7 @@ } } AuthenticateClientOkDPEsipa ::= SEQUENCE { - transactionId [0] TransactionId OPTIONAL, + transactionId [0] TransactionId OPTIONAL, -- The TransactionID generated by the SM-DP+ profileMetaData [37] SGP32-StoreMetadataRequest OPTIONAL, smdpSigned2 SmdpSigned2, -- Signed information smdpSignature2 [APPLICATION 55] OCTET STRING, -- Tag '5F37' @@ -950,7 +956,7 @@ hashCc Octet32 OPTIONAL -- Hash of confirmation code } AuthenticateClientOkDSEsipa ::= SEQUENCE { - transactionId [0] TransactionId, + transactionId [0] TransactionId, -- The TransactionID generated by the SM-DS profileDownloadTrigger [84] ProfileDownloadTriggerRequest OPTIONAL -- Tag 'BF54' } -- ASN1STOP @@ -958,7 +964,7 @@ -- Section 6.3.2.3 -- ASN1START GetBoundProfilePackageRequestEsipa ::= [58] SEQUENCE { -- Tag 'BF3A' - transactionId [0] TransactionId, + transactionId [0] TransactionId, -- The TransactionID generated by the SM-DP+ prepareDownloadResponse [33] SGP32-PrepareDownloadResponse -- This is the response from ES10b.PrepareDownload, possibly in compact format } GetBoundProfilePackageResponseEsipa ::= [58] CHOICE { -- Tag 'BF3A' @@ -976,7 +982,7 @@ } } GetBoundProfilePackageOkEsipa ::= SEQUENCE { - transactionId [0] TransactionId OPTIONAL, + transactionId [0] TransactionId OPTIONAL, -- The TransactionID generated by the SM-DP+ boundProfilePackage [54] BoundProfilePackage } -- ASN1STOP @@ -992,8 +998,8 @@ -- Section 6.3.2.5 -- ASN1START CancelSessionRequestEsipa ::= [65] SEQUENCE { -- Tag 'BF41' - transactionId TransactionId, - cancelSessionResponse SGP32-CancelSessionResponse -- This is the response from ES10b. CancelSession function, possibly in compact format + transactionId TransactionId, -- The TransactionID generated by the SM-DP+ + cancelSessionResponse SGP32-CancelSessionResponse -- This is the response from ES10b. CancelSession function, possibly in compact format } CancelSessionResponseEsipa ::= [65] CHOICE { -- Tag 'BF41' cancelSessionOk SGP32-CancelSessionOk, diff --git a/src/ipa/libasn/EuiccPackageErrorCode.h b/src/ipa/libasn/EuiccPackageErrorCode.h index 94382d6..9827660 100644 --- a/src/ipa/libasn/EuiccPackageErrorCode.h +++ b/src/ipa/libasn/EuiccPackageErrorCode.h @@ -24,6 +24,7 @@ EuiccPackageErrorCode_replayError = 4, EuiccPackageErrorCode_counterValueOutOfRange = 6, EuiccPackageErrorCode_sizeOverflow = 15, + EuiccPackageErrorCode_ecallActive = 104, EuiccPackageErrorCode_undefinedError = 127 } e_EuiccPackageErrorCode; diff --git a/src/ipa/libasn/EuiccPackageErrorDataSigned.c b/src/ipa/libasn/EuiccPackageErrorDataSigned.c index 59a6925..44e4a4c 100644 --- a/src/ipa/libasn/EuiccPackageErrorDataSigned.c +++ b/src/ipa/libasn/EuiccPackageErrorDataSigned.c @@ -66,14 +66,14 @@ 0, 0, /* No default value */ "counterValue" }, - { ATF_POINTER, 1, offsetof(struct EuiccPackageErrorDataSigned, transactionId), + { ATF_POINTER, 1, offsetof(struct EuiccPackageErrorDataSigned, eimTransactionId), (ASN_TAG_CLASS_CONTEXT | (2 << 2)), -1, /* IMPLICIT tag at current level */ &asn_DEF_TransactionId, 0, { 0, 0, 0 }, 0, 0, /* No default value */ - "transactionId" + "eimTransactionId" }, { ATF_NOFLAGS, 0, offsetof(struct EuiccPackageErrorDataSigned, euiccPackageErrorCode), (ASN_TAG_CLASS_UNIVERSAL | (2 << 2)), @@ -93,7 +93,7 @@ { (ASN_TAG_CLASS_UNIVERSAL | (2 << 2)), 3, 0, 0 }, /* euiccPackageErrorCode */ { (ASN_TAG_CLASS_CONTEXT | (0 << 2)), 0, 0, 0 }, /* eimId */ { (ASN_TAG_CLASS_CONTEXT | (1 << 2)), 1, 0, 0 }, /* counterValue */ - { (ASN_TAG_CLASS_CONTEXT | (2 << 2)), 2, 0, 0 } /* transactionId */ + { (ASN_TAG_CLASS_CONTEXT | (2 << 2)), 2, 0, 0 } /* eimTransactionId */ }; asn_SEQUENCE_specifics_t asn_SPC_EuiccPackageErrorDataSigned_specs_1 = { sizeof(struct EuiccPackageErrorDataSigned), diff --git a/src/ipa/libasn/EuiccPackageErrorDataSigned.h b/src/ipa/libasn/EuiccPackageErrorDataSigned.h index 81d22f3..b27aca4 100644 --- a/src/ipa/libasn/EuiccPackageErrorDataSigned.h +++ b/src/ipa/libasn/EuiccPackageErrorDataSigned.h @@ -26,7 +26,7 @@ typedef struct EuiccPackageErrorDataSigned { UTF8String_t eimId; long counterValue; - TransactionId_t *transactionId /* OPTIONAL */; + TransactionId_t *eimTransactionId /* OPTIONAL */; EuiccPackageErrorCode_t euiccPackageErrorCode; /* * This type is extensible, diff --git a/src/ipa/libasn/EuiccPackageSigned.c b/src/ipa/libasn/EuiccPackageSigned.c index abc9109..1c4cd5e 100644 --- a/src/ipa/libasn/EuiccPackageSigned.c +++ b/src/ipa/libasn/EuiccPackageSigned.c @@ -7,13 +7,53 @@ #include "EuiccPackageSigned.h" +static int +memb_eimId_constraint_1(const asn_TYPE_descriptor_t *td, const void *sptr, + asn_app_constraint_failed_f *ctfailcb, void *app_key) { + const UTF8String_t *st = (const UTF8String_t *)sptr; + size_t size; + + if(!sptr) { + ASN__CTFAIL(app_key, td, sptr, + "%s: value not given (%s:%d)", + td->name, __FILE__, __LINE__); + return -1; + } + + size = UTF8String_length(st); + if((ssize_t)size < 0) { + ASN__CTFAIL(app_key, td, sptr, + "%s: UTF-8: broken encoding (%s:%d)", + td->name, __FILE__, __LINE__); + return -1; + } + + if((size >= 1 && size <= 128)) { + /* Constraint check succeeded */ + return 0; + } else { + ASN__CTFAIL(app_key, td, sptr, + "%s: constraint failed (%s:%d)", + td->name, __FILE__, __LINE__); + return -1; + } +} + +static asn_oer_constraints_t asn_OER_memb_eimId_constr_2 CC_NOTUSED = { + { 0, 0 }, + -1 /* (SIZE(0..MAX)) */}; +static asn_per_constraints_t asn_PER_memb_eimId_constr_2 CC_NOTUSED = { + { APC_UNCONSTRAINED, -1, -1, 0, 0 }, + { APC_UNCONSTRAINED, -1, -1, 0, 0 }, + 0, 0 /* No PER value map */ +}; asn_TYPE_member_t asn_MBR_EuiccPackageSigned_1[] = { { ATF_NOFLAGS, 0, offsetof(struct EuiccPackageSigned, eimId), (ASN_TAG_CLASS_CONTEXT | (0 << 2)), -1, /* IMPLICIT tag at current level */ &asn_DEF_UTF8String, 0, - { 0, 0, 0 }, + { &asn_OER_memb_eimId_constr_2, &asn_PER_memb_eimId_constr_2, memb_eimId_constraint_1 }, 0, 0, /* No default value */ "eimId" }, -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43073?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: Ifa2546784a6d75170e4f5aae2a5e98edb2c0627a Gerrit-Change-Number: 43073 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu>

1 0

2026

2025

2024

2023

2022

gerrit-log July 2026