Skip to content

Comments

fixed K_AMF derivation#1181

Closed
koh-osug wants to merge 1 commit intosrsran:masterfrom
koh-osug:master
Closed

fixed K_AMF derivation#1181
koh-osug wants to merge 1 commit intosrsran:masterfrom
koh-osug:master

Conversation

@koh-osug
Copy link
Contributor

@koh-osug koh-osug commented Jun 2, 2023

The K_AMF derivation for 5G is incorrect. According to ETSI TS 133 501 A.7 KAMF derivation function uses K_SEAF as input key.

@ismagom ismagom requested a review from alvasMan June 5, 2023 18:38
@CLAassistant
Copy link

CLAassistant commented Aug 25, 2024

CLA assistant check
All committers have signed the CLA.

@koh-osug
Copy link
Contributor Author

koh-osug commented Jan 6, 2026

will this be added now after 3 years?

@andrepuschmann
Copy link
Collaborator

Hey @koh-osug - apologies for the delay. As you might have guessed there is very little activity on the 5G UE side in this repo. I'll cherry-pick the PR locally and run through our (private) CI. If all is good it'll be merged and pushed upstream with the next official push.

@andrepuschmann
Copy link
Collaborator

@koh-osug Just seeing this is the patch for the PCSC USIM code. Have you tried this with a USIM against a commercial 5GC and can confirm it's working?

@koh-osug
Copy link
Contributor Author

koh-osug commented Jan 7, 2026

I have used a smartcard from sysmocom in a lab set up with commercial gNBs. But the specification part seems to be quite clear, and it was not working otherwise:

A.7.0 Parameters for the input S to the KDF
When deriving a KAMF from KSEAF the following parameters shall be used to form the input S to the KDF.

  • FC = 0x6D
  • P0 = SUPI
  • L0 = P0 length - number of octets in P0
  • P1 = ABBA parameter
  • L1 = P1 length - number of octets in P1
    The input key KEY shall be the 256-bit KSEAF.

@andrepuschmann
Copy link
Collaborator

Cherry-picked and put through internal CI. Thanks again

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.

4 participants