objectionary#3160 copy array instead of storing directly

c71n93 · May 14, 2024 · b2a049c · b2a049c
1 parent 380976e
commit b2a049c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/eo-runtime/src/main/java/org/eolang/BytesOf.java b/eo-runtime/src/main/java/org/eolang/BytesOf.java
@@ -45,9 +45,8 @@ public final class BytesOf implements Bytes {
      * Ctor.
      * @param data Data.
      */
-    @SuppressWarnings("PMD.ArrayIsStoredDirectly")
     public BytesOf(final byte[] data) {
-        this.data = data;
+        this.data = Arrays.copyOf(data, data.length);
     }
 
     /**

diff --git a/eo-runtime/src/main/java/org/eolang/VerboseBytesAsString.java b/eo-runtime/src/main/java/org/eolang/VerboseBytesAsString.java
@@ -44,9 +44,8 @@ public final class VerboseBytesAsString implements Supplier<String> {
      * Ctor.
      * @param data Data.
      */
-    @SuppressWarnings("PMD.ArrayIsStoredDirectly")
     public VerboseBytesAsString(final byte[] data) {
-        this.data = data;
+        this.data = Arrays.copyOf(data, data.length);
     }
 
     @Override