From a388520664ac7994606e05460708539d6e63431c Mon Sep 17 00:00:00 2001
From: Srinivasan Ramasubramanian <srini@bigswitch.com>
Date: Sun, 22 Jul 2012 11:27:41 -0700
Subject: [PATCH] Bug fix: prevCluster assignment is changed to reflect the L2
domain ID of the prev entity. It was possible for cur and prev to be swapped
when duplicate attachment points were detected, where the prevCluster could
have the wrong assignment.
---
.../net/floodlightcontroller/devicemanager/internal/Device.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/main/java/net/floodlightcontroller/devicemanager/internal/Device.java b/src/main/java/net/floodlightcontroller/devicemanager/internal/Device.java
index 18afaee39..dc7c0da7b 100755
--- a/src/main/java/net/floodlightcontroller/devicemanager/internal/Device.java
+++ b/src/main/java/net/floodlightcontroller/devicemanager/internal/Device.java
@@ -311,7 +311,7 @@ public class Device implements IDevice {
}
prev = clentities[clEntIndex] = cur;
- prevCluster = curCluster;
+ prevCluster = topology.getL2DomainId(prev.getSwitchDPID());
long prevLastSeen =
deviceManager.apComparator.
--
GitLab