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

bgpd: strncmp -> strcmp in community hash foo #10485

Merged
merged 1 commit into from Feb 3, 2022

Conversation

qlyoung
Copy link
Member

@qlyoung qlyoung commented Feb 2, 2022

buffers are null terminated

fixes #10484

Signed-off-by: Quentin Young qlyoung@nvidia.com

buffers are null terminated

Signed-off-by: Quentin Young <qlyoung@nvidia.com>
@frrbot frrbot bot added the bgp label Feb 2, 2022
@qlyoung qlyoung requested a review from ton31337 February 2, 2022 21:35
@NetDEF-CI
Copy link
Collaborator

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-3186/

This is a comment from an automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.

@ton31337
Copy link
Member

ton31337 commented Feb 3, 2022

@Mergifyio backport dev/8.2 stable/8.1

@mergify
Copy link

mergify bot commented Feb 3, 2022

backport dev/8.2 stable/8.1

🟠 Waiting for conditions to match

  • merged [:pushpin: backport requirement]

@mergify
Copy link

mergify bot commented Feb 3, 2022

backport dev/8.2 stable/8.1

✅ Backports have been created

ton31337 added a commit that referenced this pull request Feb 3, 2022
bgpd: strncmp -> strcmp in community hash foo (backport #10485)
donaldsharp added a commit that referenced this pull request Feb 4, 2022
bgpd: strncmp -> strcmp in community hash foo (backport #10485)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

strncmp warnings in bgp_community_alias.c
5 participants