commit | f6911bde9b53b15ba6d6f51526fadafe8dde3c7d | [log] [tgz] |
---|---|---|
author | Ray Milkey <ray@opennetworking.org> | Tue Jul 10 08:50:10 2018 -0700 |
committer | Ray Milkey <ray@opennetworking.org> | Tue Jul 10 16:13:03 2018 +0000 |
tree | ce74512b41f69ceb5e3be5618d1935b3cdb9db25 | |
parent | 1ae26c6c2812ce247f608c0aa782015b361ffd76 [diff] |
Fix Sonar warnings about NPEs after an exception Change-Id: Ic6ddfe4cef6a1e1df32b896a2dfdbc6053d0a4eb
diff --git a/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackAddAclCommand.java b/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackAddAclCommand.java index 9610603..1b9a707 100644 --- a/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackAddAclCommand.java +++ b/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackAddAclCommand.java
@@ -76,6 +76,7 @@ dstIpAddress = IpAddress.valueOf(dstIpStr); } catch (IllegalArgumentException e) { log.error("IllegalArgumentException occurred because of {}", e.toString()); + return; } TrafficSelector.Builder sBuilder = DefaultTrafficSelector.builder()
diff --git a/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackRemoveAclCommand.java b/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackRemoveAclCommand.java index 47dc137..cba0c7a 100644 --- a/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackRemoveAclCommand.java +++ b/apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/cli/OpenstackRemoveAclCommand.java
@@ -73,6 +73,7 @@ dstIpAddress = IpAddress.valueOf(dstIpStr); } catch (IllegalArgumentException e) { log.error("IllegalArgumentException occurred because of {}", e.toString()); + return; } TrafficSelector.Builder sBuilder = DefaultTrafficSelector.builder()