Skip to content

zebra: Fix memory leak when SRv6 dynamic SID allocation fails#20445

Merged
Jafaral merged 1 commit intoFRRouting:masterfrom
cscarpitta:fix/fix_srv6_dynamic_alloc_memory_leak
Jan 12, 2026
Merged

zebra: Fix memory leak when SRv6 dynamic SID allocation fails#20445
Jafaral merged 1 commit intoFRRouting:masterfrom
cscarpitta:fix/fix_srv6_dynamic_alloc_memory_leak

Conversation

@cscarpitta
Copy link
Contributor

Ensure that the zebra_srv6_sid_ctx object is properly freed if zebra_srv6_sid_alloc fails, to prevent memory leaks in the SRv6 dynamic SID allocation code path.

Ensure that the zebra_srv6_sid_ctx object is properly freed if
zebra_srv6_sid_alloc fails, to prevent memory leaks in the SRv6
dynamic SID allocation code path.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
@frrbot frrbot bot added the zebra label Jan 12, 2026
@cscarpitta cscarpitta changed the title zebra: Fix memory leak when SRv6 dyanmic SID allocation fails zebra: Fix memory leak when SRv6 dynamic SID allocation fails Jan 12, 2026
@cscarpitta
Copy link
Contributor Author

@Mergifyio backport stable/10.5 stable/10.4 stable/10.3 stable/10.2

@mergify
Copy link

mergify bot commented Jan 12, 2026

backport stable/10.5 stable/10.4 stable/10.3 stable/10.2

✅ Backports have been created

Details

@Jafaral Jafaral merged commit 7897136 into FRRouting:master Jan 12, 2026
22 of 26 checks passed
Jafaral added a commit that referenced this pull request Jan 12, 2026
zebra: Fix memory leak when SRv6 dynamic SID allocation fails (backport #20445)
Jafaral added a commit that referenced this pull request Jan 12, 2026
zebra: Fix memory leak when SRv6 dynamic SID allocation fails (backport #20445)
ton31337 added a commit that referenced this pull request Jan 16, 2026
zebra: Fix memory leak when SRv6 dynamic SID allocation fails (backport #20445)
ton31337 added a commit that referenced this pull request Jan 16, 2026
zebra: Fix memory leak when SRv6 dynamic SID allocation fails (backport #20445)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants