commit | ae4e5ed53b4bd9ccced37c165449b1c5a21edaf3 | [log] [tgz] |
---|---|---|
author | Ray Milkey <ray@opennetworking.org> | Wed Jan 17 15:24:52 2018 -0800 |
committer | Ray Milkey <ray@opennetworking.org> | Thu Jan 25 14:23:59 2018 -0800 |
tree | cfafedc21c6b7554f82d67f062ddec922aab4539 | |
parent | a5d31a613aa14db8bf01b8fcdd8427ed0cd1f878 [diff] [blame] |
Sonar suggestions - fixes to potential null pointer dereferences Change-Id: I4e350a9d72a9322971d5d4f831f1bdf167986b87
diff --git a/drivers/default/src/main/java/org/onosproject/driver/pipeline/SoftRouterPipeline.java b/drivers/default/src/main/java/org/onosproject/driver/pipeline/SoftRouterPipeline.java index 03a77ba..9ca9420 100644 --- a/drivers/default/src/main/java/org/onosproject/driver/pipeline/SoftRouterPipeline.java +++ b/drivers/default/src/main/java/org/onosproject/driver/pipeline/SoftRouterPipeline.java
@@ -270,6 +270,12 @@ } + if (v == null || e == null) { + log.warn("Soft Router Pipeline ETH_DST and/or VLAN_ID not specified"); + fail(filt, ObjectiveError.BADPARAMS); + return; + } + log.debug("Modifying Port/VLAN/MAC filtering rules in filter table: {}/{}/{}", p.port(), v.vlanId(), e.mac()); TrafficSelector.Builder selector = DefaultTrafficSelector.builder();