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

Avoid removing a VRF routing table when there are pending creation en… #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yuazhe
Copy link
Owner

@yuazhe yuazhe commented Feb 14, 2025

…tries in gRouteBulker (sonic-net#3477)

  • Set bytes to 0 before copying IPv4 addresses/masks to avoid uninitialized bytes
  • Avoid removing a VRF routing table when there are pending creation entries in gRouteBulker

Remove a VRF routing table when a routing entry is removed only if there is no pending creation entry in gRouteBulker Avoid uninitialized value SAI IP address/prefix structure Why I did it

Fix issue: out of range exception can be thrown in addRoutePost due to non exist VRF

What I did

Why I did it

How I verified it

Details if related

…tries in gRouteBulker (sonic-net#3477)

* Set bytes to 0 before copying IPv4 addresses/masks to avoid uninitialized bytes
* Avoid removing a VRF routing table when there are pending creation entries in gRouteBulker

Remove a VRF routing table when a routing entry is removed only if there is no pending creation entry in gRouteBulker
Avoid uninitialized value SAI IP address/prefix structure
Why I did it

Fix issue: out of range exception can be thrown in addRoutePost due to non exist VRF
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