fix: move back positivity check to helper method

Author: ppkarwasz

src/main/java/org/apache/commons/io/IOUtils.java

@@ -2684,13 +2684,8 @@ public static byte[] toByteArray(final InputStream inputStream) throws IOExcepti
      * @throws NullPointerException     if {@code input} is {@code null}.
      * @since 2.1
      */
-    @SuppressWarnings("resource")
     public static byte[] toByteArray(final InputStream input, final int size) throws IOException {
-        Objects.requireNonNull(input, "input");
-        if (size < 0) {
-            throw new IllegalArgumentException("Size must be equal or greater than zero: " + size);
-        }
-        return toByteArray(input::read, size);
+        return toByteArray(Objects.requireNonNull(input, "input")::read, size);
     }
 
     /**
@@ -2790,6 +2785,9 @@ public static byte[] toByteArray(final InputStream input, final int size, final
      * @throws IllegalArgumentException if {@code size} is less than zero.
      */
     static byte[] toByteArray(final IOTriFunction<byte[], Integer, Integer, Integer> input, final int size) throws IOException {
+        if (size < 0) {
+            throw new IllegalArgumentException("Size must be equal or greater than zero: " + size);
+        }
         if (size == 0) {
             return EMPTY_BYTE_ARRAY;
         }

src/main/java/org/apache/commons/io/RandomAccessFiles.java

@@ -76,11 +76,7 @@ private static long length(final RandomAccessFile raf) throws IOException {
      *                     other I/O error occurs.
      */
     public static byte[] read(final RandomAccessFile input, final long position, final int length) throws IOException {
-        Objects.requireNonNull(input, "input");
         input.seek(position);
-        if (length < 0) {
-            throw new IllegalArgumentException("Size must be equal or greater than zero: " + length);
-        }
         return IOUtils.toByteArray(input::read, length);
     }