commit faf79321c0253bf2b058a7ad1e23e328852a55b3
author David K. Bainbridge <dbainbri@ciena.com> Thu Jan 23 10:15:06 2020 -0800
committer David K. Bainbridge <dbainbri@ciena.com> Thu Jan 23 10:15:11 2020 -0800
tree 427afcdb8fb72c3aece9e0eb99a6518c84efb6c7
parent 2a597c48855dea5f27f24af1f3d2c67c34d0f19f

pass a nullable optional value instead of null as it is dereferenced in the called function without a null check

Change-Id: I723cfd469de3e3d5392445371b7736d0d59c31bf

app/src/main/java/org/opencord/dhcpl2relay/impl/DhcpL2Relay.java

diff --git a/app/src/main/java/org/opencord/dhcpl2relay/impl/DhcpL2Relay.java b/app/src/main/java/org/opencord/dhcpl2relay/impl/DhcpL2Relay.java
index 39e1cb3..5a1266d 100755
--- a/app/src/main/java/org/opencord/dhcpl2relay/impl/DhcpL2Relay.java
+++ b/app/src/main/java/org/opencord/dhcpl2relay/impl/DhcpL2Relay.java
@@ -1131,7 +1131,7 @@
                         if (useOltUplink && isUplinkPortOfOlt(event.subject().id(), event.port())) {
                             requestDhcpPacketsFromConnectPoint(
                                 new ConnectPoint(event.subject().id(), event.port().number()),
-                                null);
+                                        Optional.ofNullable(null));
                         }
                         break;
                     default: