Fix bug in fwd app
The pickForwardPathIfPossible function will not return null even if no desired path is found, this make the backTrackBadNodes function work incorrectly, result in the failover mechanism doesn't work
Change-Id: I510e4a3d6c0c99c68bf24b58a8d523dd70a1b2c6
diff --git a/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java b/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java
index 8eac83a..8ed55c6 100644
--- a/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java
+++ b/apps/fwd/src/main/java/org/onosproject/fwd/ReactiveForwarding.java
@@ -539,14 +539,12 @@
// Selects a path from the given set that does not lead back to the
// specified port if possible.
private Path pickForwardPathIfPossible(Set<Path> paths, PortNumber notToPort) {
- Path lastPath = null;
for (Path path : paths) {
- lastPath = path;
if (!path.src().port().equals(notToPort)) {
return path;
}
}
- return lastPath;
+ return null;
}
// Floods the specified packet if permissible.