commit 3febe367d56ef3c89f07844b26037241d85be6f1
author Nandakumar Vadivelu <nanda@apache.org> Thu Dec 14 00:07:07 2023 +0530
committer GitHub <noreply@github.com> Wed Dec 13 10:37:07 2023 -0800
tree 2c58052eda66d8a173cbf2b0306fd8f90fd9f09e
parent 894403557c776649a8eea2b6f39e987c5c748636

RATIS-1648. Fix EI_EXPOSE_REP in MD5Hash. (#992)

ratis-common/src/main/java/org/apache/ratis/io/MD5Hash.java

diff --git a/ratis-common/src/main/java/org/apache/ratis/io/MD5Hash.java b/ratis-common/src/main/java/org/apache/ratis/io/MD5Hash.java
index 76be2d5..e60bef9 100644
--- a/ratis-common/src/main/java/org/apache/ratis/io/MD5Hash.java
+++ b/ratis-common/src/main/java/org/apache/ratis/io/MD5Hash.java
@@ -18,8 +18,6 @@
 
 package org.apache.ratis.io;
 
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
-
 import java.io.DataInput;
 import java.io.DataOutput;
 import java.io.IOException;
@@ -53,12 +51,11 @@
   }
 
   /** Constructs an MD5Hash with a specified value. */
-  @SuppressFBWarnings("EI_EXPOSE_REP2")
   public MD5Hash(byte[] digest) {
     if (digest.length != MD5_LEN) {
       throw new IllegalArgumentException("Wrong length: " + digest.length);
     }
-    this.digest = digest;
+    this.digest = digest.clone();
   }
 
   public void readFields(DataInput in) throws IOException {
@@ -82,9 +79,8 @@
   }
 
   /** Returns the digest bytes. */
-  @SuppressFBWarnings("EI_EXPOSE_REP")
   public byte[] getDigest() {
-    return digest;
+    return digest.clone();
   }
 
   /** Construct a hash value for a byte array. */