commit 6db55b971ef299b83adab0396ee2e92fad5d3945
author Charles Chan <rascov@gmail.com> Mon Sep 11 15:21:57 2017 -0700
committer Ray Milkey <ray@opennetworking.org> Tue Sep 12 22:41:13 2017 +0000
tree ba8528d12f7c0b5533c34723852f0b7a7d5356cf
parent 910be6acc35bbdc23ea6bb20b9a47c514041f4f4

Fix NoSuchElementException

Change-Id: I2c80400d10dffd7dfda777de5279135ae687af35

src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java

diff --git a/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java b/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
index 0d263e7..709d05e 100644
--- a/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
+++ b/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
@@ -244,6 +244,11 @@
      */
     // XXX refactor
     protected void populateSubnet(Set<ConnectPoint> cpts, Set<IpPrefix> subnets) {
+        if (cpts == null || cpts.size() < 1 || cpts.size() > 2) {
+            log.warn("Skipping populateSubnet due to illegal size of connect points. {}", cpts);
+            return;
+        }
+
         lastRoutingChange = DateTime.now();
         statusLock.lock();
         try {