From 2e3eef3e811940b6a8c877e60ac0520a814f87bb Mon Sep 17 00:00:00 2001
From: Jonathan Hart <jono@onlab.us>
Date: Wed, 12 Nov 2014 11:05:40 -0800
Subject: [PATCH] Compare ConnectPoints rather than Interfaces.

Now we can have multiple Interfaces per port, and the old code results in an
intent using the same port for ingress and egress.

Change-Id: I9f9ad396a0c6e7d31d654896f31c59a34240f82f
---
 apps/sdnip/src/main/java/org/onlab/onos/sdnip/Router.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/sdnip/src/main/java/org/onlab/onos/sdnip/Router.java b/apps/sdnip/src/main/java/org/onlab/onos/sdnip/Router.java
index cffdc09f21..3ae5b82e33 100644
--- a/apps/sdnip/src/main/java/org/onlab/onos/sdnip/Router.java
+++ b/apps/sdnip/src/main/java/org/onlab/onos/sdnip/Router.java
@@ -598,7 +598,7 @@ public class Router implements RouteListener {
         Set<ConnectPoint> ingressPorts = new HashSet<>();
 
         for (Interface intf : interfaceService.getInterfaces()) {
-            if (!intf.equals(egressInterface)) {
+            if (!intf.connectPoint().equals(egressInterface.connectPoint())) {
                 ConnectPoint srcPort = intf.connectPoint();
                 ingressPorts.add(srcPort);
             }