From 83c5c15cd38e262fe8283b4982d329e59fe87902 Mon Sep 17 00:00:00 2001 From: ISHIDA Wataru Date: Fri, 13 Feb 2015 05:58:37 +0900 Subject: [PATCH] bgp/bmp: bug fix. remove redundant construction of path attributes BGPPathAttributeNextHop and BGPPathAttributeMpReachNLRI is already in new_pathattr. so stop construct and append it to new_pathattr list. Signed-off-by: ISHIDA Wataru Signed-off-by: FUJITA Tomonori --- ryu/services/protocols/bgp/bmp.py | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/ryu/services/protocols/bgp/bmp.py b/ryu/services/protocols/bgp/bmp.py index 9af87ea3..c14988ab 100644 --- a/ryu/services/protocols/bgp/bmp.py +++ b/ryu/services/protocols/bgp/bmp.py @@ -173,18 +173,8 @@ class BMPClient(Activity): new_pathattr.append(mpunreach_attr) else: if isinstance(path, Ipv4Path): - nexthop_attr = BGPPathAttributeNextHop(path.nexthop) - new_pathattr.append(nexthop_attr) return BGPUpdate(nlri=[path.nlri], path_attributes=new_pathattr) - else: - mpnlri_attr = BGPPathAttributeMpReachNLRI( - path.route_family.afi, - path.route_family.safi, - path.nexthop, - [path.nlri] - ) - new_pathattr.append(mpnlri_attr) return BGPUpdate(path_attributes=new_pathattr)