Skip to content

fix(mme): Frees esmmessagecontainer on attach reject proc#8635

Merged
themarwhal merged 1 commit intomagma:v1.5from
ardzoht:backport_v15_free_esmmessagecontainer
Aug 13, 2021
Merged

fix(mme): Frees esmmessagecontainer on attach reject proc#8635
themarwhal merged 1 commit intomagma:v1.5from
ardzoht:backport_v15_free_esmmessagecontainer

Conversation

@ardzoht
Copy link
Copy Markdown
Contributor

@ardzoht ardzoht commented Aug 13, 2021

Signed-off-by: Alex Rodriguez [email protected]

Summary

Test Plan

  • tested locally on s1ap tester based off v1.5 and verified memory leak does not happen

Additional Information

  • This change is backwards-breaking

@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines. label Aug 13, 2021
@ardzoht ardzoht linked an issue Aug 13, 2021 that may be closed by this pull request
@magmabot magmabot added component: agw Access gateway-related issue component: cwag CWAG related issues labels Aug 13, 2021
Copy link
Copy Markdown
Contributor

@ulaskozat ulaskozat left a comment

Choose a reason for hiding this comment

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

LGTM

@themarwhal themarwhal merged commit c6d0262 into magma:v1.5 Aug 13, 2021
@ardzoht ardzoht deleted the backport_v15_free_esmmessagecontainer branch August 13, 2021 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: agw Access gateway-related issue component: cwag CWAG related issues size/XS Denotes a PR that changes 0-9 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[agw] Memory leak: decode_esm_message_container

4 participants