commit 93ba53cd592157c67f860a6a0bdd173a63416f06
author Yi Tseng <yi@opennetworking.org> Thu Sep 14 13:24:21 2017 -0700
committer Yi Tseng <yi@opennetworking.org> Thu Sep 14 13:24:21 2017 -0700
tree dd3979cf00aed1015bf706581ce4c0bb9c01a275
parent f50a1775a74615bbfbb1acfe842a61c63faf9c3e

[CORD-1715] The dest udp port for DHCP reply packets should be 68

Change-Id: I57ca054b452fda7e80da3c2c46e65fe14efd25d8

apps/dhcprelay/src/main/java/org/onosproject/dhcprelay/Dhcp4HandlerImpl.java

diff --git a/apps/dhcprelay/src/main/java/org/onosproject/dhcprelay/Dhcp4HandlerImpl.java b/apps/dhcprelay/src/main/java/org/onosproject/dhcprelay/Dhcp4HandlerImpl.java
index 39e53ff..2ef43b7 100644
--- a/apps/dhcprelay/src/main/java/org/onosproject/dhcprelay/Dhcp4HandlerImpl.java
+++ b/apps/dhcprelay/src/main/java/org/onosproject/dhcprelay/Dhcp4HandlerImpl.java
@@ -722,7 +722,10 @@
             udpPacket.setDestinationPort(UDP.DHCP_CLIENT_PORT);
         } else {
             // forward to another dhcp relay
-            udpPacket.setDestinationPort(UDP.DHCP_SERVER_PORT);
+            // FIXME: Currently we assume the DHCP comes from a L2 relay with
+            // Option 82, this might not work if DHCP message comes from
+            // L3 relay.
+            udpPacket.setDestinationPort(UDP.DHCP_CLIENT_PORT);
         }
 
         udpPacket.setPayload(dhcpPayload);