commit 8bf12805a634d9832cac7616086beb1cf01f8ec9
author Ray Milkey <ray@opennetworking.org> Fri Aug 03 16:55:18 2018 +0000
committer Ray Milkey <ray@opennetworking.org> Fri Aug 03 17:14:16 2018 +0000
tree 875950883ebece608c06caf512410e389c5ee7c2
parent 34328629697b0a3840a5336f4d794170acc36afd

Revert "Fixing errorprone's warning."

This reverts commit c5836479cf04a23bc24f097ea606153b572eb76b.

Change-Id: Ia662d4b35a7d063b45876315d8bf97444ca41c5b

utils/misc/src/main/java/org/onlab/packet/ICMPEcho.java

diff --git a/utils/misc/src/main/java/org/onlab/packet/ICMPEcho.java b/utils/misc/src/main/java/org/onlab/packet/ICMPEcho.java
index ff0a76a..aa06cc6 100644
--- a/utils/misc/src/main/java/org/onlab/packet/ICMPEcho.java
+++ b/utils/misc/src/main/java/org/onlab/packet/ICMPEcho.java
@@ -19,6 +19,7 @@
 
 import static org.onlab.packet.PacketUtils.checkInput;
 import static com.google.common.base.MoreObjects.toStringHelper;
+import static com.google.common.base.Preconditions.checkNotNull;
 
 /**
  * ICMP packet class for echo purpose.
@@ -76,6 +77,9 @@
      */
     @Override
     public byte[] serialize() {
+        checkNotNull(this.identifier);
+        checkNotNull(this.sequenceNum);
+
         int length = ICMP_ECHO_HEADER_LENGTH;
         byte[] payloadData = null;
         if (this.payload != null) {