commit 9281593e8bc6c0f7875c6192bf2551ab99fa10a5
author Ray Milkey <ray@opennetworking.org> Tue Aug 28 10:32:17 2018 -0700
committer Thomas Vachuska <tom@opennetworking.org> Tue Aug 28 18:24:28 2018 +0000
tree de92ef6fd99aba887be357d95a67828233603f39
parent 0c19e650dd08c89402700fb93685e4e54fa8c7a1

Fix Sonar reported potential null pointer dereference

Change-Id: I8e9282daabd676434e60ad0a8c25c4f5e763fa18

apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/impl/OpenstackSwitchingDhcpHandler.java

diff --git a/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/impl/OpenstackSwitchingDhcpHandler.java b/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/impl/OpenstackSwitchingDhcpHandler.java
index b3fb034..32c1ad9 100644
--- a/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/impl/OpenstackSwitchingDhcpHandler.java
+++ b/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/impl/OpenstackSwitchingDhcpHandler.java
@@ -285,6 +285,7 @@
             if (osPort == null) {
                 log.error("Failed to retrieve openstack port information for instance port {}",
                         reqInstPort.toString());
+                return null;
             }
             // pick one IP address to make a reply
             IP fixedIp = osPort.getFixedIps().stream().findFirst().get();