Fix issues found by FindBugs: RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE
http://findbugs.sourceforge.net/bugDescriptions.html#RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE
Note: The warning in FlowPusher.createQueue() is suppressed for now,
because it is not clear what the right solution is.
Change-Id: Id5716121692093400c9854910dfd7a61e20474e9
diff --git a/src/main/java/net/onrc/onos/apps/bgproute/BgpRoute.java b/src/main/java/net/onrc/onos/apps/bgproute/BgpRoute.java
index bb4c7d5..b766448 100644
--- a/src/main/java/net/onrc/onos/apps/bgproute/BgpRoute.java
+++ b/src/main/java/net/onrc/onos/apps/bgproute/BgpRoute.java
@@ -431,6 +431,9 @@
}
private void executeRibAdd(RibUpdate update) {
+ // TODO: Fix the code below. Note that "deviceStorage" was removed.
+
+ /*
Prefix prefix = update.getPrefix();
RibEntry rib = update.getRibEntry();
@@ -439,8 +442,6 @@
// See if we know the MAC address of the next hop
// TODO if we do not treat the next hop as a device in the future, we need to update this
- // TODO: Fix the code below after deviceStorage was removed
- /*
IDeviceObject nextHopDevice =
deviceStorage.getDeviceByIP(InetAddresses.coerceToInteger(dstIpAddress));
@@ -453,7 +454,6 @@
}
nextHopMacAddress = MACAddress.valueOf(nextHopDevice.getMACAddress());
- */
// Find the attachment point (egress interface) of the next hop
Interface egressInterface = null;
@@ -496,6 +496,7 @@
//For all prefixes we need to add the first-hop mac-rewriting flows
addPrefixFlows(prefix, egressInterface, nextHopMacAddress);
}
+ */
}
/**
@@ -685,6 +686,10 @@
* switches.
*/
private void setupFullMesh() {
+ // TODO: Fix the code below. Note that "deviceStorage" was removed.
+
+ /*
+
//For each border router, calculate and install a path from every other
//border switch to said border router. However, don't install the entry
//in to the first hop switch, as we need to install an entry to rewrite
@@ -701,9 +706,7 @@
//See if we know the MAC address of the peer. If not we can't
//do anything until we learn it
- // TODO: Fix the code below after deviceStorage was removed
MACAddress macAddress = null;
- /*
IDeviceObject nextHopDevice =
deviceStorage.getDeviceByIP(InetAddresses.coerceToInteger(peer.getIpAddress()));
@@ -716,7 +719,6 @@
}
macAddress = MACAddress.valueOf(nextHopDevice.getMACAddress());
- */
if (macAddress == null) {
log.debug("Don't know MAC for {}", peer.getIpAddress().getHostAddress());
@@ -729,6 +731,7 @@
//If we know the MAC, lets go ahead and push the paths to this peer
calculateAndPushPath(path, macAddress);
}
+ */
}
private void calculateAndPushPath(Path path, MACAddress dstMacAddress) {
diff --git a/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java b/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java
index 0bcb001..fe29efd 100644
--- a/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java
+++ b/src/main/java/net/onrc/onos/core/flowprogrammer/FlowPusher.java
@@ -18,6 +18,8 @@
import java.util.concurrent.locks.Lock;
import java.util.concurrent.locks.ReentrantLock;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
+
import net.floodlightcontroller.core.FloodlightContext;
import net.floodlightcontroller.core.IFloodlightProviderService;
import net.floodlightcontroller.core.IOFMessageListener;
@@ -572,6 +574,8 @@
}
@Override
+ @SuppressFBWarnings(value = "RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE",
+ justification = "Future versions of createQueueImpl() might return null")
public boolean createQueue(IOFSwitch sw) {
SwitchQueue queue = createQueueImpl(sw);