Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

sysInfo: Check BarringInfoList presence before unpacking #746

Merged
merged 1 commit into from
Feb 2, 2025

Conversation

Jojeker
Copy link
Contributor

@Jojeker Jojeker commented Jan 11, 2025

Otherwise, the UE crashes with a NULL Derference. This patch should introduce the check before accessing the barring bits.

From the specification 3GPP TS 38.331 the uac-BarringInfo is defined as optional:

uac-BarringInfo SEQUENCE {
      uac-BarringForCommon [UAC-BarringPerCatList] OPTIONAL, -- Need S
      uac-BarringPerPLMN-List [UAC-BarringPerPLMN-List] OPTIONAL, -- Need S
      uac-BarringInfoSetList [UAC-BarringInfoSetList],
      uac-AccessCategory1-SelectionAssistanceInfo CHOICE {
         plmnCommon [UAC-AccessCategory1-SelectionAssistanceInfo],
         individualPLMNList SEQUENCE (SIZE (2..[maxPLMN]) OF [UAC-AccessCategory1-SelectionAssistanceInfo]
         } OPTIONAL-- Need S
      } OPTIONAL, -- Need R

Cheers,
Eduard

Otherwise, the UE crashes with a NULL Derference. This patch should
introduce the check before accessing the barring bits.

Signed-off-by: Eduard Vlad <eduard.vlad@rwth-aachen.de>
@aligungr aligungr merged commit 7bf0cc9 into aligungr:master Feb 2, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants