commit 93f59de1acb47744dda54602e4cab253625e93c5
author Kalhee Kim <Kalhee.Kim@nokia.com> Wed Oct 18 22:00:30 2017 +0000
committer Charles Chan <charles@opennetworking.org> Thu Oct 19 19:49:10 2017 +0000
tree 47fc47195eda795fe41f38ad3933e9cb6ec64c71
parent 683ba3bb3fde148b57e3059c271e5b01f876275a

[CORD-2064] When parsing InterfaceID buffer size is insufficient

Change-Id: I92966a6e59e4ba0319eef025f87193651f5918b1

utils/misc/src/main/java/org/onlab/packet/dhcp/Dhcp6InterfaceIdOption.java

diff --git a/utils/misc/src/main/java/org/onlab/packet/dhcp/Dhcp6InterfaceIdOption.java b/utils/misc/src/main/java/org/onlab/packet/dhcp/Dhcp6InterfaceIdOption.java
index 0270032..2bd47e1 100644
--- a/utils/misc/src/main/java/org/onlab/packet/dhcp/Dhcp6InterfaceIdOption.java
+++ b/utils/misc/src/main/java/org/onlab/packet/dhcp/Dhcp6InterfaceIdOption.java
@@ -31,6 +31,8 @@
  * Based on RFC-3315.
  */
 public final class Dhcp6InterfaceIdOption extends Dhcp6Option {
+    private static final short VLAN_LEN = 2;
+    private static final short SEPARATOR_LEN = 1;
     private MacAddress peerMacAddr;
     private byte[] inPort;
     private short vlanId;
@@ -138,7 +140,8 @@
                 ByteBuffer bb = ByteBuffer.wrap(optionData);
 
                 byte[] macAddr = new byte[MacAddress.MAC_ADDRESS_LENGTH];
-                byte[] port = new byte[21];
+                byte[] port = new byte[optionData.length - MacAddress.MAC_ADDRESS_LENGTH -
+                                                        VLAN_LEN - SEPARATOR_LEN * 2];
                 short vlan;
                 bb.get(macAddr);
                 bb.get();  // separator "-"