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 888ede979fe9d784ca9f5fc6d90f22d54783775a Author: Gary Gregory <[email protected]> AuthorDate: Tue Jul 4 09:29:06 2023 -0400 Reuse IOUtils --- src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java | 5 +++-- 1 file changed, 3 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 f3e65796..4aeee8c8 100644 --- a/src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java +++ b/src/main/java/org/apache/commons/imaging/bytesource/ByteSource.java @@ -23,6 +23,7 @@ import java.nio.file.Path; import java.util.Objects; import org.apache.commons.imaging.common.BinaryFunctions; +import org.apache.commons.io.IOUtils; import org.apache.commons.io.build.AbstractOrigin; import org.apache.commons.io.build.AbstractOrigin.ByteArrayOrigin; import org.apache.commons.io.build.AbstractOrigin.FileOrigin; @@ -78,8 +79,8 @@ public class ByteSource { BinaryFunctions.skipBytes(is, start); succeeded = true; } finally { - if (!succeeded && is != null) { - is.close(); + if (!succeeded) { + IOUtils.close(is); } } return is;
