From 0f1503f59962293a5bcdbe1e2a6adfc9b8afa1a6 Mon Sep 17 00:00:00 2001 From: Gowri R Date: Thu, 21 Sep 2017 17:20:39 +0530 Subject: [PATCH] Bug 9196 - Maxpath value should be between 1 to 64 in BGP multipath Adding a check to ensure maxpath value for BGP multipath is in the range 1 to 64 Change-Id: I03786726053672d04b8ef552cb16cc45a9c4a840 Signed-off-by: Gowri R --- .../opendaylight/netvirt/bgpmanager/commands/Multipath.java | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/commands/Multipath.java b/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/commands/Multipath.java index 45820df99a..09019f6dc2 100644 --- a/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/commands/Multipath.java +++ b/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/commands/Multipath.java @@ -60,6 +60,7 @@ public class Multipath extends OsgiCommandSupport { af_afi afi = null; af_safi safi = null; + int imaxpath = maxpath == null ? -1 : Integer.parseInt(maxpath); if (addrFamily != null) { if (!addrFamily.equals("lu")) { @@ -72,6 +73,11 @@ public class Multipath extends OsgiCommandSupport { safi = af_safi.SAFI_MPLS_VPN; } + if (imaxpath < 1 && imaxpath > 64) { + session.getConsole().println("error: " + MAXPATH + " must be between 1 and 64"); + return null; + } + if (multipathEnable != null) { switch (multipathEnable) { -- 2.36.6