Skip to content

Commit 634c76c

Browse files
authored
RIS: Accept post-policy messages only (#378)
1 parent d2e8978 commit 634c76c

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

protocols/bgp/server/bmp_router.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -210,6 +210,10 @@ func (r *Router) processMsg(msg []byte) {
210210
func (r *Router) processRouteMonitoringMsg(msg *bmppkt.RouteMonitoringMsg) {
211211
atomic.AddUint64(&r.counters.routeMonitoringMessages, 1)
212212

213+
if !msg.PerPeerHeader.GetLFlag() { // we're only interested in post-policy routes
214+
return
215+
}
216+
213217
n := r.neighborManager.getNeighbor(msg.PerPeerHeader.PeerDistinguisher, msg.PerPeerHeader.PeerAddress)
214218
if n == nil {
215219
log.Errorf("Received route monitoring message for non-existent neighbor %d/%v on %s", msg.PerPeerHeader.PeerDistinguisher, msg.PerPeerHeader.PeerAddress, r.address.String())

protocols/bgp/server/bmp_server_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ func TestBMPServer(t *testing.T) {
227227
0, // Msg Type (route monitoring)
228228

229229
0, // Peer Type (global instance peer)
230-
0b00100000, // Peer Flags
230+
0b01100000, // Peer Flags
231231
0, 0, 0, 0, 0, 0, 0, 123, // Peer Distinguisher
232232
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 10, 1, 1, 1, // Peer Address (10.1.1.1)
233233
0, 0, 0, 200, // Peer AS = 200
@@ -318,7 +318,7 @@ func TestBMPServer(t *testing.T) {
318318
0, // Msg Type (route monitoring)
319319

320320
0, // Peer Type (global instance peer)
321-
0b00100000, // Peer Flags
321+
0b01100000, // Peer Flags
322322
0, 0, 0, 0, 0, 0, 0, 123, // Peer Distinguisher
323323
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 10, 1, 2, 1, // Peer Address (10.1.2.1)
324324
0, 0, 0, 222, // Peer AS = 222

0 commit comments

Comments
 (0)