Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[lte][agw] Add condition for context not found in delete session response #3878

Merged
merged 2 commits into from
Dec 3, 2020

Conversation

ulaskozat
Copy link
Contributor

Signed-off-by: Ulas Kozat kozat@fb.com

Summary

When context is not found in SPGW task, session request response does not include any LBI information; instead a specific CONTEXT_NOT_FOUND cause is returned back to MME. As MME code was not considering this situation, it was leading to a crash while trying to access the session context.

Test Plan

Sanity tests.
Also tested with a modified MME binary to trigger the case.

Signed-off-by: Ulas Kozat <kozat@fb.com>
@ulaskozat ulaskozat added component: agw Access gateway-related issue apply-v1.2 Needs to be applied to v1.2 release branch as well apply-v1.3 Needs to be applied to v1.3 release branch as well apply-v1.1 Needs to be applied to v1.1 release branch as well labels Dec 3, 2020
@ulaskozat ulaskozat requested a review from ssanadhya as a code owner December 3, 2020 05:45
Signed-off-by: Ulas Kozat <kozat@fb.com>
@ulaskozat ulaskozat removed apply-v1.1 Needs to be applied to v1.1 release branch as well apply-v1.2 Needs to be applied to v1.2 release branch as well labels Dec 3, 2020
@ulaskozat ulaskozat merged commit ab0c752 into magma:master Dec 3, 2020
@themarwhal themarwhal added the backported-v1.3 Has been backported to v1.3 release branch label Dec 3, 2020
themarwhal pushed a commit that referenced this pull request Dec 3, 2020
…onse (#3878)

* Add condition for context not found in delete session response

Signed-off-by: Ulas Kozat <kozat@fb.com>
jaredmullane pushed a commit to jaredmullane/magma that referenced this pull request Dec 3, 2020
…onse (magma#3878)

* Add condition for context not found in delete session response

Signed-off-by: Ulas Kozat <kozat@fb.com>
Signed-off-by: Jared Mullane <jmullane@fb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apply-v1.3 Needs to be applied to v1.3 release branch as well backported-v1.3 Has been backported to v1.3 release branch component: agw Access gateway-related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants