This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-imaging.git

commit 643b283a6899105f7b76f9c0e1f26d6087f2e856
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sun Oct 6 17:03:07 2024 -0400

    Fail-fast in ByteSource constructor
    
    Comment
---
 src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java 
b/src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java
index a7dcf612..c53a61de 100644
--- a/src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java
+++ b/src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java
@@ -70,8 +70,8 @@ public class ByteSource {
     private final AbstractOrigin<?, ?> origin;
 
     ByteSource(final AbstractOrigin<?, ?> origin, final String fileName) {
-        this.fileName = fileName;
-        this.origin = origin;
+        this.origin = Objects.requireNonNull(origin, "origin");
+        this.fileName = fileName; // may be null
     }
 
     public byte[] getByteArray(final long position, final int length) throws 
IOException {

Reply via email to