Improve the resiliency of the packet deserialization code.
Packet deserializers now check for malformed input while reading the byte
stream. Deserializers are re-implemented as functions that take a byte array
and return a packet object. The old IPacket.deserialize(...) methods have been
deprecated with the goal of eventually moving to immutable packet objects.
Unit tests have been implemented for all Deserializer functions.
ONOS-1589
Change-Id: I9073d5e6e7991e15d43830cfd810989256b71c56
diff --git a/utils/misc/src/test/java/org/onlab/packet/ipv6/BaseOptionsTest.java b/utils/misc/src/test/java/org/onlab/packet/ipv6/BaseOptionsTest.java
index 885b4af..bb91e9e 100644
--- a/utils/misc/src/test/java/org/onlab/packet/ipv6/BaseOptionsTest.java
+++ b/utils/misc/src/test/java/org/onlab/packet/ipv6/BaseOptionsTest.java
@@ -16,9 +16,11 @@
package org.onlab.packet.ipv6;
+import org.junit.Before;
import org.junit.BeforeClass;
import org.junit.Test;
import org.onlab.packet.Data;
+import org.onlab.packet.Deserializer;
import org.onlab.packet.IPv6;
import org.onlab.packet.UDP;
@@ -40,6 +42,8 @@
};
private static byte[] bytePacket;
+ private Deserializer<BaseOptions> deserializer;
+
@BeforeClass
public static void setUpBeforeClass() throws Exception {
data = new Data();
@@ -57,6 +61,11 @@
System.arraycopy(bytePayload, 0, bytePacket, byteHeader.length, bytePayload.length);
}
+ @Before
+ public void setUp() {
+ deserializer = BaseOptions.deserializer();
+ }
+
/**
* Tests serialize and setters.
*/
@@ -75,9 +84,8 @@
* Tests deserialize and getters.
*/
@Test
- public void testDeserialize() {
- BaseOptions baseopt = new BaseOptions();
- baseopt.deserialize(bytePacket, 0, bytePacket.length);
+ public void testDeserialize() throws Exception {
+ BaseOptions baseopt = deserializer.deserialize(bytePacket, 0, bytePacket.length);
assertThat(baseopt.getNextHeader(), is((byte) 0x11));
assertThat(baseopt.getHeaderExtLength(), is((byte) 0x00));