Skip to content

bgpd: Ensure that bgp open message stream has enough data to read (backport #12247)#12250

Merged
Jafaral merged 1 commit intostable/8.3from
mergify/bp/stable/8.3/pr-12247
Nov 3, 2022
Merged

bgpd: Ensure that bgp open message stream has enough data to read (backport #12247)#12250
Jafaral merged 1 commit intostable/8.3from
mergify/bp/stable/8.3/pr-12247

Conversation

@mergify
Copy link

@mergify mergify bot commented Nov 2, 2022

This is an automatic backport of pull request #12247 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

If a operator receives an invalid packet that is of insufficient size
then it is possible for BGP to assert during reading of the packet
instead of gracefully resetting the connection with the peer.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
(cherry picked from commit 766eec1)
@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-8181/

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.

@Jafaral Jafaral merged commit 6ddd99b into stable/8.3 Nov 3, 2022
@mergify mergify bot deleted the mergify/bp/stable/8.3/pr-12247 branch November 3, 2022 03:37
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.

3 participants