From 808331b0f4208c15cffbd8ea235c35a79e37e5a0 Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori Date: Tue, 10 May 2016 16:55:35 +0900 Subject: [PATCH] docs: add comment for dubious code [ci skip] Signed-off-by: FUJITA Tomonori --- server/peer.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/server/peer.go b/server/peer.go index 8d517dee..4ed46bbf 100644 --- a/server/peer.go +++ b/server/peer.go @@ -169,6 +169,13 @@ func (peer *Peer) processOutgoingPaths(paths, withdrawals []*table.Path) []*tabl } outgoing := make([]*table.Path, 0, len(paths)) + // Note: multiple paths having the same prefix could exist the + // withdrawals list in the case of Route Server setup with + // import policies modifying paths. In such case, gobgp sends + // duplicated update messages; withdraw messages for the same + // prefix. + // However, currently we don't support local path for Route + // Server setup so this is NOT the case. for _, path := range withdrawals { if path.IsLocal() { if _, ok := peer.fsm.rfMap[path.GetRouteFamily()]; ok {