Skip to content

Dynamic Policy bugfix#197

Open
ErikGaida wants to merge 2 commits into
5G-MAG:developmentfrom
ErikGaida:bugfix/dynamic-policy
Open

Dynamic Policy bugfix#197
ErikGaida wants to merge 2 commits into
5G-MAG:developmentfrom
ErikGaida:bugfix/dynamic-policy

Conversation

@ErikGaida

Copy link
Copy Markdown
Contributor

Summary

Fixes a crash when creating a Dynamic Policy from a Policy Template that does not include a QoS specification.

Closes #189

@ErikGaida ErikGaida requested a review from davidjwbbc June 10, 2026 09:19
@ErikGaida ErikGaida added this to the v1.4.7 milestone Jun 10, 2026
@ErikGaida ErikGaida force-pushed the bugfix/dynamic-policy branch from 27a18a4 to 3e98158 Compare June 10, 2026 12:07
@jordijoangimenez

Copy link
Copy Markdown
Contributor

Reproducing the error accoring to the issue #189 now leads to 👍

application-function-1   | 06/12 09:57:56.974: [msaf] ERROR: Policy Templates are not available on this instance of the 5GMS Application Function. (../src/5gmsaf/msaf-m1-sm.c:636)
application-function-1   | 06/12 09:57:56.974: [msaf] ERROR: To allow Policy Templates please set open5gsIntegration to true in the configuration file and point the nrf section to a valid 5G core. (../src/5gmsaf/msaf-m1-sm.c:637)

The AF is not crashing

@jordijoangimenez

Copy link
Copy Markdown
Contributor

Now yes!

application-function-1   | 06/12 10:20:27.224: [msaf] INFO: policy template id: 54d6dc1a-6648-41f1-b5b4-7b5d81e9f9df (../src/5gmsaf/msaf-m1-sm.c:667)
application-function-1   | 06/12 10:20:27.224: [msaf] INFO: msaf_policy_template->policy_template->state: 1 (../src/5gmsaf/local.c:55)
application-function-1   | 06/12 10:20:27.224: [msaf] INFO: msaf_policy_template->policy_template->state: 1 (../src/5gmsaf/local.c:64)
application-function-1   | 06/12 10:20:27.224: [msaf] INFO: msaf_policy_template->policy_template->state: 3 (../src/5gmsaf/local.c:55)
application-function-1   | 06/12 10:20:27.224: [msaf] INFO: msaf_policy_template->policy_template->state: 3 (../src/5gmsaf/local.c:64)

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.

2 participants