Skip to content

Conversation

@mnowak2-onepin-com
Copy link

…is ALGORITHM_KNOWN_BY_BOTH_ENTITIES

When KiC coding is set to x4, the card profile verification fails with the following exception:

Caused by: org.opentelecoms.gsm0348.api.PacketBuilderConfigurationException: KIC CipheringAlgorithmMode cannot be null for ciphered command
at org.opentelecoms.gsm0348.impl.PacketBuilderImpl.verifyProfile(PacketBuilderImpl.java:136)
at org.opentelecoms.gsm0348.impl.PacketBuilderImpl.setProfile(PacketBuilderImpl.java:296)
at org.opentelecoms.gsm0348.impl.PacketBuilderImpl.(PacketBuilderImpl.java:87)

The ciphering algorithm mode should not be checked for null if an explicit ciphering algorithm is specified.

…is ALGORITHM_KNOWN_BY_BOTH_ENTITIES

When KiC coding is set to x4, the card profile verification fails with the following exception:

Caused by: org.opentelecoms.gsm0348.api.PacketBuilderConfigurationException: KIC CipheringAlgorithmMode cannot be null for ciphered command
	at org.opentelecoms.gsm0348.impl.PacketBuilderImpl.verifyProfile(PacketBuilderImpl.java:136)
	at org.opentelecoms.gsm0348.impl.PacketBuilderImpl.setProfile(PacketBuilderImpl.java:296)
	at org.opentelecoms.gsm0348.impl.PacketBuilderImpl.<init>(PacketBuilderImpl.java:87)

The ciphering algorithm mode should not be checked for null if an explicit ciphering algorithm is specified.
@mnowak2-onepin-com
Copy link
Author

mnowak2-onepin-com commented May 31, 2024

hello, there is a problem with KiC decoding and verification (when AlgorithmImplementation ALGORITHM_KNOWN_BY_BOTH_ENTITIES) the "mode" bits are undefined (per specification), and resulting mode is set to null (which is expected), but the card profile verification expects mode<>null.

Example: KiC coding set to 0x14 - keys set 1 and ALGORITHM_KNOWN_BY_BOTH_ENTITIES causes rg.opentelecoms.gsm0348.api.PacketBuilderConfigurationException: KIC CipheringAlgorithmMode cannot be null for ciphered command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant