diff options
| author | Donatas Abraitis <donatas.abraitis@gmail.com> | 2021-10-17 23:23:28 +0300 |
|---|---|---|
| committer | FUJITA Tomonori <fujita.tomonori@gmail.com> | 2021-10-18 16:58:11 +0900 |
| commit | d3ee05ad24192ff0f2b876532fb6fd9776d540dd (patch) | |
| tree | cbbea8463ce5f4a8c66e5517c42066c1bac347ac /pkg/packet | |
| parent | 313b5cf02cb225204372ced27dea8ab3b2fd3315 (diff) | |
bgp: Validate router-id against 0.0.0.0 and ASN-wide
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
Diffstat (limited to 'pkg/packet')
| -rw-r--r-- | pkg/packet/bgp/validate.go | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/pkg/packet/bgp/validate.go b/pkg/packet/bgp/validate.go index f14458e7..34a7eb6a 100644 --- a/pkg/packet/bgp/validate.go +++ b/pkg/packet/bgp/validate.go @@ -313,7 +313,7 @@ func ValidateBGPMessage(m *BGPMessage) error { return nil } -func ValidateOpenMsg(m *BGPOpen, expectedAS uint32) (uint32, error) { +func ValidateOpenMsg(m *BGPOpen, expectedAS uint32, myAS uint32, myId net.IP) (uint32, error) { if m.Version != 4 { return 0, NewMessageError(BGP_ERROR_OPEN_MESSAGE_ERROR, BGP_ERROR_SUB_UNSUPPORTED_VERSION_NUMBER, nil, fmt.Sprintf("unsupported version %d", m.Version)) } @@ -331,6 +331,20 @@ func ValidateOpenMsg(m *BGPOpen, expectedAS uint32) (uint32, error) { } } } + + // rfc6286 (Autonomous-System-Wide Unique BGP Identifier for BGP-4) + // If the BGP Identifier field of the OPEN message is zero, or if it + // is the same as the BGP Identifier of the local BGP speaker and the + // message is from an internal peer, then the Error Subcode is set to + // "Bad BGP Identifier". + routerId := m.ID + if routerId.IsUnspecified() { + return 0, NewMessageError(BGP_ERROR_OPEN_MESSAGE_ERROR, BGP_ERROR_SUB_BAD_BGP_IDENTIFIER, nil, fmt.Sprintf("bad BGP identifier %s (0.0.0.0)", routerId.String())) + } + if as == myAS && routerId.Equal(myId) { + return 0, NewMessageError(BGP_ERROR_OPEN_MESSAGE_ERROR, BGP_ERROR_SUB_BAD_BGP_IDENTIFIER, nil, fmt.Sprintf("bad BGP identifier %s", routerId.String())) + } + if expectedAS != 0 && as != expectedAS { return 0, NewMessageError(BGP_ERROR_OPEN_MESSAGE_ERROR, BGP_ERROR_SUB_BAD_PEER_AS, nil, fmt.Sprintf("as number mismatch expected %d, received %d", expectedAS, as)) } |