Skip to content

AMF sends in-valid GUTI in registration accept #645

Merged
gab-arrobo merged 2 commits intoomec-project:mainfrom
ashithacdac:invalid-guti-fix
Mar 31, 2026
Merged

AMF sends in-valid GUTI in registration accept #645
gab-arrobo merged 2 commits intoomec-project:mainfrom
ashithacdac:invalid-guti-fix

Conversation

@ashithacdac
Copy link
Copy Markdown
Contributor

Issue:
AMF sends an invalid GUTI in registration accept.

RCA:
During troubleshooting, it was observed that when the UE initiates a Registration Request with an invalid GUTI, the core attempts to fetch the AMF UE context using the received GUTI. Since the GUTI is invalid, the UE cannot be found in the core. As a result, the core creates a new AMF UE context and allocates a new valid GUTI for the UE.

However, during the handling of the Registration Request, when the IEs are decoded, the invalid GUTI received from the UE is set again to the UE context. This incorrectly replaces the valid GUTI that was newly assigned by the core. As a result, the core sends the invalid GUTI in the Registration Accept.

Fix:
Update GUTI only when GUAMI is valid.

Supporting Data:
Invalid-guti.zip

@ashithacdac ashithacdac requested a review from a team March 19, 2026 06:36
@gab-arrobo gab-arrobo requested a review from Copilot March 19, 2026 13:09
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses an issue where an invalid UE-provided 5G-GUTI could overwrite a newly allocated (valid) ue.Guti, causing the AMF to send an invalid GUTI in Registration Accept.

Changes:

  • Only updates ue.Guti from the decoded Registration Request when the GUAMI derived from the UE’s GUTI matches the AMF’s served GUAMI.
  • Adds a warning log when the received GUAMI does not match.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@gab-arrobo
Copy link
Copy Markdown
Contributor

@ashithacdac,
Thanks for your PR. Please rebase it and address Copilot's comments (if applicable)

@ashithacdac
Copy link
Copy Markdown
Contributor Author

Changes have been incorporated, and the PR has been rebased.

Signed-off-by: AshithaCDAC <ashitha@cdac.in>
Signed-off-by: AshithaCDAC <ashitha@cdac.in>
@gab-arrobo gab-arrobo merged commit d29629a into omec-project:main Mar 31, 2026
12 checks passed
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.

3 participants